diff options
author | 2024-07-11 13:12:37 -0700 | |
---|---|---|
committer | 2024-07-11 15:26:08 -0700 | |
commit | 59517cb4c6d9b269dd6723688022ccaf56dfd047 (patch) | |
tree | 999e95fd217b095f34f2054b6553ece683d08609 | |
parent | 93547bd63fca04530fd87356005b7c7b784a24a7 (diff) |
Fix flaky passengerShowImeNotAffectDriver()
The test was flaky because sometimes the app window on the passenger
display stayed black forever after the test activity was launched on
the passenger display.
This CL fixed by removing problematic and unnecessary window animation.
Bug: 350562427
Test: atest com.android.server.inputmethod.multisessiontest.ConcurrentMultiUserTest --iterations 100
Flag: TEST_ONLY
Change-Id: Ieec9d815a85dfd218cf51f5960cb4172ba537af1
-rw-r--r-- | tests/inputmethod/ConcurrentMultiSessionImeTest/src/com/android/server/inputmethod/multisessiontest/ConcurrentMultiUserTest.java | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/tests/inputmethod/ConcurrentMultiSessionImeTest/src/com/android/server/inputmethod/multisessiontest/ConcurrentMultiUserTest.java b/tests/inputmethod/ConcurrentMultiSessionImeTest/src/com/android/server/inputmethod/multisessiontest/ConcurrentMultiUserTest.java index 8433071506d9..f3851790688e 100644 --- a/tests/inputmethod/ConcurrentMultiSessionImeTest/src/com/android/server/inputmethod/multisessiontest/ConcurrentMultiUserTest.java +++ b/tests/inputmethod/ConcurrentMultiSessionImeTest/src/com/android/server/inputmethod/multisessiontest/ConcurrentMultiUserTest.java @@ -54,7 +54,6 @@ import com.android.compatibility.common.util.SystemUtil; import org.junit.After; import org.junit.Before; import org.junit.ClassRule; -import org.junit.Ignore; import org.junit.Rule; import org.junit.Test; import org.junit.runner.RunWith; @@ -72,7 +71,6 @@ public final class ConcurrentMultiUserTest { private static final ComponentName TEST_ACTIVITY = new ComponentName( getInstrumentation().getTargetContext().getPackageName(), MainActivity.class.getName()); - private static final long WAIT_TIME_MS = 3000L; private final Context mContext = getInstrumentation().getTargetContext(); private final InputMethodManager mInputMethodManager = mContext.getSystemService(InputMethodManager.class); @@ -111,7 +109,6 @@ public final class ConcurrentMultiUserTest { assertPassengerImeHidden(); } - @Ignore("b/350562427") @Test public void passengerShowImeNotAffectDriver() throws Exception { assertDriverImeHidden(); @@ -259,8 +256,6 @@ public final class ConcurrentMultiUserTest { float[] driverEditTextCenter = mActivity.getEditTextCenter(); SystemUtil.runShellCommand(mUiAutomation, String.format("input tap %f %f", driverEditTextCenter[0], driverEditTextCenter[1])); - // TODO(b/350562427): get rid of Thread.sleep(). - Thread.sleep(WAIT_TIME_MS); } private void movePassengerDisplayToTop() throws Exception { @@ -276,8 +271,6 @@ public final class ConcurrentMultiUserTest { final int passengerDisplayId = receivedBundle.getInt(KEY_DISPLAY_ID); SystemUtil.runShellCommand(mUiAutomation, String.format("input -d %d tap %f %f", passengerDisplayId, passengerEditTextCenter[0], passengerEditTextCenter[1])); - // TODO(b/350562427): get rid of Thread.sleep(). - Thread.sleep(WAIT_TIME_MS); } /** |