-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update mockito monorepo to v5 (major) #3041
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
2 times, most recently
from
July 26, 2023 09:10
cf096ff
to
e7a0d8e
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
July 26, 2023 11:51
e7a0d8e
to
e32f49e
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
July 26, 2023 13:11
e32f49e
to
8b2c380
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
July 26, 2023 14:12
8b2c380
to
81716fa
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
3 times, most recently
from
July 27, 2023 14:36
52e2bfb
to
1e41096
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
July 28, 2023 10:57
1e41096
to
0976e34
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
July 28, 2023 17:15
0976e34
to
7782695
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
July 31, 2023 12:02
7782695
to
5368e90
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
3 times, most recently
from
August 3, 2023 17:01
8f7367b
to
91016ba
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 29, 2023 14:31
c181540
to
9a21f3a
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 29, 2023 15:43
9a21f3a
to
03d08ae
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 29, 2023 17:20
03d08ae
to
74fddcd
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 30, 2023 09:51
74fddcd
to
40630c3
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 30, 2023 12:03
40630c3
to
6638c54
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 30, 2023 12:46
6638c54
to
eb89ee7
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 30, 2023 14:10
eb89ee7
to
c4163b2
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 30, 2023 15:57
c4163b2
to
0280d0a
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 31, 2023 08:17
0280d0a
to
0ba39b6
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 31, 2023 10:21
0ba39b6
to
25efdd7
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 31, 2023 11:31
25efdd7
to
4040627
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 31, 2023 14:22
4040627
to
2f4e5d4
Compare
renovate
bot
force-pushed
the
renovate/major-mockito-monorepo
branch
from
August 31, 2023 15:07
2f4e5d4
to
d3657cc
Compare
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
4.11.0
->5.2.0
4.11.0
->5.5.0
4.11.0
->5.5.0
Release Notes
mockito/mockito (org.mockito:mockito-inline)
v5.2.0
Compare Source
Changelog generated by Shipkit Changelog Gradle Plugin
5.2.0
v5.1.1
Compare Source
Changelog generated by Shipkit Changelog Gradle Plugin
5.1.1
v5.1.0
Compare Source
Changelog generated by Shipkit Changelog Gradle Plugin
5.1.0
CONTRIBUTING.md
(#2870)CONTRIBUTING.md
(#2868)Mockito#{mock,spy}(T... reified)
with@SafeVarargs
(#2866)v5.0.0
Compare Source
Mockito 5: prepare for future JDK versions
For a while now, we have seen an increase in problems/incompatibilities with recent versions of the JDK due to our usage of JVM-internal API.
Most notably, JDK 17 made some changes which are incompatible with the current subclass mockmaker.
Therefore, to prepare for the future of JDK, we are making some core changes to ensure Mockito keeps on working.
Switch the default mockmaker to
mockito-inline
Back in Mockito 2.7.6, we published a new mockmaker based on the "inline bytecode" principle.
This mockmaker creates mocks manipulating bytecode equivalent within the original class such that its method implementations hook into the normal Mockito machinery.
As a comparison, the subclass mockmaker generates "real" subclasses for mocks, to mimic the same behavior.
While the approaches are similar, the inline mockmaker avoids certain restrictions that the JDK imposes.
For example, it does not violate module boundaries (introduced in JDK 9, but more heavily used in JDK 17) and avoids the leaking of the creation of the subclass.
Massive thanks to community member @reta who implemented this change.
Note: this does not affect
mockito-android
nor testing on Android.When should I still be using the subclass mockmaker?
There are legitimate remaining use cases for the subclass mockmaker.
For example, on the Graal VM's native image, the inline mockmaker will not work and the subclass mockmaker is the appropriate choice.
Additionally, if you would like to avoid mocking final classes, using the subclass mockmaker is a possibibility.
Note however that if you solely want to use the subclass mockmaker to avoid mocking final, you will run into the above mentioned issues on JDK 17+.
We want to leave this choice up to our users, which is why we will keep on supporting the subclass mockmaker.
If you want to use the subclass mockmaker instead, you can use the new
mockito-subclass
artifact (published on Maven Central along with all our other artifacts).Update the minimum supported Java version to 11
Mockito 4 supports Java 8 and above.
Similar to other open source projects, we are moving away from JDK 8 and to newer versions.
The primary reason for moving away from JDK 8 is the increasing maintenance costs with keeping our own infrastructure working.
Lately we have been running into more and more JDK 8 breakages.
Additionally, while we want to support the newest JDK API's, our current solution to support both JDK 8 and newer versions causes issues with the
SecurityManager
.Since we want Mockito to work on the newest version and more and more businesses adopting JDK 11, we have decided to make the switch as well.
Massive thanks to community member @reta who implemented this change.
What should I do if I still run JDK 8?
For JDK 8 and below, you can keep on using Mockito 4.
This is similar to if you are using JDK 6, for which you can keep on using Mockito 2.
The changes in Mockito 5 (for now) are primarily focused on the latest JDK versions, which means the API differences between Mockito 4 and 5 are minimal.
However, over time this will most likely widen, so we do recommend adopting JDK 11 in the future.
New
type()
method onArgumentMatcher
One of our most used public API's for customizing Mockito is the
ArgumentMatcher
interface.The interface allows you to define a custom matcher, which you can pass into method arguments to provide more targeted matches.
One major shortcoming of the
ArgumentMatcher
was the lack of varargs support.There were many, many issues filed related to varargs and Mockito unable to handle them.
Community member @big-andy-coates put in a lot of effort to come up with an appropriate solution, including fully implementing and comparing 2 approaches.
Ultimately, we decided that introducing a new
type()
method onArgumentMatcher
is the best solution.As a result, it is now possible to update your custom matchers to implement varargs support, if you so desire.
Note that
ArgumentMatcher
is still a@FunctionalInterface
and can therefore still be written as a lambda.Massive thanks to community member @big-andy-coates who implemented this change.
What is the effect of this new method?
For varargs methods, there was previously a way to only match zero arguments, or two or more arguments, by using the exact number of matchers, i.e.
But following the pattern to match exactly one argument:
doesn't work, as
any
is "vararg aware", so Mockito matched theany
against each element of the varargs parameter, meaning it will match any number of arguments, i.e. the above would of matched all of these:With the new
type
method, it's now possible to differentiate matching calls with any exact number of arguments, or to match any number of arguments.Therefore, if you want to match 0 or more arguments, use
any(String[].class)
.If you want to match an exact number of arguments, use
any(String.class)
(and specify as manyany
matchers as arguments you want to match on).In a similar fashion, the behavior of
ArgumentCaptor.forClass
has changed as well.If you want to capture all arguments, use an
ArgumentCaptor
forString[]
, otherwiseString
:For more information, see the description and conversation in pull request 2835 and pull request 2807.
At the same time,
ArgumentCaptor
is now fully type-aware.This allows for capturing specific subclasses on a generic method:
Do I need to implement this new method?
No, you don't need to.
Mockito 5 declares a default implementation, returning
Void.type
as the type of anArgumentMatcher
.This essentially means that Mockito will not consider the type when handling varargs.
However, if you do return a specific type, Mockito will consider this when matching arguments.
As a result, this new method is not a source-breaking change, but is a bytecode-breaking change.
All code working on Mockito 4 should work as-is when recompiled with Mockito 5.
Configuration
📅 Schedule: Branch creation - "after 7am and before 11am every weekday" in timezone Europe/London, Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR has been generated by Mend Renovate. View repository job log here.