feat: Add comprehensive Python testing infrastructure with Poetry#67
Open
llbbl wants to merge 1 commit intoBMW-InnovationLab:masterfrom
Open
feat: Add comprehensive Python testing infrastructure with Poetry#67llbbl wants to merge 1 commit intoBMW-InnovationLab:masterfrom
llbbl wants to merge 1 commit intoBMW-InnovationLab:masterfrom
Conversation
- Set up Poetry package manager for dependency management - Add pytest, pytest-cov, and pytest-mock as dev dependencies - Configure pytest with coverage thresholds and custom markers - Create test directory structure (unit/integration) - Add shared fixtures in conftest.py for common test needs - Configure test commands via Poetry scripts - Update .gitignore with testing and Claude-specific entries - Add validation tests to verify infrastructure setup
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.
Add Python Testing Infrastructure
Summary
This PR sets up a comprehensive testing infrastructure for the BMW LabelTool Lite project using Poetry as the package manager and pytest as the testing framework.
Changes Made
Package Management
pyproject.tomlwith Poetry configuration as the project's package managerpytest ^7.4.4- Main testing frameworkpytest-cov ^4.1.0- Coverage reporting pluginpytest-mock ^3.12.0- Mocking utilitiesTesting Configuration
Pytest Configuration: Configured in
pyproject.tomlwith:unit,integration,slowCoverage Configuration: Set up comprehensive coverage tracking:
docker_sdk_api,inference_api,training_apiDirectory Structure
Shared Fixtures (conftest.py)
Created comprehensive fixtures for common testing needs:
temp_dir- Temporary directory managementmock_config- Configuration mockingmock_logger- Logger mockingsample_image_path,sample_model_path- File fixturesmock_request,mock_response- HTTP mockingsample_training_data,sample_inference_data- Data fixturesmock_docker_client- Docker client mockingDevelopment Commands
Configured Poetry scripts for running tests:
All standard pytest options are available (e.g.,
-v,-k,-m,--no-cov)Additional Setup
Updated .gitignore: Added comprehensive Python-specific entries including:
.pytest_cache/,coverage.xml,htmlcov/).claude/*)Validation Tests: Created
test_infrastructure_validation.pyto verify:How to Use
Install Poetry (if not already installed):
curl -sSL https://install.python-poetry.org | python3 -Install dependencies:
Run tests:
View coverage reports:
htmlcov/index.htmlin a browsercoverage.xmlfor CI integrationNotes
poetry.lock) should be committed to ensure reproducible builds