From 563c4384bb55c1f5ea7495ac8ded8b3cf0d75720 Mon Sep 17 00:00:00 2001 From: Sebastian Franco Date: Mon, 28 Aug 2023 13:13:13 -0700 Subject: [PATCH] Adding wait to test if the flake is a race condition Test: atest getIconsPosition_afterIconRemoved_notContained Test: postsubmit flake percentage Bug: 288944469 Change-Id: I51253c066159fbc4d347bceff0f4017c03b01e74 --- tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java b/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java index 168ed9f18c..0aa9e4f3f2 100644 --- a/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java +++ b/tests/src/com/android/launcher3/ui/TaplTestsLauncher3.java @@ -621,6 +621,10 @@ public class TaplTestsLauncher3 extends AbstractLauncherUiTest { mLauncher.getWorkspace().verifyWorkspaceAppIconIsGone( DUMMY_APP_NAME + " was expected to disappear after uninstall.", DUMMY_APP_NAME); + // Debug for b/288944469 I want to test if we are not waiting enough after removing + // the icon to request the list of icons again, since the items are not removed + // immediately. This should reduce the flake rate + SystemClock.sleep(500); Map finalPositions = mLauncher.getWorkspace().getWorkspaceIconsPositions(); assertThat(finalPositions).doesNotContainKey(DUMMY_APP_NAME);