75

I have a very simple test case that is using Mockito and Spring Test framework. When I do

when(pcUserService.read("1")).thenReturn(pcUser);

I get this exception.

org.mockito.exceptions.misusing.MissingMethodInvocationException: 
when() requires an argument which has to be 'a method call on a mock'.
For example:
    when(mock.getArticles()).thenReturn(articles);

Also, this error might show up because:
1. you stub either of: final/private/equals()/hashCode() methods.
   Those methods *cannot* be stubbed/verified.
2. inside when() you don't call method on mock but on some other object.

    at com.project.cleaner.controller.test.PcUserControllerTest.shouldGetPcUser(PcUserControllerTest.java:93)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
    at java.lang.reflect.Method.invoke(Unknown Source)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:44)

I have tried with different methods but keep on getting this error message. I am using Spring 3.1.0.RELEASE with Mockito. Please share and guide me in the right direction.

1
  • I've the same problem, but I'm using: @Autowired @ReplaceWithMock(beanName="logDao") private LogDao logDaoMock; Dec 23, 2013 at 9:36

17 Answers 17

71

You need to create a MOCK of pcUserService first, and then use that mock.

PcUserService mock = org.mockito.Mockito.mock(PcUserService.class);
when(mock.read("1")).thenReturn(pcUser);
14
  • But I can not debug in that case. do it actually calls that method? Jun 10, 2014 at 13:13
  • @eatSleepCode: the REAL method PcUserService.read gets never invoke in this example. Instead an Mockito Mock gets invoked, and this mock return pcUser
    – Ralph
    Jun 10, 2014 at 13:43
  • 2
    yes, I am facing a issue with this, I have a service lets say TestService and it has a method testMethod() and code inside testMethod includes call to another service method for eg. AnotherTestService.getData(). So in my case I am not able to mock AnotherTestService what should I do? Jun 11, 2014 at 4:52
  • Why are you not able to replace AnotherTestService with an mock?
    – Ralph
    Jun 11, 2014 at 5:21
  • @eatSleepCode: Maybe you should have a look at this answer: stackoverflow.com/a/20856647/280244
    – Ralph
    Jun 11, 2014 at 5:33
36

In case others hit this issue....

It could also be the case that the method you are trying to mock out,pcUserService.read, is declared as a final method. From what I've noticed this appears to cause issues with Mockito.

3
  • 3
    The diagnostic is ok, but what is the solution :p
    – amdev
    Jul 22, 2018 at 7:07
  • @amdev Don't make pcUserService.read final :) Not ideal I know, but there's not a whole lot you can do about it. Mockito can't mock a final method. See stackoverflow.com/questions/3793791/final-method-mocking for more detail and possible workarounds
    – djkelly99
    Jul 24, 2018 at 21:37
  • 2
    Actually, sometimes you don't mock your own class but some class from an external dependencies. But it's ok I found how to mock final method with mockito stackoverflow.com/questions/14292863/…
    – amdev
    Jul 25, 2018 at 7:32
25

If you use Kotlin, you should know that methods are final by default. So write open fun instead of fun. Thanks to @djkelly99 for a tip.

5
9

Another solution to this issue might be that in case of a test class that is using PowerMockRunner, you might have to add the class that you are mocking to the list, in @PrepareForTest annotation.

For instance -

@PrepareForTest({ PcUserService.class })

1
  • 1
    Thanks, I thought I had, but I actually added the interface to the @preparefortest list, not the class, this answer made me double check there.
    – lauksas
    Dec 30, 2021 at 11:36
5

In my case it was solved by injecting @MockBean.

For ex.

@MockBean
StateRepository mockStateRepository;
4

There's another possible reason for such error - sometimes IDE prefers to statically import Mockito.when() from another package:

import static io.codearte.catchexception.shade.mockito.Mockito.when;

vs

import static org.mockito.Mockito.when; //should normally use this one

The thing is 'when' from io.codearte package is compliant with org.mockito.Mockito.any() on compilation level, but fails during runtime with that exact same error message.

4

If you get this exception when using MockedStatic or Mockito.mockStatic, it can also mean that you are mixing matchers and literals in the call to the static method.

Try changing any mixes like YourClass.staticMethod(any(), "literal") to YourClass.staticMethod(any(), eq("literal"))

3

I had the same issue, the method that I was trying to mock it was a final method. I removed the modifier and it worked fine.

2

For the help of others who stuck with the same problem;

The method you are trying to mock , pcUserService.read, is declared as a final method. Static methods appears to cause issues with Mockito.

1

Basically You need to use the PowerMockito.mockStatic to enable static mocking for all static methods of a class. This means make it possible to stub them using the when-thenReturn syntax. For example: PowerMockito.mockStatic(TestClass.class); when(TestClass.getString()).thenReturn("HelloWorld!"); Note: you have to add @PrepareForTest({ TestClass.class }) to your unit test class.

1

When I got this exception, I was using @InjectMocks on the class that I needed to have the @Mock objects injected into (via constructor injection).

AFter much searching, I finally stumbled across this article:

https://mkyong.com/spring-boot/mockito-when-requires-an-argument-which-has-to-be-a-method-call-on-a-mock/

The key part to take away from it is (from the article):

When Mockito see this @InjectMocks, it doesn’t mock it, it just creates a normal instance, so the when() will be failed.

So this make this exception message I was getting

when() requires an argument which has to be 'a method call on a mock'.

make sense now; you aren't using when on an actual mock but an actual instance.

The link also provides the solution to the problem:

To solve it, annotate @Spy to mock it partially.

On top of @InjectMocks, put @Spy.

As a side note, if you try to make it a Mock by putting @Mock on top of @InjectMocks, you will get exception:

org.mockito.exceptions.base.MockitoException: This combination of annotations is not permitted on a single field: @Mock and @InjectMocks

Using @Spy on top of @InjectMocks solved the problem for me.

1

I have faced a similar issue. You can simply use the “spy” annotation on the “pcUserService” bean, if you cannot mock the “pcUserService” bean (e.g. when the “pcUserService” bean itself is under test and has @InjectMocks annotation). Briefly, “spy” means you mock the bean partially.

The Mockito – when() requires an argument which has to be ‘a method call on a mock’ is so great and makes the point in a nutshell with a simple and useful example.

0

If you use KOIN, include in the gradle's dependencies:

dependencies {
    ...
    testImplementation "org.koin:koin-test:2.0.0"
}
0

I faced similar issue when mocking static method of anotherClass called inside testing a method of someClass. In that case, we need to add @PrepareForTest({anotherClass.class, someClass.class}) both the class having the static method and caller class.

0

I faced same issue and it was because the mocked object is defined as @Beanand the overriden @Bean in the test was not working because of a missing spring property value

@Bean
@Primary
public JwtTokenUtil jwtTokenUtil() {
    return Mockito.mock(JwtTokenUtil.class);
}

@Autowired
private JwtTokenUtil jwtTokenUtil;

when(jwtTokenUtil.validateToken(jwtToken)).thenReturn(true);

Fix

spring.main.allow-bean-definition-overriding=true in application-test.properties

0

In my case, I had to attach the attach the Mockito extension to the Class

@ExtendWith(MockitoExtension.class)
class pcUserService{
 ... 
}
-1

if anything above answers your case, then check that you did missed the @Test annotation for your test method.

@Test
public void my_test_method(){
}

just quoting, as this was my case. just missed the annotation.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.