Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6119b5c3 authored by Tyler Gunn's avatar Tyler Gunn
Browse files

Remove verification of native method in InCallTonePlayerTest. 

Remove verification of ToneGenerator#release(); its a native method and
apparently mockito has issues with this in some cases.  Local runs are
all fine, but apparently Cuttlefish is not.

Fixes: 231726501
Test: Run unit tests and presubmit.
Change-Id: I337df13987f2738a08b36aa12de4e8682535ab49
parent 37699d93

tests/src/com/android/server/telecom/tests/InCallTonePlayerTest.java

+3 −1
Original line number Diff line number Diff line
@@ -181,7 +181,9 @@ public class InCallTonePlayerTest extends TelecomTestCase { 
        mInCallTonePlayer.stopTone();

        // Timeouts due to threads!

        verify(mCallAudioManager, timeout(5000)).setIsTonePlaying(eq(false));

        verify(mToneGenerator, timeout(5000)).release();

        // Ideally it would be nice to verify this, however release is a native method so appears to

        // cause flakiness when testing on Cuttlefish.

        // verify(mToneGenerator, timeout(5000)).release();



        // Correctness check: ensure we can't start the tone again.

        assertFalse(mInCallTonePlayer.startTone());