Include cause when FieldReader fails to access field

[A248]

Fixes #2266

Simply passes the exception cause to the thrown MockitoException

[TimvdLippe]

Thanks for the PR! Could you add a regression test to our test suite as well? You can use the snippet that you posted in the issue and put it in the moduleTest subproject.

[codecov-io]

Codecov Report

Merging #2267 (37e96c7) into release/3.x (2fa2a63) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 37e96c7 differs from pull request most recent head 51c6ea9. Consider uploading reports for the commit 51c6ea9 to get more accurate results

@@                Coverage Diff                @@
##             release/3.x    #2267      +/-   ##
=================================================
+ Coverage          84.82%   84.84%   +0.02%     
  Complexity          2734     2734              
=================================================
  Files                326      326              
  Lines               8342     8342              
  Branches             997      997              
=================================================
+ Hits                7076     7078       +2     
  Misses               990      990              
+ Partials             276      274       -2     

Impacted Files	Coverage Δ	Complexity Δ
.../mockito/internal/util/reflection/FieldReader.java	77.77% <ø> (ø)	4.00 <0.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 2fa2a63...51c6ea9. Read the comment docs.

[A248]

I can confirm that the regression test fails when the change to FieldReader is reverted.

[TimvdLippe]

Great unit test, thank you so much for your contribution!