NullPointerException when stubbing a class in Groovy 2.3.6 about mockito HOT 14 CLOSED

tough one ;)

from mockito.

Possible duplicate of https://code.google.com/p/mockito/issues/detail?id=303
Solved by https://github.com/cyrusinnovation/mockito-groovy-support

from mockito.

==> Solved by https://github.com/cyrusinnovation/mockito-groovy-support
did not solve my problem. I still have the same error when mocking a groovy class in groovy.

If I mock the groovy class in java I works fine

from mockito.

A very similar problem recently happened to me. I got a NullPointerException in PogoMetaClassSite, where the metaClass was null during a unit test. I was using mockito-groovy-support.

Then I noticed that my mockito-groovy-support depended on Mockito 1.9.5. In my project I was using Mockito 1.8.5. After upgrading my version, everything worked.

from mockito.

@xnickmx Cool thanks for the feeback

from mockito.

The same here. mockito-groovy-support solved the issue for me, however it would be nice to have the root cause of the issue fixed in Mockito, instead of relaying on the 3rd aprty jar. @moss Can you consider issuing the pull request to Mockito with your fixes? You will make many folks happy ;) .

from mockito.

Problem would be that the workarround requires Groovy on the class path. One could avoid this by using reflection. Also, I wonder if there is a better way to check if a method is for retreiving the Groovy meta class than by checking the signature of the method as the current approach would break non-Groovy classes defining a method named getMetaClass.

One could however wire this. I do not think an auto-detection would be difficult to implement.

from mockito.

This should do the trick: https://github.com/raphw/mockito/commit/32340365469b99ec5ad2cf2c87b7a79cc472118b

Any objections against this approach?

from mockito.

I have mixed feelings about these changes, but I must agreed this certainly looks like a simple enough solution. @raphw Make a PR :)

from mockito.

Will do tomorrow. Simply ignoring the methods is actually easier.

from mockito.

Many thanks guys!

from mockito.

Okay, I pushed a change and put a pull request on the line. Using this, Mockito should support Groovy out of the box.

from mockito.

@raphw I guess this can be closed because Mockito is now using a byte buddy version with your fix?

from mockito.

You are right.

from mockito.