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

Support Mockito 1.10.9+ #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atreides322
Copy link

This is an inelegant solution, but as of 1.10.9, classes that this
depended on became package protected. The simplest fix is to move the
child classes to the same package.

This addresses issue #13

This is an inelegant solution, but as of 1.10.9, classes that this
depended on became package protected. The simplest fix is to move the
child classes to the same package.

This addresses issue cyrusinnovation#13
@sfitts
Copy link

sfitts commented Apr 16, 2015

@atreides322 thanks for the fix. Any word on whether this might get pulled into a release?

@carl-pritchett
Copy link

Hi - I'd also like this to get pulled if possible. Is there any other way around relying on the internal classes of Mockito?

@charliemidtlyng
Copy link

Any updates on this? Would be nice to be able to support Mockito 1.10.x

@snekse
Copy link

snekse commented Aug 19, 2016

If needed, this project could adopt artifact versioning/packaging similar to Spock.

mockito-groovy-support:1.3-mockito-1.10.x

Then pulling in this fix could be isolated to those that want it. I'm guessing this will be more common as Spring Boot defaults to 1.10.x for testing. I'm guessing Grails 3 does the same.

@teicher
Copy link

teicher commented Sep 19, 2016

So if this project is abandoned, does anybody have the time and expertise to fork and apply the workaround ?
Indeed, spring-boot:1.4.0 needs mockito:1.10 which now is a showstopper to upgrading spring-boot when using Groovy...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants