Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1a86eef7 authored by Pablo Gamito's avatar Pablo Gamito
Browse files

Promote stable Flicker tests

Bug: 228012334
Bug: 240499181
Bug: 227313015
Bug: 197726610
Bug: 239807171

Test: N/A
Change-Id: I057422a547e3f75ef398996c542c9753a0968a60
parent 240763b8
Loading
Loading
Loading
Loading
+0 −10
Original line number Diff line number Diff line
@@ -16,14 +16,12 @@

package com.android.wm.shell.flicker.pip

import android.platform.test.annotations.FlakyTest
import android.platform.test.annotations.Presubmit
import android.view.Surface
import androidx.test.filters.RequiresDevice
import com.android.server.wm.flicker.FlickerParametersRunnerFactory
import com.android.server.wm.flicker.FlickerTestParameter
import com.android.server.wm.flicker.dsl.FlickerBuilder
import com.android.server.wm.flicker.helpers.isShellTransitionsEnabled
import com.android.server.wm.flicker.helpers.setRotation
import com.android.server.wm.flicker.helpers.wakeUpAndGoToHomeScreen
import com.android.server.wm.flicker.rules.RemoveAllTasksButHomeRule
@@ -108,14 +106,6 @@ class EnterPipOnUserLeaveHintTest(testSpec: FlickerTestParameter) : EnterPipTest
    @Presubmit
    @Test
    override fun entireScreenCovered() {
        Assume.assumeFalse(isShellTransitionsEnabled)
        super.entireScreenCovered()
    }

    @FlakyTest(bugId = 227313015)
    @Test
    fun entireScreenCovered_ShellTransit() {
        Assume.assumeTrue(isShellTransitionsEnabled)
        super.entireScreenCovered()
    }

+5 −4
Original line number Diff line number Diff line
@@ -16,7 +16,6 @@

package com.android.wm.shell.flicker.pip

import android.platform.test.annotations.FlakyTest
import android.platform.test.annotations.Presubmit
import android.view.Surface
import androidx.test.filters.RequiresDevice
@@ -83,8 +82,10 @@ open class EnterPipTest(testSpec: FlickerTestParameter) : PipTransition(testSpec
        testSpec.assertWm { this.isAppWindowVisible(pipApp) }
    }

    /** Checks [pipApp] layer remains visible throughout the animation */
    @FlakyTest(bugId = 239807171)
    /**
     * Checks [pipApp] layer remains visible throughout the animation
     */
    @Presubmit
    @Test
    open fun pipAppLayerAlwaysVisible() {
        testSpec.assertLayers { this.isVisible(pipApp) }
@@ -104,7 +105,7 @@ open class EnterPipTest(testSpec: FlickerTestParameter) : PipTransition(testSpec
     * Checks that the pip app layer remains inside the display bounds throughout the whole
     * animation
     */
    @FlakyTest(bugId = 239807171)
    @Presubmit
    @Test
    open fun pipLayerRemainInsideVisibleBounds() {
        testSpec.assertLayersVisibleRegion(pipApp) { coversAtMost(displayBounds) }
+1 −2
Original line number Diff line number Diff line
@@ -18,7 +18,6 @@ package com.android.wm.shell.flicker.pip

import android.app.Activity
import android.platform.test.annotations.FlakyTest
import android.platform.test.annotations.Postsubmit
import android.platform.test.annotations.Presubmit
import android.view.Surface
import androidx.test.filters.RequiresDevice
@@ -225,7 +224,7 @@ class EnterPipToOtherOrientationTest(
    }

    /** {@inheritDoc}  */
    @Postsubmit
    @Presubmit
    @Test
    override fun visibleLayersShownMoreThanOneConsecutiveEntry() =
        super.visibleLayersShownMoreThanOneConsecutiveEntry()
+3 −3
Original line number Diff line number Diff line
@@ -16,7 +16,7 @@

package com.android.wm.shell.flicker.pip

import android.platform.test.annotations.FlakyTest
import android.platform.test.annotations.Presubmit
import android.view.Surface
import androidx.test.filters.RequiresDevice
import com.android.server.wm.flicker.FlickerParametersRunnerFactory
@@ -76,12 +76,12 @@ class ExitPipViaExpandButtonClickTest(
        }

    /** {@inheritDoc}  */
    @FlakyTest(bugId = 227313015)
    @Presubmit
    @Test
    override fun entireScreenCovered() = super.entireScreenCovered()

    /** {@inheritDoc}  */
    @FlakyTest(bugId = 197726610)
    @Presubmit
    @Test
    override fun pipLayerExpands() = super.pipLayerExpands()

+4 −2
Original line number Diff line number Diff line
@@ -75,7 +75,9 @@ class ExitPipViaIntentTest(testSpec: FlickerTestParameter) : ExitPipToAppTransit
        }

    /** {@inheritDoc}  */
    @FlakyTest @Test override fun entireScreenCovered() = super.entireScreenCovered()
    @Presubmit
    @Test
    override fun entireScreenCovered() = super.entireScreenCovered()

    /** {@inheritDoc} */
    @Presubmit
Loading