-
Notifications
You must be signed in to change notification settings - Fork 5
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
GAIAPLAT-2070 - Adding coverage support to docker_dev #1377
Conversation
…ions more evident.
… into gaiaplat-2066
…kage after lifting out Abort exception. https://gaiaplatform.atlassian.net/browse/GAIAPLAT-2067
def __repr__ | ||
if self\.debug | ||
|
||
# Don't complain if tests don't hit defensive assertion code: |
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.
I suspect that this does not cover our own assertions, which means that our coverage numbers will be lower than they would be without assertions. If that's the case, perhaps we could run coverage on code built with assertions disabled?
Just brainstorming here. Whatever good ideas come from this, we should track them in JIRA.
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.
This is just the standard boilerplate for coverage.
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.
I left a comment but it's not blocking, although it may lead to future work.
https://gaiaplatform.atlassian.net/browse/GAIAPLAT-2070
Adding the reporting of coverage. Currently at 86%.