Fix #2915 Forbid spy on mocked interface

[bmaggi]

A proposition of implementation for 2915

I only raise exception for interface since it's working with class.

[TimvdLippe]

Let's use MockUtil.isMock instead and block both interfaces and classes. Spying on mocks was never supported and this is a safe change to make imo. Blocking only interfaces does not make sense to me, we should block all mocks.

[bmaggi]

I changed the code forbid all mocks

[codecov-commenter]

Codecov Report

Patch coverage has no change and project coverage change: -0.32 ⚠️

Comparison is base (f9fbf50) 85.70% compared to head (696758b) 85.39%.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2989      +/-   ##
============================================
- Coverage     85.70%   85.39%   -0.32%     
- Complexity     2865     2869       +4     
============================================
  Files           325      325              
  Lines          8696     8723      +27     
  Branches       1074     1078       +4     
============================================
- Hits           7453     7449       -4     
- Misses          966      992      +26     
- Partials        277      282       +5     

Impacted Files	Coverage Δ
src/main/java/org/mockito/Mockito.java	96.62% <ø> (+0.07%)	⬆️

... and 13 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[TimvdLippe]

Thank you!