Skip to content

cleanup: exclude test suites from coverage metrics (#981)#1108

Open
arybhatt4533 wants to merge 4 commits intoopenzim:mainfrom
arybhatt4533:cleanup/exclude-tests-coverage-981
Open

cleanup: exclude test suites from coverage metrics (#981)#1108
arybhatt4533 wants to merge 4 commits intoopenzim:mainfrom
arybhatt4533:cleanup/exclude-tests-coverage-981

Conversation

@arybhatt4533
Copy link
Copy Markdown

This PR addresses the issue of "artificially inflated" coverage numbers by excluding test code and non-production files from the coverage reports.

Changes:
Added .coveragerc to the root directory.
Configured omit patterns to exclude tests/, setup.py, and virtual environments.
This aligns with the "Quality Gate Finalization" phase of my GSoC proposal, ensuring that our 90% coverage target reflects 100% production logic.
@audiodude @rgaudin @legoktm @kelson42 @Jaifroid

Copy link
Copy Markdown
Member

@audiodude audiodude left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is incorrect, and looks very much like it was hallucinated by an LLM.

We do not have any */tests/ directories in the project. Did you even look?

@arybhatt4533
Copy link
Copy Markdown
Author

You're right, my apologies. I generalized the omit patterns based on standard Python templates and missed the specific structure of this repository. I should have verified the exact paths before pushing.

I’ll rectify this immediately by mapping the actual test locations and ensuring the .coveragerc reflects the true project layout. I’ll update the PR shortly with the corrected paths. @audiodude

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants