test: add missing test coverage for config.py (Fixes #36)#37
Merged
KHolodilin merged 4 commits intomainfrom Jan 26, 2026
Merged
test: add missing test coverage for config.py (Fixes #36)#37KHolodilin merged 4 commits intomainfrom
KHolodilin merged 4 commits intomainfrom
Conversation
- Add config.yaml.example to email_processor package directory - Configure package-data in pyproject.toml to include config.yaml.example - Update config.py to find config.yaml.example in package using importlib.resources - Keep MANIFEST.in for additional files (README.md, LICENSE) - Files are now accessible in installed package Fixes #32
- Add README.md to email_processor package directory - Update package-data to include README.md - README.md is now accessible in installed package Fixes #32
- Add tests for _find_config_example() function - Test finding file in current directory - Test finding file in package via importlib.resources - Test fallback when package resource not found - Add test for create_default_config when example file not found - Add tests for file overwrite confirmation logic - Add test for create_default_config without rich console - Coverage for config.py now reaches 100%
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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
2 participants
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.
Description
Adds missing test coverage for
email_processor/cli/commands/config.pyto reach 100% coverage. Codecov reported 72.72% patch coverage with 3 lines missing.Changes
_find_config_example()function:importlib.resourcescreate_default_config()when example file is not foundcreate_default_config()without rich console (fallback to info)Testing
config.pynow reaches 100%Coverage Results
Before: 72.72% (3 lines missing)
After: 100% ✅
Checklist
Fixes #36