diff options
author | Brad Stenning <stenning@google.com> | 2018-05-30 15:20:54 -0700 |
---|---|---|
committer | Brad Stenning <stenning@google.com> | 2018-05-30 15:20:54 -0700 |
commit | 20aa0fc555505f2c5d7f9badc46f21ed6a20631d (patch) | |
tree | 3f6d36fc039aa1d68a9aa9b3d1b4faaeb108110a /tests/android_support_car_api_test | |
parent | 5d1ddac6b5c186e2884a632b2afac32d30ecc922 (diff) | |
download | Car-20aa0fc555505f2c5d7f9badc46f21ed6a20631d.tar.gz |
Fix setup for test CarAppFocusManagerTest (the other ones)
The test setup was using a async call that would not finish before the first
assert. This will not block until the test environment is ready.
Bug:79152037
Test: run the test
Change-Id: I3914769b437e44e31b7799c7d28a9a7c5d440bbb
Diffstat (limited to 'tests/android_support_car_api_test')
-rw-r--r-- | tests/android_support_car_api_test/src/com/android/support/car/apitest/CarAppFocusManagerTest.java | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/tests/android_support_car_api_test/src/com/android/support/car/apitest/CarAppFocusManagerTest.java b/tests/android_support_car_api_test/src/com/android/support/car/apitest/CarAppFocusManagerTest.java index 31670a0ebb..2d58e563a2 100644 --- a/tests/android_support_car_api_test/src/com/android/support/car/apitest/CarAppFocusManagerTest.java +++ b/tests/android_support_car_api_test/src/com/android/support/car/apitest/CarAppFocusManagerTest.java @@ -36,10 +36,16 @@ public class CarAppFocusManagerTest extends CarApiTestBase { // Request all application focuses and abandon them to ensure no active context is present // when test starts. + int[] activeTypes = {CarAppFocusManager.APP_FOCUS_TYPE_NAVIGATION, + CarAppFocusManager.APP_FOCUS_TYPE_VOICE_COMMAND}; FocusOwnershipCallback owner = new FocusOwnershipCallback(); - mManager.requestAppFocus(CarAppFocusManager.APP_FOCUS_TYPE_NAVIGATION, owner); - mManager.requestAppFocus(CarAppFocusManager.APP_FOCUS_TYPE_VOICE_COMMAND, owner); - mManager.abandonAppFocus(owner); + for (int i = 0; i < activeTypes.length; i++) { + mManager.requestAppFocus(activeTypes[i], owner); + owner.waitForOwnershipGrantAndAssert(DEFAULT_WAIT_TIMEOUT_MS, activeTypes[i]); + mManager.abandonAppFocus(owner, activeTypes[i]); + owner.waitForOwnershipLossAndAssert( + DEFAULT_WAIT_TIMEOUT_MS, activeTypes[i]); + } } public void testSetActiveNullListener() throws Exception { |