Skip to content

Configure test coverage files and add tests#20

Merged
zkoppert merged 4 commits intomainfrom
coverage
Jan 13, 2024
Merged

Configure test coverage files and add tests#20
zkoppert merged 4 commits intomainfrom
coverage

Conversation

@zkoppert
Copy link
Copy Markdown
Collaborator

@zkoppert zkoppert commented Jan 12, 2024

Pull Request

Proposed Changes

fixes #5

This pull request includes changes to improve the test coverage and code quality. The most important changes include adding a .coveragerc file to omit test files from coverage, increasing the coverage threshold in the Makefile, adding # pragma: no cover to the main() function in evergreen.py, and modifying test cases in test_dependabot_file.py and test_evergreen.py.

Coverage improvements:

  • .coveragerc: Added a new file to omit test files from coverage.
  • Makefile: Increased the coverage threshold from 50 to 80.
  • evergreen.py: Added # pragma: no cover to the main() function to exclude it from coverage. [1] [2]

Test case modifications:

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing

Reviewer

  • Label as either bug, documentation, enhancement, infrastructure, or breaking

Signed-off-by: Zack Koppert <zkoppert@github.com>
@zkoppert zkoppert added the enhancement New feature or request label Jan 12, 2024
@zkoppert zkoppert self-assigned this Jan 12, 2024
Signed-off-by: Zack Koppert <zkoppert@github.com>
Signed-off-by: Zack Koppert <zkoppert@github.com>
@zkoppert zkoppert marked this pull request as ready for review January 13, 2024 00:03
Signed-off-by: Zack Koppert <zkoppert@github.com>
@zkoppert zkoppert merged commit f43591f into main Jan 13, 2024
@zkoppert zkoppert deleted the coverage branch January 13, 2024 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test coverage

1 participant