-
Notifications
You must be signed in to change notification settings - Fork 1.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
chore(testing): stop log4js output in unit tests #3634
Conversation
✅ Build karma 2883 completed (commit 026732dbce by @johnjbarton) |
✅ Build karma 485 completed (commit 026732dbce by @johnjbarton) |
✅ Build karma 486 completed (commit 026732dbce by @johnjbarton) |
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.
Nice change 👍
test/unit/mocha-globals.js
Outdated
@@ -17,11 +18,13 @@ beforeEach(() => { | |||
|
|||
// set logger to log INFO, but do not append to console |
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.
nit: I guess this comment is obsolete now? Maybe replace it with the text from the PR description.
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.
yes, thanks!
8b29b03
to
8d9fe31
Compare
✅ Build karma 2893 completed (commit faf67d722b by @johnjbarton) |
✅ Build karma 496 completed (commit faf67d722b by @johnjbarton) |
✅ Build karma 495 completed (commit faf67d722b by @johnjbarton) |
🎉 This PR is included in version 6.0.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Use https://log4js-node.github.io/log4js-node/recording.html to verify logs