Codecov Report

Merging #1832 into release/3.x will decrease coverage by 0.07%.
The diff coverage is 79.03%.

@@                Coverage Diff                @@
##             release/3.x    #1832      +/-   ##
=================================================
- Coverage          86.71%   86.64%   -0.08%     
- Complexity          2492     2507      +15     
=================================================
  Files                311      314       +3     
  Lines               6551     6613      +62     
  Branches             822      829       +7     
=================================================
+ Hits                5681     5730      +49     
- Misses               674      682       +8     
- Partials             196      201       +5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc6eadc...4b3cedf. Read the comment docs.

@TimvdLippe
Why not fix the implementation of any(Class<T> type) ? any(..) can check if the passed type is a primitive and then delegate to the required matcher.

Sample:

public static <T> T any(Class<T> type) {
        if (type==int.class || type==Integer.class){ 
            return anyInt()
        }

        reportMatcher(new InstanceOf.VarArgAware(type, "<any " + type.getCanonicalName() + ">"));
        return defaultValue(type);
}

@ChristianSchwarz That would lead to a lot of confusing code. E.g. half of the matchers would be anyInt and other half would be any(Integer.class). I would also be hesitant to introducing an API that takes the primitives as classes.

For now, we can keep this checker and we can revisit your suggestion in the future. Do you mind filing a bug?

I am merging this for now, as it upstreams a checker that prevents real issues on runtime. We can change the Mockito API, but I would rather prevent users from running into runtime exceptions when we can.