diff --git a/src/com/android/launcher3/testing/TestLogging.java b/src/com/android/launcher3/testing/TestLogging.java index 024110da5a..fd066c1ce1 100644 --- a/src/com/android/launcher3/testing/TestLogging.java +++ b/src/com/android/launcher3/testing/TestLogging.java @@ -21,7 +21,7 @@ import android.util.Log; import com.android.launcher3.Utilities; public final class TestLogging { - public static synchronized void recordEvent(String event) { + public static void recordEvent(String event) { if (Utilities.IS_RUNNING_IN_TEST_HARNESS) { Log.d(TestProtocol.TAPL_EVENTS_TAG, event); } diff --git a/tests/tapl/com/android/launcher3/tapl/AddToHomeScreenPrompt.java b/tests/tapl/com/android/launcher3/tapl/AddToHomeScreenPrompt.java index 03d1600783..afb50e06f3 100644 --- a/tests/tapl/com/android/launcher3/tapl/AddToHomeScreenPrompt.java +++ b/tests/tapl/com/android/launcher3/tapl/AddToHomeScreenPrompt.java @@ -37,8 +37,10 @@ public class AddToHomeScreenPrompt { } public void addAutomatically() { - mLauncher.waitForObjectInContainer( - mWidgetCell.getParent().getParent().getParent().getParent(), - By.text(ADD_AUTOMATICALLY)).click(); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + mLauncher.waitForObjectInContainer( + mWidgetCell.getParent().getParent().getParent().getParent(), + By.text(ADD_AUTOMATICALLY)).click(); + } } } diff --git a/tests/tapl/com/android/launcher3/tapl/AllApps.java b/tests/tapl/com/android/launcher3/tapl/AllApps.java index 3bdeb14925..cab7f54712 100644 --- a/tests/tapl/com/android/launcher3/tapl/AllApps.java +++ b/tests/tapl/com/android/launcher3/tapl/AllApps.java @@ -99,8 +99,9 @@ public class AllApps extends LauncherInstrumentation.VisibleContainer { */ @NonNull public AppIcon getAppIcon(String appName) { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "getting app icon " + appName + " on all apps")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "getting app icon " + appName + " on all apps")) { final UiObject2 allAppsContainer = verifyActiveContainer(); final UiObject2 appListRecycler = mLauncher.waitForObjectInContainer(allAppsContainer, "apps_list_view"); @@ -197,7 +198,8 @@ public class AllApps extends LauncherInstrumentation.VisibleContainer { * Flings forward (down) and waits the fling's end. */ public void flingForward() { - try (LauncherInstrumentation.Closable c = + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer("want to fling forward in all apps")) { final UiObject2 allAppsContainer = verifyActiveContainer(); // Start the gesture in the center to avoid starting at elements near the top. @@ -211,7 +213,8 @@ public class AllApps extends LauncherInstrumentation.VisibleContainer { * Flings backward (up) and waits the fling's end. */ public void flingBackward() { - try (LauncherInstrumentation.Closable c = + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer("want to fling backward in all apps")) { final UiObject2 allAppsContainer = verifyActiveContainer(); // Start the gesture in the center, for symmetry with forward. diff --git a/tests/tapl/com/android/launcher3/tapl/AllAppsFromOverview.java b/tests/tapl/com/android/launcher3/tapl/AllAppsFromOverview.java index f48d4dd4c7..8f9fec91f2 100644 --- a/tests/tapl/com/android/launcher3/tapl/AllAppsFromOverview.java +++ b/tests/tapl/com/android/launcher3/tapl/AllAppsFromOverview.java @@ -42,8 +42,9 @@ public final class AllAppsFromOverview extends AllApps { */ @NonNull public Overview switchBackToOverview() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to switch back from all apps to overview")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to switch back from all apps to overview")) { final UiObject2 allAppsContainer = verifyActiveContainer(); // Swipe from the search box to the bottom. final UiObject2 qsb = mLauncher.waitForObjectInContainer( diff --git a/tests/tapl/com/android/launcher3/tapl/AppIcon.java b/tests/tapl/com/android/launcher3/tapl/AppIcon.java index db685dbb2f..0a6ed7f2e9 100644 --- a/tests/tapl/com/android/launcher3/tapl/AppIcon.java +++ b/tests/tapl/com/android/launcher3/tapl/AppIcon.java @@ -39,8 +39,10 @@ public final class AppIcon extends Launchable { * Long-clicks the icon to open its menu. */ public AppIconMenu openMenu() { - return new AppIconMenu(mLauncher, mLauncher.clickAndGet( - mObject, "deep_shortcuts_container")); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + return new AppIconMenu(mLauncher, mLauncher.clickAndGet( + mObject, "deep_shortcuts_container")); + } } @Override diff --git a/tests/tapl/com/android/launcher3/tapl/Background.java b/tests/tapl/com/android/launcher3/tapl/Background.java index d9ae778c7c..50bdf5c54e 100644 --- a/tests/tapl/com/android/launcher3/tapl/Background.java +++ b/tests/tapl/com/android/launcher3/tapl/Background.java @@ -52,8 +52,9 @@ public class Background extends LauncherInstrumentation.VisibleContainer { */ @NonNull public BaseOverview switchToOverview() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to switch from background to overview")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to switch from background to overview")) { verifyActiveContainer(); goToOverviewUnchecked(); return mLauncher.isFallbackOverview() ? @@ -124,8 +125,9 @@ public class Background extends LauncherInstrumentation.VisibleContainer { * Swipes right or double presses the square button to switch to the previous app. */ public Background quickSwitchToPreviousApp() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to quick switch to the previous app")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to quick switch to the previous app")) { verifyActiveContainer(); quickSwitchToPreviousApp(getExpectedStateForQuickSwitch()); return new Background(mLauncher); diff --git a/tests/tapl/com/android/launcher3/tapl/BaseOverview.java b/tests/tapl/com/android/launcher3/tapl/BaseOverview.java index 339e14feca..e13ea52001 100644 --- a/tests/tapl/com/android/launcher3/tapl/BaseOverview.java +++ b/tests/tapl/com/android/launcher3/tapl/BaseOverview.java @@ -48,6 +48,12 @@ public class BaseOverview extends LauncherInstrumentation.VisibleContainer { * Flings forward (left) and waits the fling's end. */ public void flingForward() { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + flingForwardImpl(); + } + } + + private void flingForwardImpl() { try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer("want to fling forward in overview")) { LauncherInstrumentation.log("Overview.flingForward before fling"); @@ -65,14 +71,15 @@ public class BaseOverview extends LauncherInstrumentation.VisibleContainer { * Dismissed all tasks by scrolling to Clear-all button and pressing it. */ public void dismissAllTasks() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "dismissing all tasks")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "dismissing all tasks")) { final BySelector clearAllSelector = mLauncher.getOverviewObjectSelector("clear_all"); for (int i = 0; i < FLINGS_FOR_DISMISS_LIMIT && !verifyActiveContainer().hasObject(clearAllSelector); ++i) { - flingForward(); + flingForwardImpl(); } mLauncher.waitForObjectInContainer(verifyActiveContainer(), clearAllSelector).click(); @@ -83,7 +90,8 @@ public class BaseOverview extends LauncherInstrumentation.VisibleContainer { * Flings backward (right) and waits the fling's end. */ public void flingBackward() { - try (LauncherInstrumentation.Closable c = + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer("want to fling backward in overview")) { LauncherInstrumentation.log("Overview.flingBackward before fling"); final UiObject2 overview = verifyActiveContainer(); diff --git a/tests/tapl/com/android/launcher3/tapl/Home.java b/tests/tapl/com/android/launcher3/tapl/Home.java index 1e4d937a81..1a0fe3d1e4 100644 --- a/tests/tapl/com/android/launcher3/tapl/Home.java +++ b/tests/tapl/com/android/launcher3/tapl/Home.java @@ -48,8 +48,9 @@ public abstract class Home extends Background { @NonNull @Override public Overview switchToOverview() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to switch from home to overview")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to switch from home to overview")) { verifyActiveContainer(); goToOverviewUnchecked(); try (LauncherInstrumentation.Closable c1 = mLauncher.addContextLayer( diff --git a/tests/tapl/com/android/launcher3/tapl/Launchable.java b/tests/tapl/com/android/launcher3/tapl/Launchable.java index 3aedf99a1c..f88a616c4d 100644 --- a/tests/tapl/com/android/launcher3/tapl/Launchable.java +++ b/tests/tapl/com/android/launcher3/tapl/Launchable.java @@ -46,7 +46,9 @@ abstract class Launchable { * Clicks the object to launch its app. */ public Background launch(String expectedPackageName) { - return launch(By.pkg(expectedPackageName)); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + return launch(By.pkg(expectedPackageName)); + } } private Background launch(BySelector selector) { diff --git a/tests/tapl/com/android/launcher3/tapl/LauncherInstrumentation.java b/tests/tapl/com/android/launcher3/tapl/LauncherInstrumentation.java index 196c6b749d..6df8790564 100644 --- a/tests/tapl/com/android/launcher3/tapl/LauncherInstrumentation.java +++ b/tests/tapl/com/android/launcher3/tapl/LauncherInstrumentation.java @@ -560,61 +560,63 @@ public final class LauncherInstrumentation { * @return the Workspace object. */ public Workspace pressHome() { - // Click home, then wait for any accessibility event, then wait until accessibility events - // stop. - // We need waiting for any accessibility event generated after pressing Home because - // otherwise waitForIdle may return immediately in case when there was a big enough pause in - // accessibility events prior to pressing Home. - final String action; - if (getNavigationModel() == NavigationModel.ZERO_BUTTON) { - checkForAnomaly(); + try (LauncherInstrumentation.Closable e = eventsCheck()) { + // Click home, then wait for any accessibility event, then wait until accessibility + // events stop. + // We need waiting for any accessibility event generated after pressing Home because + // otherwise waitForIdle may return immediately in case when there was a big enough + // pause in accessibility events prior to pressing Home. + final String action; + if (getNavigationModel() == NavigationModel.ZERO_BUTTON) { + checkForAnomaly(); - final Point displaySize = getRealDisplaySize(); + final Point displaySize = getRealDisplaySize(); - if (hasLauncherObject(CONTEXT_MENU_RES_ID)) { - linearGesture( - displaySize.x / 2, displaySize.y - 1, - displaySize.x / 2, 0, - ZERO_BUTTON_STEPS_FROM_BACKGROUND_TO_HOME, - false); - try (LauncherInstrumentation.Closable c = addContextLayer( - "Swiped up from context menu to home")) { - waitUntilGone(CONTEXT_MENU_RES_ID); - } - } - if (hasLauncherObject(WORKSPACE_RES_ID)) { - log(action = "already at home"); - } else { - log("Hierarchy before swiping up to home:"); - dumpViewHierarchy(); - log(action = "swiping up to home from " + getVisibleStateMessage()); - - try (LauncherInstrumentation.Closable c = addContextLayer(action)) { - swipeToState( + if (hasLauncherObject(CONTEXT_MENU_RES_ID)) { + linearGesture( displaySize.x / 2, displaySize.y - 1, displaySize.x / 2, 0, - ZERO_BUTTON_STEPS_FROM_BACKGROUND_TO_HOME, NORMAL_STATE_ORDINAL); + ZERO_BUTTON_STEPS_FROM_BACKGROUND_TO_HOME, + false); + try (LauncherInstrumentation.Closable c = addContextLayer( + "Swiped up from context menu to home")) { + waitUntilGone(CONTEXT_MENU_RES_ID); + } + } + if (hasLauncherObject(WORKSPACE_RES_ID)) { + log(action = "already at home"); + } else { + log("Hierarchy before swiping up to home:"); + dumpViewHierarchy(); + log(action = "swiping up to home from " + getVisibleStateMessage()); + + try (LauncherInstrumentation.Closable c = addContextLayer(action)) { + swipeToState( + displaySize.x / 2, displaySize.y - 1, + displaySize.x / 2, 0, + ZERO_BUTTON_STEPS_FROM_BACKGROUND_TO_HOME, NORMAL_STATE_ORDINAL); + } + } + } else { + log("Hierarchy before clicking home:"); + dumpViewHierarchy(); + log(action = "clicking home button from " + getVisibleStateMessage()); + try (LauncherInstrumentation.Closable c = addContextLayer(action)) { + mDevice.waitForIdle(); + runToState( + waitForSystemUiObject("home")::click, + NORMAL_STATE_ORDINAL, + !hasLauncherObject(WORKSPACE_RES_ID) + && (hasLauncherObject(APPS_RES_ID) + || hasLauncherObject(OVERVIEW_RES_ID))); + mDevice.waitForIdle(); } } - } else { - log("Hierarchy before clicking home:"); - dumpViewHierarchy(); - log(action = "clicking home button from " + getVisibleStateMessage()); - try (LauncherInstrumentation.Closable c = addContextLayer(action)) { - mDevice.waitForIdle(); - runToState( - waitForSystemUiObject("home")::click, - NORMAL_STATE_ORDINAL, - !hasLauncherObject(WORKSPACE_RES_ID) - && (hasLauncherObject(APPS_RES_ID) - || hasLauncherObject(OVERVIEW_RES_ID))); - mDevice.waitForIdle(); + try (LauncherInstrumentation.Closable c = addContextLayer( + "performed action to switch to Home - " + action)) { + return getWorkspace(); } } - try (LauncherInstrumentation.Closable c = addContextLayer( - "performed action to switch to Home - " + action)) { - return getWorkspace(); - } } /** @@ -1218,6 +1220,6 @@ public final class LauncherInstrumentation { private String formatEventMismatchMessage(String message, List actual, int position) { return message + ", pos=" + position + ", expected=" + mExpectedEvents - + ", actual" + actual; + + ", actual=" + actual; } } \ No newline at end of file diff --git a/tests/tapl/com/android/launcher3/tapl/OptionsPopupMenuItem.java b/tests/tapl/com/android/launcher3/tapl/OptionsPopupMenuItem.java index 8527d05998..461610db49 100644 --- a/tests/tapl/com/android/launcher3/tapl/OptionsPopupMenuItem.java +++ b/tests/tapl/com/android/launcher3/tapl/OptionsPopupMenuItem.java @@ -35,12 +35,14 @@ public class OptionsPopupMenuItem { */ @NonNull public void launch(@NonNull String expectedPackageName) { - LauncherInstrumentation.log("OptionsPopupMenuItem before click " - + mObject.getVisibleCenter() + " in " + mObject.getVisibleBounds()); - mObject.click(); - mLauncher.assertTrue( - "App didn't start: " + By.pkg(expectedPackageName), - mLauncher.getDevice().wait(Until.hasObject(By.pkg(expectedPackageName)), - LauncherInstrumentation.WAIT_TIME_MS)); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + LauncherInstrumentation.log("OptionsPopupMenuItem before click " + + mObject.getVisibleCenter() + " in " + mObject.getVisibleBounds()); + mObject.click(); + mLauncher.assertTrue( + "App didn't start: " + By.pkg(expectedPackageName), + mLauncher.getDevice().wait(Until.hasObject(By.pkg(expectedPackageName)), + LauncherInstrumentation.WAIT_TIME_MS)); + } } } diff --git a/tests/tapl/com/android/launcher3/tapl/Overview.java b/tests/tapl/com/android/launcher3/tapl/Overview.java index 16a64a7499..6622c6e6a7 100644 --- a/tests/tapl/com/android/launcher3/tapl/Overview.java +++ b/tests/tapl/com/android/launcher3/tapl/Overview.java @@ -45,8 +45,9 @@ public final class Overview extends BaseOverview { */ @NonNull public AllAppsFromOverview switchToAllApps() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to switch from overview to all apps")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to switch from overview to all apps")) { verifyActiveContainer(); // Swipe from an app icon to the top. diff --git a/tests/tapl/com/android/launcher3/tapl/OverviewTask.java b/tests/tapl/com/android/launcher3/tapl/OverviewTask.java index 46f8ba57a4..0d93cbcd32 100644 --- a/tests/tapl/com/android/launcher3/tapl/OverviewTask.java +++ b/tests/tapl/com/android/launcher3/tapl/OverviewTask.java @@ -45,8 +45,9 @@ public final class OverviewTask { * Swipes the task up. */ public void dismiss() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to dismiss a task")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to dismiss a task")) { verifyActiveContainer(); // Dismiss the task via flinging it up. final Rect taskBounds = mTask.getVisibleBounds(); @@ -61,15 +62,17 @@ public final class OverviewTask { * Clicks at the task. */ public Background open() { - verifyActiveContainer(); - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "clicking an overview task")) { - mLauncher.executeAndWaitForEvent( - () -> mTask.click(), - event -> event.getEventType() == TYPE_WINDOW_STATE_CHANGED, - () -> "Launching task didn't open a new window: " - + mTask.getParent().getContentDescription()); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + verifyActiveContainer(); + try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "clicking an overview task")) { + mLauncher.executeAndWaitForEvent( + () -> mTask.click(), + event -> event.getEventType() == TYPE_WINDOW_STATE_CHANGED, + () -> "Launching task didn't open a new window: " + + mTask.getParent().getContentDescription()); + } + return new Background(mLauncher); } - return new Background(mLauncher); } } diff --git a/tests/tapl/com/android/launcher3/tapl/Widgets.java b/tests/tapl/com/android/launcher3/tapl/Widgets.java index d208c667b7..ede5bd9e41 100644 --- a/tests/tapl/com/android/launcher3/tapl/Widgets.java +++ b/tests/tapl/com/android/launcher3/tapl/Widgets.java @@ -41,8 +41,9 @@ public final class Widgets extends LauncherInstrumentation.VisibleContainer { * Flings forward (down) and waits the fling's end. */ public void flingForward() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to fling forward in widgets")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to fling forward in widgets")) { LauncherInstrumentation.log("Widgets.flingForward enter"); final UiObject2 widgetsContainer = verifyActiveContainer(); mLauncher.scroll( @@ -62,8 +63,9 @@ public final class Widgets extends LauncherInstrumentation.VisibleContainer { * Flings backward (up) and waits the fling's end. */ public void flingBackward() { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "want to fling backwards in widgets")) { + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "want to fling backwards in widgets")) { LauncherInstrumentation.log("Widgets.flingBackward enter"); final UiObject2 widgetsContainer = verifyActiveContainer(); mLauncher.scroll( diff --git a/tests/tapl/com/android/launcher3/tapl/Workspace.java b/tests/tapl/com/android/launcher3/tapl/Workspace.java index af7e552b37..1d2c821066 100644 --- a/tests/tapl/com/android/launcher3/tapl/Workspace.java +++ b/tests/tapl/com/android/launcher3/tapl/Workspace.java @@ -85,7 +85,8 @@ public final class Workspace extends Home { */ @NonNull public AllApps switchToAllApps() { - try (LauncherInstrumentation.Closable c = + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck(); + LauncherInstrumentation.Closable c = mLauncher.addContextLayer("want to switch from workspace to all apps")) { verifyActiveContainer(); final int deviceHeight = mLauncher.getDevice().getDisplayHeight(); @@ -156,21 +157,23 @@ public final class Workspace extends Home { * second screen. */ public void ensureWorkspaceIsScrollable() { - final UiObject2 workspace = verifyActiveContainer(); - if (!isWorkspaceScrollable(workspace)) { - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( - "dragging icon to a second page of workspace to make it scrollable")) { - dragIconToWorkspace( - mLauncher, - getHotseatAppIcon("Chrome"), - new Point(mLauncher.getDevice().getDisplayWidth(), - workspace.getVisibleBounds().centerY()), - "deep_shortcuts_container"); - verifyActiveContainer(); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + final UiObject2 workspace = verifyActiveContainer(); + if (!isWorkspaceScrollable(workspace)) { + try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer( + "dragging icon to a second page of workspace to make it scrollable")) { + dragIconToWorkspace( + mLauncher, + getHotseatAppIcon("Chrome"), + new Point(mLauncher.getDevice().getDisplayWidth(), + workspace.getVisibleBounds().centerY()), + "deep_shortcuts_container"); + verifyActiveContainer(); + } } + assertTrue("Home screen workspace didn't become scrollable", + isWorkspaceScrollable(workspace)); } - assertTrue("Home screen workspace didn't become scrollable", - isWorkspaceScrollable(workspace)); } private boolean isWorkspaceScrollable(UiObject2 workspace) { @@ -213,11 +216,13 @@ public final class Workspace extends Home { * recoil to complete. */ public void flingForward() { - final UiObject2 workspace = verifyActiveContainer(); - mLauncher.scroll(workspace, Direction.RIGHT, - new Rect(0, 0, mLauncher.getEdgeSensitivityWidth() + 1, 0), - FLING_STEPS, false); - verifyActiveContainer(); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + final UiObject2 workspace = verifyActiveContainer(); + mLauncher.scroll(workspace, Direction.RIGHT, + new Rect(0, 0, mLauncher.getEdgeSensitivityWidth() + 1, 0), + FLING_STEPS, false); + verifyActiveContainer(); + } } /** @@ -225,11 +230,13 @@ public final class Workspace extends Home { * recoil to complete. */ public void flingBackward() { - final UiObject2 workspace = verifyActiveContainer(); - mLauncher.scroll(workspace, Direction.LEFT, - new Rect(mLauncher.getEdgeSensitivityWidth() + 1, 0, 0, 0), - FLING_STEPS, false); - verifyActiveContainer(); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + final UiObject2 workspace = verifyActiveContainer(); + mLauncher.scroll(workspace, Direction.LEFT, + new Rect(mLauncher.getEdgeSensitivityWidth() + 1, 0, 0, 0), + FLING_STEPS, false); + verifyActiveContainer(); + } } /** @@ -239,10 +246,12 @@ public final class Workspace extends Home { */ @NonNull public Widgets openAllWidgets() { - verifyActiveContainer(); - mLauncher.getDevice().pressKeyCode(KeyEvent.KEYCODE_W, KeyEvent.META_CTRL_ON); - try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer("pressed Ctrl+W")) { - return new Widgets(mLauncher); + try (LauncherInstrumentation.Closable e = mLauncher.eventsCheck()) { + verifyActiveContainer(); + mLauncher.getDevice().pressKeyCode(KeyEvent.KEYCODE_W, KeyEvent.META_CTRL_ON); + try (LauncherInstrumentation.Closable c = mLauncher.addContextLayer("pressed Ctrl+W")) { + return new Widgets(mLauncher); + } } }