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
Fixes #3219: Add support for static mocks on DoNotMockEnforcer #3220
Conversation
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 tests! I think an enum API is easier to manage and avoids introducing many more overloads, as well as clear migration path for future implementations of DoNotMockEnforcer
.
.../extTest/src/test/java/org/mockitousage/plugins/donotmockenforcer/DoNotmockEnforcerTest.java
Show resolved
Hide resolved
9c58028
to
f3e2bef
Compare
@TimvdLippe changes performed. It's technically missing a test project, though. Let me know if you'd still like to have it and I'll make it on Monday. |
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.
Awesome thanks!
src/main/java/org/mockito/internal/configuration/plugins/PluginRegistry.java
Outdated
Show resolved
Hide resolved
b5cd2c8
to
01b32ab
Compare
Fixes mockito#3219 Fix mockStatic bypassing DoNotMockEnforcer Add (optional) method on DoNotMockEnforcer for static mocks
01b32ab
to
8c0239c
Compare
- Follow up to mockito#3220 - Make the method return the lowest common denominator type
- Follow up to mockito#3220 - Make the method return the lowest common denominator type
Make the method return the lowest common denominator type Follow up to #3220
Fixes #3219
Fix mockStatic bypassing DoNotMockEnforcer
Add (optional) method on DoNotMockEnforcer for static mocks
Checklist
including project members to get a better picture of the change
commit is meaningful and help the people that will explore a change in 2 years
Fixes #<issue number>
in the description if relevantFixes #<issue number>
if relevant