tree 89bfa5fa9e8ec09c16d4ee1fb3f42293a99b0f91
parent fa521454359715f99f0b18b542f85d4b923c6f28
author Winson Chiu <chiuwinson@google.com> 1662654610 +0000
committer TreeHugger Robot <treehugger-gerrit@google.com> 1663171975 +0000

Prevent ArtServiceTests Mockito strictness from masking test error

The finishMocking logic should only be run if no exception was thrown
by the test. Otherwise the strictness violation could mask any real
test errors and make the logs less useful.

This moves it out of the finally block and makes it mirror the original
MockitoRule, passing the Exception into Mockito so that it can skip
the validation logic if another error was encountered.

Test: atest ArtServiceTests

Ignore-AOSP-First: ART Services.

Change-Id: I499356f49bb4e3325de38959c5d2609bc95c0486