From 39521ce7a5420493bdad6f599f515f7c2e9d1a75 Mon Sep 17 00:00:00 2001 From: Pat Manning Date: Tue, 15 Aug 2023 13:30:54 +0100 Subject: [PATCH] Add test to dismiss AllApps by swipe up to home gesture. Fix: 292080259 Test: TaplTestsLauncher3 Change-Id: I6053078cbfaf37d26c11376aecd21ba4edcbba44 --- .../launcher3/ui/TaplTestsLauncher3.java | 13 ++++++++++++- .../android/launcher3/tapl/HomeAllApps.java | 19 +++++++++++++------ 2 files changed, 25 insertions(+), 7 deletions(-) diff --git a/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java b/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java index 753d89d8b8..1ade1b0c55 100644 --- a/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java +++ b/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java @@ -229,7 +229,18 @@ public class TaplTestsLauncher3 extends AbstractLauncherUiTest { @PortraitLandscape public void testAllAppsSwitchToWorkspace() { assertNotNull("switchToWorkspace() returned null", - mLauncher.getWorkspace().switchToAllApps().switchToWorkspace()); + mLauncher.getWorkspace().switchToAllApps() + .switchToWorkspace(/* swipeDown= */ true)); + assertTrue("Launcher internal state is not Workspace", + isInState(() -> LauncherState.NORMAL)); + } + + @Test + @PortraitLandscape + public void testAllAppsSwipeUpToWorkspace() { + assertNotNull("testAllAppsSwipeUpToWorkspace() returned null", + mLauncher.getWorkspace().switchToAllApps() + .switchToWorkspace(/* swipeDown= */ false)); assertTrue("Launcher internal state is not Workspace", isInState(() -> LauncherState.NORMAL)); } diff --git a/tests/tapl/com/android/launcher3/tapl/HomeAllApps.java b/tests/tapl/com/android/launcher3/tapl/HomeAllApps.java index 1239d7a0b3..a03472a553 100644 --- a/tests/tapl/com/android/launcher3/tapl/HomeAllApps.java +++ b/tests/tapl/com/android/launcher3/tapl/HomeAllApps.java @@ -30,23 +30,28 @@ public class HomeAllApps extends AllApps { } /** - * Swipes down to Workspace. + * Swipes up or down to dismiss to Workspace. + * @param swipeDown Swipe all apps down to dismiss, otherwise swipe up to dismiss by going home. * * @return the Workspace object. */ @NonNull - public Workspace switchToWorkspace() { + public Workspace switchToWorkspace(boolean swipeDown) { try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); LauncherInstrumentation.Closable c = mLauncher.addContextLayer("want to switch from all apps to workspace")) { UiObject2 allAppsContainer = verifyActiveContainer(); final int startX = allAppsContainer.getVisibleCenter().x; - final int startY = getTopVisibleIconBounds(allAppsContainer).centerY(); - final int endY = mLauncher.getDevice().getDisplayHeight(); + final int startY = swipeDown ? getTopVisibleIconBounds(allAppsContainer).centerY() + : mLauncher.getDevice().getDisplayHeight(); + final int endY = + swipeDown ? mLauncher.getDevice().getDisplayHeight() : getTopVisibleIconBounds( + allAppsContainer).centerY(); LauncherInstrumentation.log( "switchToWorkspace: startY = " + startY + ", endY = " + endY - + ", slop = " + mLauncher.getTouchSlop()); + + ", slop = " + mLauncher.getTouchSlop() + + ", swipeDown = " + swipeDown); mLauncher.swipeToState( startX, @@ -54,7 +59,9 @@ public class HomeAllApps extends AllApps { startX, endY, 5 /* steps */, - NORMAL_STATE_ORDINAL, LauncherInstrumentation.GestureScope.INSIDE); + NORMAL_STATE_ORDINAL, + swipeDown ? LauncherInstrumentation.GestureScope.INSIDE + : LauncherInstrumentation.GestureScope.OUTSIDE_WITH_PILFER); try (LauncherInstrumentation.Closable c1 = mLauncher.addContextLayer( "swiped to workspace")) {