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

Commit e83f0519 authored by yuanjiahsu's avatar yuanjiahsu Committed by Yuanjia Hsu
Browse files

Fix flaky test case

The callback is called from executor, so verify it with timeout time.

Bug: 220358587
Test: atest FrameworksCoreTests:android.companion.virtual, run over 30
times locally and it all passed

Change-Id: Idf598396bfb0b09fea8f755e170fea0b5f4eb98e
parent 83fbe6e5
Loading
Loading
Loading
Loading
+3 −2
Original line number Diff line number Diff line
@@ -22,6 +22,7 @@ import static android.media.AudioFormat.ENCODING_PCM_16BIT;

import static com.google.common.truth.Truth.assertThat;

import static org.mockito.Mockito.timeout;
import static org.mockito.Mockito.verify;
import static org.testng.Assert.assertThrows;

@@ -168,7 +169,7 @@ public class VirtualAudioSessionTest {

        mVirtualAudioSession.onPlaybackConfigChanged(configs);

        verify(mCallback).onPlaybackConfigChanged(configs);
        verify(mCallback, timeout(2000)).onPlaybackConfigChanged(configs);
    }

    @Test
@@ -177,6 +178,6 @@ public class VirtualAudioSessionTest {

        mVirtualAudioSession.onRecordingConfigChanged(configs);

        verify(mCallback).onRecordingConfigChanged(configs);
        verify(mCallback, timeout(2000)).onRecordingConfigChanged(configs);
    }
}