summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Yabin Huang <yabinh@google.com> 2024-07-11 13:12:37 -0700
committer Yabin Huang <yabinh@google.com> 2024-07-11 15:26:08 -0700
commit59517cb4c6d9b269dd6723688022ccaf56dfd047 (patch)
tree999e95fd217b095f34f2054b6553ece683d08609
parent93547bd63fca04530fd87356005b7c7b784a24a7 (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.java7
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);
}
/**