ArgumentMatchers not working for Mockito.mockStatic

[nicolasmafra]

When I do a mockStatic with argument matchers like this:

var mockedRequestClass = Mockito.mockStatic(RequestClass.class);
mockedRequestClass.when(() -> RequestClass.sendRequest(mockedUrl, any(), any())).thenReturn(null);

Mockito throws the following exception:
org.mockito.exceptions.misusing.MissingMethodInvocationException: when() requires an argument which has to be 'a method call on a mock'.

How can I use argument matchers with Mockito.mockStatic? Shouldn't that be possible?

[nicolasmafra]

Note: if I remove ArgumentMatchers no exceptions are thrown.

[Cybermite]

@nickmafra try using matchers for all the params (see the added eq matcher below):

var mockedRequestClass = Mockito.mockStatic(RequestClass.class);
mockedRequestClass.when(() -> RequestClass.sendRequest(eq(mockedUrl), any(), any())).thenReturn(null);

When using traditional mocks, it'll tell you to use matchers for all the params when at least one of them uses a matcher. My hunch is it's the same for MockedStatic and the exception you're getting is misleading because it swallowed the root cause of the issue.

[TimvdLippe]

Sounds like we should update mockStatic to behave similar to the others, where we validate the argument matchers. PRs welcome!

[temp-droid]

Hi @TimvdLippe , I think the issue comes from here:
https://github.com/mockito/mockito/blob/main/src/main/java/org/mockito/internal/MockedStaticImpl.java#L48-L60

The validation is happening but ignored. Not sure why, but this could fix it if we want to keep the catch Throwable for backwards compatibility:

        try {
            verification.apply();
        } catch (MockitoException exception) {
            throw exception;
        } catch (Throwable ignored) {
        }

What do you think?