From 3abc8511a51afe983b481bdf6631535aa8b94f28 Mon Sep 17 00:00:00 2001 From: vadimt Date: Thu, 5 Mar 2020 14:01:23 -0800 Subject: [PATCH] Enabling widget config tests after fixing flakiness Bug: 148867106 Change-Id: I8bbd9ef9b1ca574f79f3f76869051495b59734ce --- .../com/android/launcher3/ui/widget/AddConfigWidgetTest.java | 3 --- tests/tapl/com/android/launcher3/tapl/Widgets.java | 3 ++- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/src/com/android/launcher3/ui/widget/AddConfigWidgetTest.java b/tests/src/com/android/launcher3/ui/widget/AddConfigWidgetTest.java index 4b1a06799b..de9757fa58 100644 --- a/tests/src/com/android/launcher3/ui/widget/AddConfigWidgetTest.java +++ b/tests/src/com/android/launcher3/ui/widget/AddConfigWidgetTest.java @@ -41,7 +41,6 @@ import com.android.launcher3.util.Wait.Condition; import com.android.launcher3.util.rule.ShellCommandRule; import org.junit.Before; -import org.junit.Ignore; import org.junit.Rule; import org.junit.Test; import org.junit.runner.RunWith; @@ -71,14 +70,12 @@ public class AddConfigWidgetTest extends AbstractLauncherUiTest { @Test @PortraitLandscape - @Ignore // b/148867106 public void testWidgetConfig() throws Throwable { runTest(true); } @Test @PortraitLandscape - @Ignore // b/148867106 public void testConfigCancelled() throws Throwable { runTest(false); } diff --git a/tests/tapl/com/android/launcher3/tapl/Widgets.java b/tests/tapl/com/android/launcher3/tapl/Widgets.java index 8659aa7a35..084138cb5d 100644 --- a/tests/tapl/com/android/launcher3/tapl/Widgets.java +++ b/tests/tapl/com/android/launcher3/tapl/Widgets.java @@ -115,7 +115,8 @@ public final class Widgets extends LauncherInstrumentation.VisibleContainer { int visibleDelta = maxWidth - widget.getVisibleBounds().width(); if (visibleDelta > 0) { Rect parentBounds = cell.getVisibleBounds(); - mLauncher.linearGesture(parentBounds.centerX() + visibleDelta, + mLauncher.linearGesture(parentBounds.centerX() + visibleDelta + + mLauncher.getTouchSlop(), parentBounds.centerY(), parentBounds.centerX(), parentBounds.centerY(), 10, true, GestureScope.INSIDE); }