From 22a86d7c8c281466e76ae3fd22f395eba80bb26f Mon Sep 17 00:00:00 2001 From: Alex Chau Date: Tue, 13 Jul 2021 16:23:51 +0100 Subject: [PATCH] Fix TaplTestQuickStep.testOverview for large screen cuttlefish - Use gesture (or button) to press home instead of UIDevice in the test, as UIDevice is sometimes failing to go home in cuttlefish Bug: 193440212 Test: TaplTestQuickStep.testOverview Change-Id: I834f1c93c24db24edbccd0bbf0ed5cfcf39a9b4d --- .../tests/src/com/android/quickstep/TaplTestsQuickstep.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/quickstep/tests/src/com/android/quickstep/TaplTestsQuickstep.java b/quickstep/tests/src/com/android/quickstep/TaplTestsQuickstep.java index 9a6eda4767..c14a5904f3 100644 --- a/quickstep/tests/src/com/android/quickstep/TaplTestsQuickstep.java +++ b/quickstep/tests/src/com/android/quickstep/TaplTestsQuickstep.java @@ -145,12 +145,8 @@ public class TaplTestsQuickstep extends AbstractQuickStepTest { launcher -> assertEquals("Dismissing a task didn't remove 1 task from Overview", numTasks - 1, getTaskCount(launcher))); - // Test UIDevice.pressHome, once we are in AllApps. - mDevice.pressHome(); - waitForState("Launcher internal state didn't switch to Home", () -> LauncherState.NORMAL); - // Test dismissing all tasks. - mLauncher.getWorkspace().switchToOverview().dismissAllTasks(); + mLauncher.pressHome().switchToOverview().dismissAllTasks(); waitForState("Launcher internal state didn't switch to Home", () -> LauncherState.NORMAL); executeOnLauncher( launcher -> assertEquals("Still have tasks after dismissing all",