Skip to content

Add comprehensive Python testing infrastructure #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

llbbl
Copy link

@llbbl llbbl commented Jun 6, 2025

Summary

  • Set up Poetry as the package manager with comprehensive dependency management
  • Added complete pytest testing infrastructure with coverage reporting (80% threshold)
  • Created organized test directory structure with unit and integration test separation
  • Implemented comprehensive shared fixtures for NeRF-specific testing scenarios
  • Added validation tests to verify the testing infrastructure functionality

Test plan

  • Install dependencies with Poetry (poetry install)
  • Run all tests (poetry run pytest)
  • Verify unit tests work (poetry run pytest -m unit)
  • Verify integration tests work (poetry run pytest -m integration)
  • Confirm coverage reporting generates HTML and XML outputs
  • Test shared fixtures for rays, points, configs, and mock models
  • Validate pytest markers (unit, integration, slow) function correctly

🤖 Generated with Claude Code

- Set up Poetry package manager with pyproject.toml configuration
- Added testing dependencies: pytest, pytest-cov, pytest-mock
- Configured pytest with coverage reporting (80% threshold)
- Created tests/ directory structure with unit/ and integration/ subdirs
- Added conftest.py with comprehensive shared fixtures for NeRF testing
- Created validation tests to verify infrastructure functionality
- Updated .gitignore with IDE-specific entries
- Supports test markers: unit, integration, slow
- Coverage reports in HTML, XML, and terminal formats

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant