feat: After the process run print a test summary in console#542
feat: After the process run print a test summary in console#542astenstrasser merged 14 commits intomainfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #542 +/- ##
==========================================
+ Coverage 97.07% 97.90% +0.83%
==========================================
Files 22 23 +1
Lines 717 717
==========================================
+ Hits 696 702 +6
+ Misses 21 15 -6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
87be8f5 to
46beba5
Compare
camilamaia
left a comment
There was a problem hiding this comment.
@astenstrasser hey, thanks for the awesome PR. Great feature and really clean up. I left only some minor suggestions.
It looks great overall ⭐
| # Available imports to be used dinamically in the API spec | ||
| import datetime # noqa: F401 | ||
| import logging | ||
| import logging # noqa: F401 |
There was a problem hiding this comment.
Can we just remove this line? I believe it is not being used
| @@ -1,11 +1,8 @@ | |||
| import logging | |||
| _write_summary() | ||
|
|
||
|
|
||
| def log_report(uri): |
There was a problem hiding this comment.
maybe we could change this method name to something like print_report_path or write_report_path for consistency. Because I would assume that a log_report method would use a log object to log something and not a console to print. What do you think?
Description
Motivation behind this PR?
As discussed on #518
Checklist
Issue
Closes #538