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

Jump to bottom

Get back to 90% coverage #175

Closed
Kaliumhexacyanoferrat opened this issue Jan 17, 2021 · 2 comments
Closed

Get back to 90% coverage #175

Kaliumhexacyanoferrat opened this issue Jan 17, 2021 · 2 comments
Labels
enhancement New feature or request
Milestone
Version 5.2

Comments

@Kaliumhexacyanoferrat
Copy link
Owner

As an user of the GenHTTP webserver, I want the test coverage of the project to be high, so that I can trust the functionality.

Acceptance criteria

  • Coverage has been brought back over 90%
@Kaliumhexacyanoferrat Kaliumhexacyanoferrat added the enhancement New feature or request label Jan 17, 2021
@Kaliumhexacyanoferrat
Copy link
Owner Author

Kaliumhexacyanoferrat commented Jan 17, 2021 •
edited

Looking at the test coverage, this seems to be related to the coverlet collector not correctly picking up coverage of multi-line statements.

image

@Kaliumhexacyanoferrat
Copy link
Owner Author

Back to 89% after switching to coverlet 3.0.1.

@Kaliumhexacyanoferrat Kaliumhexacyanoferrat added this to the Version 5.2 milestone Jan 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
Version 5.2
Development

No branches or pull requests
1 participant
@Kaliumhexacyanoferrat