Replace @InjectMock with @Mock in MockitoExtension about junit5-samples HOT 4 CLOSED

@smoyer64, thanks so much for raising the issue with the Mockito team! 👍

That's something we've been meaning to do for almost half a year now but kept forgetting to follow up on. 😉

from junit5-samples.

No problem ... until I'm competent with the core JUnit 5 code, I figure there are other ways I can help. Since you've stated several times that the MockitoExtension is not production code, I figured there had been compromises ... if it bugs me, it probably bugs other people.

In any case, JUnitParams and Mockito are two libraries I use in almost every project - and they have been able to run together since mock injection could be done via a @rule (JUnitParams has to be a runner to overcome the prohibition against test methods having parameters). In any case, I'd also love to create a JUnitParamsExtension but JUnit 5 isn't quite ready for it. I'm working on a proposal with a corresponding PoC for a couple callbacks that will make the JUnitParamsExtension possible:

• AfterAllDiscoveryCallback
• AfterEachDiscoveryCallback

It's not completely altruistic - I also need something like this for upREST.

from junit5-samples.

Sounds good. Keep us posted on your progress!

from junit5-samples.

Resolved in 039d920.

from junit5-samples.