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.

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

Add pytest-cov #1578

Merged
merged 5 commits into from Nov 17, 2019
Merged

Add pytest-cov #1578

merged 5 commits into from Nov 17, 2019

Conversation

@sirex
Copy link
Contributor

sirex commented Nov 17, 2019

Also added coverage report in .travis.yml file.

sirex and others added 2 commits Nov 17, 2019
Also added coverage report in .travis.yml file.
github-actions github-actions
@cclauss
Copy link
Member

cclauss commented Nov 17, 2019

Nice!! Thanks! Is there a way to get a sorted list showing those files with no tests or with little test coverage?

@sirex
Copy link
Contributor Author

sirex commented Nov 17, 2019

Nice!! Thanks! Is there a way to get a sorted list showing those files with no tests or with little test coverage?

By default it is sorted by file name, bet here are other options:

sort (string, default “Name”): Sort the text report by the named column. Allowed values are “Name”, “Stmts”, “Miss”, “Branch”, “BrPart”, or “Cover”.

Changed sorting to Miss (by missing statements).

@cclauss
Copy link
Member

cclauss commented Nov 17, 2019

Can we do a reverse sort on cover? That would put the 5% test coverage near the top of the list and the 95% near the bottom. This is super cool. Thanks for doing it!

Copy link
Member

cclauss left a comment

Awesome!!!

@sirex
Copy link
Contributor Author

sirex commented Nov 17, 2019

Can we do a reverse sort on cover? That would put the 5% test coverage near the top of the list and the 95% near the bottom. This is super cool. Thanks for doing it!

I'm not sure, but I guess reverse is not supported. Documentation lists these sort options:

  • Name
  • Stmts
  • Miss
  • Branch
  • BrPart
  • Cover
@cclauss cclauss merged commit 5616fa9 into TheAlgorithms:master Nov 17, 2019
2 checks passed
2 checks passed
LGTM analysis: Python No code changes detected
Details
Travis CI - Pull Request Build Passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.