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

Commit 69ad55f3 authored by Selim Cinek's avatar Selim Cinek
Browse files

Fixed the tests for the physical notification movement 

A conflicting change came in with a test that is now failing.
Disabling those tests is the right thing as we already cover
that physical version of that test in a new method.

Bug: 393581344
Flag: com.android.systemui.physical_notification_movement
Change-Id: I09a9fdb77baa877055c0c0561133cac529009825
parent e2c294fd

packages/SystemUI/multivalentTests/src/com/android/systemui/statusbar/notification/stack/StackStateAnimatorTest.kt

+2 −2
Original line number Diff line number Diff line
@@ -219,7 +219,6 @@ class StackStateAnimatorTest : SysuiTestCase() { 
            )

    }



    @DisableFlags(Flags.FLAG_PHYSICAL_NOTIFICATION_MOVEMENT)

    @Test

    @EnableFlags(NotificationsHunSharedAnimationValues.FLAG_NAME)

    fun startAnimationForEvents_headsUpFromBottom_startsHeadsUpAppearAnim_flagOn() {
@@ -246,7 +245,7 @@ class StackStateAnimatorTest : SysuiTestCase() { 
    }



    @Test

    @DisableFlags(NotificationsHunSharedAnimationValues.FLAG_NAME)

    @DisableFlags(NotificationsHunSharedAnimationValues.FLAG_NAME, Flags.FLAG_PHYSICAL_NOTIFICATION_MOVEMENT)

    fun startAnimationForEvents_startsHeadsUpDisappearAnim_flagOff() {

        val disappearDuration = ANIMATION_DURATION_HEADS_UP_DISAPPEAR.toLong()

        val event = AnimationEvent(view, AnimationEvent.ANIMATION_TYPE_HEADS_UP_DISAPPEAR)
@@ -277,6 +276,7 @@ class StackStateAnimatorTest : SysuiTestCase() { 


    @Test

    @EnableFlags(NotificationsHunSharedAnimationValues.FLAG_NAME)

    @DisableFlags(Flags.FLAG_PHYSICAL_NOTIFICATION_MOVEMENT)

    fun startAnimationForEvents_startsHeadsUpDisappearAnim_flagOn() {

        val disappearDuration = ANIMATION_DURATION_HEADS_UP_DISAPPEAR.toLong()

        val event = AnimationEvent(view, AnimationEvent.ANIMATION_TYPE_HEADS_UP_DISAPPEAR)