Skip to content

Bug: Accurate Coverage Checks #35

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

Open
dyga01 opened this issue Oct 16, 2024 · 2 comments
Open

Bug: Accurate Coverage Checks #35

dyga01 opened this issue Oct 16, 2024 · 2 comments
Assignees
Labels
infrastructure CI/CD configuration test Test cases or test running

Comments

@dyga01
Copy link
Collaborator

dyga01 commented Oct 16, 2024

Description:
The execexam tool currently displays all of the Python files instead of just test files when measuring coverage. The coverage checks are also inaccurate and can say that there is 100% coverage for files that have 0 tests.

Expected Behavior:
The poetry run task coverage command should return fully accurate coverage monitoring for all test files.

Actual Behavior:
The poetry run task coverage command returns coverage for all files. The coverage can also be inaccurate, as described above.

Proposed Solution:
Change the command that is run in the pyproject.toml file to only report coverage for test cases and work to ensure that this coverage is accurate.

@gkapfham
Copy link
Collaborator

Hello @dyga01 does PR #38 resolve this issue? If so, can you mark that the PR is an implementation of the proposed solution?

@dyga01
Copy link
Collaborator Author

dyga01 commented Oct 24, 2024

@gkapfham PR #38 does resolve the issue and reports accurate coverage in addition to adding more test cases.

@gkapfham gkapfham added test Test cases or test running infrastructure CI/CD configuration labels Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure CI/CD configuration test Test cases or test running
Projects
None yet
Development

No branches or pull requests

2 participants