From 006db80dae22e1be5c15ea7c6c5c604fd7f5b793 Mon Sep 17 00:00:00 2001 From: Nataniel Borges Date: Tue, 26 May 2020 11:31:50 +0200 Subject: [PATCH] Enable previously flaky IME related tests The IME layer is sometimes present but hidden on the hierarchy. The tests should ignore this condition (skip until first assertion) to ensure the tests are no longer flaky. Test: atest FlickerTests Change-Id: Ie1c9daf006eac215d310cfbb66df96501e910b5f --- .../android/server/wm/flicker/CloseImeWindowToHomeTest.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tests/FlickerTests/src/com/android/server/wm/flicker/CloseImeWindowToHomeTest.java b/tests/FlickerTests/src/com/android/server/wm/flicker/CloseImeWindowToHomeTest.java index b2dd959ce9b2a..36f4ba3b79250 100644 --- a/tests/FlickerTests/src/com/android/server/wm/flicker/CloseImeWindowToHomeTest.java +++ b/tests/FlickerTests/src/com/android/server/wm/flicker/CloseImeWindowToHomeTest.java @@ -24,7 +24,6 @@ import androidx.test.filters.LargeTest; import com.android.server.wm.flicker.helpers.ImeAppHelper; import org.junit.FixMethodOrder; -import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.MethodSorters; @@ -62,20 +61,20 @@ public class CloseImeWindowToHomeTest extends NonRotationTestBase { .forAllEntries()); } - @Ignore("Flaky") @Test public void checkVisibility_imeLayerBecomesInvisible() { checkResults(result -> LayersTraceSubject.assertThat(result) + .skipUntilFirstAssertion() .showsLayer(IME_WINDOW_TITLE) .then() .hidesLayer(IME_WINDOW_TITLE) .forAllEntries()); } - @Ignore("Flaky") @Test public void checkVisibility_imeAppLayerBecomesInvisible() { checkResults(result -> LayersTraceSubject.assertThat(result) + .skipUntilFirstAssertion() .showsLayer(mTestApp.getPackage()) .then() .hidesLayer(mTestApp.getPackage())