-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
Quieter Integration Tests #6513
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6513 +/- ##
=========================================
+ Coverage 40.27% 40.4% +0.12%
=========================================
Files 403 404 +1
Lines 54079 54089 +10
=========================================
+ Hits 21781 21852 +71
+ Misses 29279 29224 -55
+ Partials 3019 3013 -6
Continue to review full report at Codecov.
|
On test failure the logs will be outputted Unfortunately they will be prefixed with a reference to the new wrapped logger, but I'm not sure there's a way to avoid this without rewriting the go testframework. |
The integration tests produce far too much logging when they're running, so much that you cannot easily find errors.
This patch redirects the logging and adds some information to each test to print which test is running.
It requires that each integration runs PrintCurrentTest. (This has been added to prepareTestEnv.)