summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Pranav Madapurmath <pmadapurmath@google.com> 2025-01-30 20:46:45 -0800
committer Pranav Madapurmath <pmadapurmath@google.com> 2025-01-30 20:46:45 -0800
commit05be965a271de1389fc6e6b1fd56ff22261d58aa (patch)
treee731f8cd63699aa365f25b3e96125c08538f184b
parent582a607e1348cf780419fd62e59dba0908846417 (diff)
Resolve flaky audio routing CUJ tests
Cleans up the flaky audio routing tests which were verifying endpoints switches. It turned out that the test were flaky from other tests not properly cleaning up. The audio focus was not being abandoned and the routing hadn't been processed up to that point. This ended up leaking into subsequent tests where audio routing requests were piling up from multiple tests. Asserting MODE_NORMAL here isn't sufficient as it doesn't account for the audio routing to have been processed so waiting to process the tests with SystemClock.sleep is sufficient enough to give the routing enough time to finish processing the end of the call. Bug: 384101577 Test: atest CtsTelecomCujTestCases:SingleCallingTest Flag: EXEMPT test fix Change-Id: Iaa5b203b7d1d71fcd7fded2434f34e60566d882e
-rw-r--r--telecomm/java/com/android/internal/telecom/ITelecomService.aidl5
1 files changed, 5 insertions, 0 deletions
diff --git a/telecomm/java/com/android/internal/telecom/ITelecomService.aidl b/telecomm/java/com/android/internal/telecom/ITelecomService.aidl
index 4d9df4666016..6fb3bcb51d35 100644
--- a/telecomm/java/com/android/internal/telecom/ITelecomService.aidl
+++ b/telecomm/java/com/android/internal/telecom/ITelecomService.aidl
@@ -416,4 +416,9 @@ interface ITelecomService {
boolean hasForegroundServiceDelegation(in PhoneAccountHandle phoneAccountHandle,
String callingPackage);
void setMetricsTestMode(boolean enabled);
+
+ /**
+ * @see TelecomServiceImpl#waitForAudioToUpdate
+ */
+ void waitForAudioToUpdate(boolean expectActive);
}