Skip to content
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

Mockito 4 #185

Closed
wants to merge 1 commit into from
Closed

Mockito 4 #185

wants to merge 1 commit into from

Conversation

sullis
Copy link
Contributor

@sullis sullis commented Jan 26, 2022

No description provided.

@jkschneider
Copy link
Member

These particular dependencies need to be pinned to 3 because we are testing Mockito 2 to 3 changes, but you make me wonder now what breaking changes there are between Mockito 3 and 4!

@jkschneider jkschneider reopened this Jan 26, 2022
@jkschneider
Copy link
Member

jkschneider commented Jan 26, 2022

Maybe it helps to add more context. In rewrite-spring we have a similar scenario and actually create whole dependency configurations to isolate changes from one minor release to another.

Doing something similar here, we can group the unit tests for say Mockito 2->3 in one source set that depends on Mockito 2 (the beginning state of a test to Mockito 3), and group tests for Mockito 3->4 another source set that depends on Mockito 3 (the beginning state of a test to Mockito 4)

@pway99 pway99 added this to In review in Backlog via automation Jan 26, 2022
@pway99
Copy link
Contributor

pway99 commented Jan 26, 2022

added a issue #186 for tracking the mockito 3 to 4 migration

@tkvangorder tkvangorder added the enhancement New feature or request label Mar 1, 2022
@tkvangorder
Copy link
Contributor

Closing this as we are tracking the mockito 4 with #186

@tkvangorder tkvangorder closed this Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Backlog
  
In review
Development

Successfully merging this pull request may close these issues.

None yet

4 participants