Merge "Improving diagnostics for objects disappearing from screen" into sc-dev am: 5dae3f3162

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/14989389

Change-Id: I39008775e5557b5d0a21596550e4070b62b65f38
This commit is contained in:
TreeHugger Robot
2021-06-22 03:47:34 +00:00
committed by Automerger Merge Worker
2 changed files with 12 additions and 2 deletions
@@ -25,6 +25,7 @@ import androidx.annotation.NonNull;
import androidx.test.uiautomator.By;
import androidx.test.uiautomator.BySelector;
import androidx.test.uiautomator.Direction;
import androidx.test.uiautomator.StaleObjectException;
import androidx.test.uiautomator.UiObject2;
import com.android.launcher3.testing.TestProtocol;
@@ -61,7 +62,13 @@ public class AllApps extends LauncherInstrumentation.VisibleContainer {
private boolean hasClickableIcon(UiObject2 allAppsContainer, UiObject2 appListRecycler,
BySelector appIconSelector, int displayBottom) {
final UiObject2 icon = appListRecycler.findObject(appIconSelector);
final UiObject2 icon;
try {
icon = appListRecycler.findObject(appIconSelector);
} catch (StaleObjectException e) {
mLauncher.fail("All apps recycler disappeared from screen");
return false;
}
if (icon == null) {
LauncherInstrumentation.log("hasClickableIcon: icon not visible");
return false;
@@ -501,7 +501,7 @@ public final class LauncherInstrumentation {
}
}
private void fail(String message) {
void fail(String message) {
checkForAnomaly();
Assert.fail(formatSystemHealthMessage(formatErrorWithEvents(
"http://go/tapl test failure:\nSummary: " + getContextDescription()
@@ -1469,6 +1469,9 @@ public final class LauncherInstrumentation {
Rect getVisibleBounds(UiObject2 object) {
try {
return object.getVisibleBounds();
} catch (StaleObjectException e) {
fail("Object " + object + " disappeared from screen");
return null;
} catch (Throwable t) {
fail(t.toString());
return null;