feat: Set up comprehensive Python testing infrastructure#21
Open
llbbl wants to merge 1 commit intomenahishayan:masterfrom
Open
feat: Set up comprehensive Python testing infrastructure#21llbbl wants to merge 1 commit intomenahishayan:masterfrom
llbbl wants to merge 1 commit intomenahishayan:masterfrom
Conversation
- Migrate from setup.py to Poetry package management - Add pytest, pytest-cov, and pytest-mock as test dependencies - Configure comprehensive testing settings with 80% coverage threshold - Create tests/ directory structure with unit/integration subdirectories - Add shared pytest fixtures for common testing scenarios - Update .gitignore with testing and development entries - Include 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.
Set up Python Testing Infrastructure
Summary
This PR establishes a comprehensive testing infrastructure for the HomeScript project, migrating from the legacy setup.py configuration to a modern Poetry-based development environment.
Changes Made
Package Management Migration
pyproject.tomlwith complete project metadataTesting Configuration
unit,integration, andslowfor test categorizationDirectory Structure
Shared Test Fixtures
Created comprehensive fixtures in
conftest.py:temp_dir: Temporary directory managementmock_config: Mock configuration datamock_requests_session: HTTP request mockingmock_response: HTTP response mockingmock_environment: Environment variable managementsample_device_data: Test data for device operationsmock_file_system: File system simulationdisable_network_calls: Network isolation for testsDevelopment Environment
Running Tests
Install Dependencies
Run All Tests
Run Specific Test Categories
Generate Coverage Reports
Validation
✅ All dependencies install correctly
✅ pytest discovers and runs tests
✅ Coverage reporting generates HTML/XML outputs
✅ Custom markers (unit/integration/slow) work properly
✅ Shared fixtures are available across test modules
✅ Network isolation prevents external calls during testing
Development Notes
poetry.lockfile is included in version control for reproducible buildspyproject.tomlif neededtests/unit/for unit tests,tests/integration/for integration testsReady for Development
The testing infrastructure is now ready for developers to: