-
Notifications
You must be signed in to change notification settings - Fork 0
Add comprehensive test suite with 133 tests #152
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
Conversation
- Add tests for fishconsumer module (25 tests) - Add tests for config module (15 tests) - Add tests for utils module (24 tests) - Add tests for legendre_tools module (11 tests, 100% coverage) - Add tests for colors module (17 tests, 100% coverage) - Add tests for likelihood base module (9 tests) - Improved overall test coverage from ~30% to 48% - All 133 tests passing with comprehensive validation - Includes mock-based testing for complex dependencies - Fast execution tests focused on functionality validation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #152 +/- ##
===========================================
+ Coverage 46.35% 56.44% +10.09%
===========================================
Files 34 34
Lines 5993 5993
===========================================
+ Hits 2778 3383 +605
+ Misses 3215 2610 -605 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request adds a comprehensive test suite to significantly improve the test coverage of the cosmicfishpie project, expanding from approximately 30% to 48% coverage with 133 new tests across 6 key modules.
- Adds extensive test coverage for utility modules (utils, legendre_tools, colors) with validation of mathematical operations and helper functions
- Implements configuration and likelihood testing with mock-based approaches to handle complex dependencies
- Provides thorough coverage of analysis utilities including color formatting and ANSI terminal output functions
Reviewed Changes
Copilot reviewed 5 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
tests/utils_test.py | Comprehensive testing of utility classes including deepupdate, debug printing, numerical operations, and filesystem helpers |
tests/likelihood_base_test.py | Basic test coverage for likelihood module with defensive imports and error handling for missing functions |
tests/legendre_tools_test.py | Complete testing of Legendre polynomial tools and Wigner matrices with mathematical validation |
tests/config_test.py | Configuration module testing with mocked dependencies and YAML operations validation |
tests/colors_test.py | Full coverage of color utility functions including ANSI terminal formatting and RGB color generation |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…isort) and raise coverage beyond 50%
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #
Changes proposed in this pull request:
Before submitting
section of the
CONTRIBUTING
docs.Writing docstrings section of the
CONTRIBUTING
docs.