feat: Set up comprehensive Python testing infrastructure with Poetry#58
Open
llbbl wants to merge 1 commit intogoogle-research:masterfrom
Open
feat: Set up comprehensive Python testing infrastructure with Poetry#58llbbl wants to merge 1 commit intogoogle-research:masterfrom
llbbl wants to merge 1 commit intogoogle-research:masterfrom
Conversation
- Add Poetry package manager configuration with pyproject.toml - Configure pytest with coverage reporting (80% threshold) - Create testing directory structure (unit/integration) - Add development dependencies: pytest, pytest-cov, pytest-mock - Configure custom test markers (unit, integration, slow) - Set up shared fixtures in conftest.py - Update .gitignore with testing and Claude entries - Add validation tests to verify infrastructure
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Set up Python Testing Infrastructure
Summary
This PR establishes a comprehensive testing infrastructure for the batch-ppo project using Poetry as the package manager and pytest as the testing framework. The setup provides a solid foundation for writing and maintaining tests with proper coverage reporting and development tooling.
Changes Made
Package Management
pyproject.tomlsetup.pyTesting Configuration
test_*.pyand*_test.py@pytest.mark.unit,@pytest.mark.integration,@pytest.mark.slowDirectory Structure
Shared Fixtures (in conftest.py)
temp_dir: Temporary directory with automatic cleanupmock_config: Sample configuration dictionarymock_config_file: Temporary YAML config filetf_session: TensorFlow session managementmock_environment: Mock Gym environmentsample_trajectory: Sample training datacleanup_tensorflow: Automatic TF state cleanupcapture_stdout: Stdout capture for testing print statementsDevelopment Commands
Both commands are configured to run the test suite:
poetry run testpoetry run testsAdditional Updates
.gitignorewith:How to Use
Install Poetry (if not already installed):
curl -sSL https://install.python-poetry.org | python3 -Install dependencies:
Run tests:
Run specific test categories:
View coverage report:
htmlcov/index.htmlin browsercoverage.xmlfor CI integrationNotes
-vfor verbose,-xto stop on first failure)Next Steps
With this infrastructure in place, developers can now:
tests/unit/tests/integration/