Broken unused stubbing reporting when matching previous stubbed call

When stubbing the same method call twice, and the second stub setup matches to first stubbed call, the first stub is not reported as unused, because Mockito detects the next stub setup as a usage of the previous stub setup. This is an example test:

public class UnnecessaryStubbingWithImplicitEqMatchers {
    @Mock IMethods mock;
    MockitoSession mockito = Mockito.mockitoSession().initMocks(this).strictness(Strictness.STRICT_STUBS).startMocking();

    @After public void after() {
        mockito.finishMocking();
    }

    @Test public void unnecessary_stubbing() {
        given(mock.simpleMethod("1")).willReturn("one");
        given(mock.simpleMethod("2")).willReturn("three");
        given(mock.simpleMethod("1")).willReturn("two");
        given(mock.simpleMethod("1")).willReturn("four");

        mock.simpleMethod("1");
        mock.simpleMethod("2");
    }
}

The test above should fail with UnnecessaryStubbingException containing two unused stubs, but it does not.

Mockito version: v3.2.4

[TimvdLippe]

Revert in progress: https://github.com/mockito/mockito/pull/1878

[bbonanno]

@pkoenig10 I fail to see how that is less verbose than multiple stubbings, but in any case, that’s not the point. I think this change should be reverted until it can be given a deeper thought. do you agree? @mockito/developers ?