test: Add comprehensive testing infrastructure with Poetry#30
Open
llbbl wants to merge 1 commit intoKaiyangZhou:masterfrom
Open
test: Add comprehensive testing infrastructure with Poetry#30llbbl wants to merge 1 commit intoKaiyangZhou:masterfrom
llbbl wants to merge 1 commit intoKaiyangZhou:masterfrom
Conversation
- Set up Poetry as package manager with pyproject.toml - Add pytest, pytest-cov, and pytest-mock as dev dependencies - Configure pytest with coverage thresholds and custom markers - Create test directory structure with unit/integration separation - Add shared fixtures in conftest.py for common test scenarios - Include validation tests to verify infrastructure setup - Update .gitignore with testing and Claude-specific entries
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 Comprehensive Testing Infrastructure
Summary
This PR establishes a complete testing infrastructure for the Center Loss PyTorch implementation project. It sets up Poetry as the package manager and configures pytest with coverage reporting and custom test organization.
Changes Made
Package Management
pyproject.tomlwith Poetry configuration as the default package managerTesting Configuration
pytest Configuration:
unit,integration,slowCoverage Configuration:
Directory Structure
Shared Fixtures (conftest.py)
temp_dir: Temporary directory for test filesdevice: PyTorch device (CPU/CUDA) detectionrandom_seed: Reproducible random seedssample_features,sample_labels,sample_centers: Test data generatorsmock_model_config,mock_training_config: Configuration fixturessample_mnist_batch: MNIST-like data generatorcapture_stdout: Stdout capture utilitymock_data_loader: Mock data loader factoryassert_tensor_close: Tensor comparison helpercleanup_matplotlib: Automatic matplotlib cleanupAdditional Configuration
.gitignorewith:.pytest_cache/,coverage.xml,htmlcov/).claude/*)poetry.lockis NOT ignored (should be committed)How to Use
Running Tests
Writing Tests
tests/unit/directorytests/integration/directory@pytest.mark.unit,@pytest.mark.integration, or@pytest.mark.slowCoverage Reports
htmlcov/directorycoverage.xmlfor CI integrationValidation
The infrastructure has been validated with a comprehensive test suite (
test_setup_validation.py) that verifies:All validation tests pass successfully ✅
Notes