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
Allow mockito-logger to be configurable as Mockito extension #1430
Conversation
ab12d97
to
0ede2b0
Compare
Codecov Report
@@ Coverage Diff @@
## release/2.x #1430 +/- ##
=================================================
+ Coverage 87.53% 87.54% +<.01%
- Complexity 2435 2437 +2
=================================================
Files 300 300
Lines 6250 6255 +5
Branches 775 775
=================================================
+ Hits 5471 5476 +5
Misses 582 582
Partials 197 197
Continue to review full report at Codecov.
|
+1 |
5e66847
to
957ed86
Compare
e8c2eea
to
50b2366
Compare
50b2366
to
c9c42ac
Compare
c9c42ac
to
94122e6
Compare
94122e6
to
7f60590
Compare
7f60590
to
d0ee7af
Compare
rebased with current upstream |
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.
Thanks for this PR, I had missed it. Sorry for the long wait :(
Just 1 more comment regarding the documentation. We also need another rebase, as I just fixed an issue with Java 11 javadoc generation. After that we are good to go. Thanks again!
@@ -2,7 +2,7 @@ | |||
* Copyright (c) 2007 Mockito contributors | |||
* This program is made available under the terms of the MIT License. | |||
*/ | |||
package org.mockito.internal.util; | |||
package org.mockito.plugins; | |||
|
|||
public interface MockitoLogger { |
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.
We are going to need some additional documentation here. https://static.javadoc.io/org.mockito/mockito-core/2.23.4/org/mockito/plugins/MockMaker.html is probably a good starting point of what we can add as documentation.
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.
done, i hope that description is sufficient :]
d0ee7af
to
ee37eed
Compare
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.
Thanks!
thx a lot ;] |
Hi,
we have many of legacy tests that do over-mocking code base - lots of unused mocking there (we are cleaning it slowly). We have user
@MockitoSettings(strictness = Strictness.WARN)
switch to log and not fail in this situactions. By default all logs are printed out to console usingSystem.out.println
and this make us trouble because we have "silent" profile at CI server which is configured to silent Loggers / as you expect sith.println
we are not trully silent in this case :]Thats why I have created this PR proposal and want to know what do you think.
Thx,
Ivos