-
Notifications
You must be signed in to change notification settings - Fork 10
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
pytest coverage #48
pytest coverage #48
Conversation
ca66719
to
a7f7e2b
Compare
command += [ | ||
"--cov-report", | ||
f"{self.coverage.get()}:{str(self.project.build_directory / coverage_file)}", | ||
f"--cov={str(self.project.directory / tests_dir)}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The --cov
option needs to point to the source directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right, I will change that
ce2660e
to
4cb2751
Compare
9998f72
to
009ad3b
Compare
4d931b8
to
dba3397
Compare
dba3397
to
335e313
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may want to consider also testing that the coverage report contains what it should contain, but I'm happy to merge it in its current state to unblock downstream work.
command += [ | ||
"--cov-report", | ||
f"{self.coverage.get().get_format()}:{str(self.project.build_directory / coverage_file)}", | ||
f"--cov={str(self.project.directory / self.settings.source_directory)}", | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there be a check to see if pytest-cov
is installed before adding these parameters?
No description provided.