Do not pass static mocks to regular listener callback.

[raphw]

Does not include static mocks in regular listener logic as it might distort existing mock collectors that do not expect scoped mocks. Fixes #1988.

We have passed Class objects to the listener to indicate a static mock. The class object is of course itself not a regular mock and it's cleaner to create a callback method of it's own. This way, existing listeners remain logically backwards compatible, including our own listener that is used within the JUnit runner (we have only tested the rule, I added a test for the runner, too).

[codecov-commenter]

Codecov Report

Merging #1989 into release/3.x will decrease coverage by 0.04%.
The diff coverage is 70.00%.

@@                Coverage Diff                @@
##             release/3.x    #1989      +/-   ##
=================================================
- Coverage          85.28%   85.23%   -0.05%     
- Complexity          2601     2604       +3     
=================================================
  Files                322      323       +1     
  Lines               7510     7520      +10     
  Branches             898      899       +1     
=================================================
+ Hits                6405     6410       +5     
- Misses               869      873       +4     
- Partials             236      237       +1     

Impacted Files	Coverage Δ	Complexity Δ
...ito/internal/invocation/InterceptedInvocation.java	76.59% <0.00%> (-5.23%)	25.00 <0.00> (ø)
...mockito/internal/progress/MockingProgressImpl.java	97.72% <100.00%> (+0.16%)	36.00 <3.00> (+3.00)
...va/org/mockito/listeners/MockCreationListener.java	100.00% <100.00%> (ø)	1.00 <1.00> (?)
...ternal/invocation/finder/AllInvocationsFinder.java	87.50% <0.00%> (-12.50%)	4.00% <0.00%> (-1.00%)

---

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 0c56b3a...724619b. Read the comment docs.