Skip to content
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.

Sign up for GitHub

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

github: add package source dir to coverage #65

Merged
merged 2 commits into from
Aug 16, 2024
Merged

github: add package source dir to coverage #65

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2ddba27
github: add package source dir to coverage
JochenSiegWork Aug 16, 2024
f33346b
github: add tests dir to coverage source
JochenSiegWork Aug 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/linting.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ jobs:
- name: Run unit-tests
run: |
# Run only the core test suite in the tests directory.
coverage run -m unittest discover tests
coverage run --source=molpipeline,tests -m unittest discover tests
# Create a coverage report. Fail if the coverage is below 85%. Exclude extra packages from the report.
coverage report --fail-under=85 --omit="*chemprop*","*/*chemprop*/*"
Expand All @@ -204,7 +204,7 @@ jobs:
- name: Run unit-tests for chemprop
run: |
# Run only the chemprop test suite.
coverage run -m unittest discover test_extras/test_chemprop
coverage run --source=molpipeline,tests -m unittest discover test_extras/test_chemprop
# Create a coverage report. Fail if the coverage is below 85%. Include only chemprop files in the report.
coverage report --fail-under=85 --include="*chemprop*","*/*chemprop*/*"
Expand Down
Loading