diff options
author | Jimmy Dalqvist <jimmy.dalqvist@sonymobile.com> | 2013-04-12 10:50:56 +0200 |
---|---|---|
committer | Steve Kondik <shade@chemlab.org> | 2016-04-14 00:49:34 -0700 |
commit | 15e7a21560764d644673854894eefbcd759d420a (patch) | |
tree | dbe4932f39f582061edd6cbbc482b4b93dce7146 /media | |
parent | ad3fa4175b3d30bc071c516d948dd2d3cada4aff (diff) | |
download | frameworks_base-15e7a21560764d644673854894eefbcd759d420a.zip frameworks_base-15e7a21560764d644673854894eefbcd759d420a.tar.gz frameworks_base-15e7a21560764d644673854894eefbcd759d420a.tar.bz2 |
Correct MediaAudioTrackTest testPlaybackHeadPositionAfterStop
When calling AudioTrack.stop the system needs some time to stop
the audiotrack and set the position to 0.
The current sleep time of 100ms is not enough.
Update the test to use the same strategy as the CTS test,
which is to call getPlaybackHeadPosition multiple time before
giving up and failing the test.
Change-Id: Ie938929c009c41e101782b4a46ac01a0aa73ee34
Diffstat (limited to 'media')
-rw-r--r-- | media/tests/MediaFrameworkTest/src/com/android/mediaframeworktest/functional/audio/MediaAudioTrackTest.java | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/media/tests/MediaFrameworkTest/src/com/android/mediaframeworktest/functional/audio/MediaAudioTrackTest.java b/media/tests/MediaFrameworkTest/src/com/android/mediaframeworktest/functional/audio/MediaAudioTrackTest.java index e884aba..eac5c28 100644 --- a/media/tests/MediaFrameworkTest/src/com/android/mediaframeworktest/functional/audio/MediaAudioTrackTest.java +++ b/media/tests/MediaFrameworkTest/src/com/android/mediaframeworktest/functional/audio/MediaAudioTrackTest.java @@ -386,6 +386,7 @@ public class MediaAudioTrackTest extends ActivityInstrumentationTestCase2<MediaF final int TEST_FORMAT = AudioFormat.ENCODING_PCM_16BIT; final int TEST_MODE = AudioTrack.MODE_STREAM; final int TEST_STREAM_TYPE = AudioManager.STREAM_MUSIC; + final int TEST_LOOP_CNT = 10; //-------- initialization -------------- int minBuffSize = AudioTrack.getMinBufferSize(TEST_SR, TEST_CONF, TEST_FORMAT); @@ -399,9 +400,14 @@ public class MediaAudioTrackTest extends ActivityInstrumentationTestCase2<MediaF track.play(); Thread.sleep(100); track.stop(); - Thread.sleep(100); // TODO: what is a sensible value? - int pos = track.getPlaybackHeadPosition(); - log(TEST_NAME, "position ="+ pos); + int count = 0; + int pos; + do { + Thread.sleep(200); + pos = track.getPlaybackHeadPosition(); + count++; + } while((pos != 0) && (count < TEST_LOOP_CNT)); + log(TEST_NAME, "position =" + pos + ", read count ="+count); assertTrue(TEST_NAME, pos == 0); //-------- tear down -------------- track.release(); |