Fixes #1967: Correctly handle mocks with limited life-cycle in listeners.

[raphw]

Due to the limited lifetime of static mocks, they cannot be validated within a Mockito session since they might have expired when the validation is applied. This patch excludes static mocks from all validation. To support static mocks in the regular session validation, we would rather need to trigger the validation upon release of any static mock.

[TimvdLippe]

It's not clear to me how the test reproduces the issue as reported in #1967 Can we instead copy the code as reported in the issue and add that to our test suite?

[raphw]

It only tests part of the issue. For actually testing it, we would need to run the JUnit tests with the inline mock maker and create subprojects for it. I am going on a two week vacation tomorrow, so I did not want to open that barrel of wine tonight but I validated the working manually. If you have some time, could you add additional subprojects? I validated the runners manually, however.

[TimvdLippe]

Ah okay. I got confused as the commit mentions that the issue is fixed, so that's why I thought we had test the whole scenario. Please go on vacation, we can figure it out later 😄

[codecov-commenter]

Codecov Report

Merging #1968 into release/3.x will increase coverage by 0.01%.
The diff coverage is 65.21%.

@@                Coverage Diff                @@
##             release/3.x    #1968      +/-   ##
=================================================
+ Coverage          85.28%   85.30%   +0.01%     
- Complexity          2591     2594       +3     
=================================================
  Files                322      322              
  Lines               7483     7492       +9     
  Branches             892      895       +3     
=================================================
+ Hits                6382     6391       +9     
  Misses               862      862              
  Partials             239      239              

Impacted Files	Coverage Δ	Complexity Δ
src/main/java/org/mockito/MockedStatic.java	100.00% <ø> (ø)	1.00 <0.00> (ø)
...in/java/org/mockito/internal/MockedStaticImpl.java	74.66% <0.00%> (-2.05%)	9.00 <0.00> (ø)
...ternal/invocation/finder/AllInvocationsFinder.java	87.50% <0.00%> (-12.50%)	4.00 <0.00> (ø)
...kito/internal/framework/DefaultMockitoSession.java	87.50% <76.92%> (-4.61%)	8.00 <2.00> (ø)
...nternal/configuration/MockAnnotationProcessor.java	88.88% <83.33%> (+18.05%)	11.00 <0.00> (+3.00)
...to/internal/util/concurrent/WeakConcurrentMap.java	41.48% <0.00%> (+2.12%)	11.00% <0.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 05b39bf...e4e7aa9. Read the comment docs.

[raphw]

I added an additional project for the JUnit Jupiter extension and added a JUnit rule for the inline mock maker tests. I hope this should avoid any future regressions.