diff options
author | ckishan <ckishan@google.com> | 2023-03-30 08:46:16 +0000 |
---|---|---|
committer | Kishan Chandra <ckishan@google.com> | 2023-04-03 10:47:49 +0000 |
commit | 075ddc9258daaac01ca01ae1f18e9a41fb1ce30a (patch) | |
tree | 18c9541c1b533c5133e469152225f3221d8cc9d4 /service/src | |
parent | 7bd2a617bc4658fa9c4b14a0d1eeda86ae9ebfd6 (diff) | |
download | ImsMedia-075ddc9258daaac01ca01ae1f18e9a41fb1ce30a.tar.gz |
[ImsMedia] flaky test failure in ImsMediaJavaUnitTests
Flaky test failure due to different looper being used. Logic is modified
to use the same looper while performing UT.
Bug: 275030625
Test: atest -c ImsMediaJavaUnitTests --iterations 100
Basic functionality is verified manually
Change-Id: Ie17c820c8454117b109796c909e9b387da8e7241
Diffstat (limited to 'service/src')
3 files changed, 10 insertions, 9 deletions
diff --git a/service/src/com/android/telephony/imsmedia/AudioSession.java b/service/src/com/android/telephony/imsmedia/AudioSession.java index 603780c2..dae20c41 100644 --- a/service/src/com/android/telephony/imsmedia/AudioSession.java +++ b/service/src/com/android/telephony/imsmedia/AudioSession.java @@ -103,13 +103,14 @@ public final class AudioSession extends IImsAudioSession.Stub implements IMediaS @VisibleForTesting AudioSession(final int sessionId, - final @NonNull IImsAudioSessionCallback callback, - final @Nullable AudioService audioService, - final @Nullable AudioLocalSession localSession, - final @Nullable AudioOffloadService offloadService) { + @NonNull final IImsAudioSessionCallback callback, + @Nullable final AudioService audioService, + @Nullable final AudioLocalSession localSession, + @Nullable final AudioOffloadService offloadService, + Looper looper) { mSessionId = sessionId; mCallback = callback; - mHandler = new AudioSessionHandler(Looper.getMainLooper()); + mHandler = new AudioSessionHandler(looper); mAudioService = audioService; mLocalSession = localSession; mAudioListener = new AudioListener(mHandler); diff --git a/service/src/com/android/telephony/imsmedia/TextSession.java b/service/src/com/android/telephony/imsmedia/TextSession.java index b0895d8e..dd069a03 100644 --- a/service/src/com/android/telephony/imsmedia/TextSession.java +++ b/service/src/com/android/telephony/imsmedia/TextSession.java @@ -73,10 +73,10 @@ public final class TextSession extends IImsTextSession.Stub implements IMediaSes TextSession(final int sessionId, final @NonNull IImsTextSessionCallback callback, final @Nullable TextService textService, - final @Nullable TextLocalSession localSession) { + final @Nullable TextLocalSession localSession, Looper looper) { mSessionId = sessionId; mCallback = callback; - mHandler = new TextSessionHandler(Looper.getMainLooper()); + mHandler = new TextSessionHandler(looper); mTextService = textService; mLocalSession = localSession; mTextListener = new TextListener(mHandler); diff --git a/service/src/com/android/telephony/imsmedia/VideoSession.java b/service/src/com/android/telephony/imsmedia/VideoSession.java index a2351df9..5ade826e 100644 --- a/service/src/com/android/telephony/imsmedia/VideoSession.java +++ b/service/src/com/android/telephony/imsmedia/VideoSession.java @@ -85,10 +85,10 @@ public final class VideoSession extends IImsVideoSession.Stub implements IMediaS VideoSession(final int sessionId, final @NonNull IImsVideoSessionCallback callback, final @Nullable VideoService videoService, - final @Nullable VideoLocalSession localSession) { + final @Nullable VideoLocalSession localSession, Looper looper) { mSessionId = sessionId; mCallback = callback; - mHandler = new VideoSessionHandler(Looper.getMainLooper()); + mHandler = new VideoSessionHandler(looper); mVideoService = videoService; mLocalSession = localSession; mVideoListener = new VideoListener(mHandler); |