From c0da8c3f1307d8fe6f2aad656d4cdbb3a26e1a61 Mon Sep 17 00:00:00 2001 From: Nataniel Borges Date: Mon, 8 Mar 2021 16:01:38 +0100 Subject: [PATCH] New enter pip assertion Rename previous assertion to "pip app always visible" and create a new one asserting that pip window becomes visible Test: atest FlickerTests:com.android.wm.shell.flicker.pip Bug: 167521701 Change-Id: I237c8369b58a766b60f4eb96b7dc465e86f96679 --- .../android/wm/shell/flicker/pip/EnterPipTest.kt | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/EnterPipTest.kt b/libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/EnterPipTest.kt index b7774046531c2..4847c98f54e96 100644 --- a/libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/EnterPipTest.kt +++ b/libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/EnterPipTest.kt @@ -16,6 +16,7 @@ package com.android.wm.shell.flicker.pip +import android.platform.test.annotations.Postsubmit import android.platform.test.annotations.Presubmit import android.view.Surface import androidx.test.filters.FlakyTest @@ -49,7 +50,7 @@ class EnterPipTest(testSpec: FlickerTestParameter) : PipTransition(testSpec) { @Presubmit @Test - fun pipWindowBecomesVisible() { + fun pipAppWindowAlwaysVisible() { testSpec.assertWm { this.showsAppWindow(pipApp.defaultWindowName) } @@ -63,6 +64,16 @@ class EnterPipTest(testSpec: FlickerTestParameter) : PipTransition(testSpec) { } } + @Postsubmit + @Test + fun pipWindowBecomesVisible() { + testSpec.assertWm { + invoke("pipWindowIsNotVisible") { !it.wmState.hasPipWindow() } + .then() + .invoke("pipWindowIsVisible") { it.wmState.hasPipWindow() } + } + } + @FlakyTest(bugId = 140855415) @Test override fun navBarLayerIsAlwaysVisible() = super.navBarLayerIsAlwaysVisible()