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

Commit 057ecfc7 authored by Jordan Demeulenaere's avatar Jordan Demeulenaere Committed by Android (Google) Code Review
Browse files

Revert "Don't implicitly add test tags to all elements and scenes (1/2)" 

This reverts commit 92efbbfc.

Reason for revert: b/396518748

Change-Id: Ia3cb6d777304cf2c76a900e3faf58c4d86d78881
parent 92efbbfc

packages/SystemUI/compose/features/src/com/android/systemui/bouncer/ui/composable/BouncerScene.kt

+0 −3
Original line number Diff line number Diff line
@@ -24,7 +24,6 @@ import androidx.compose.material3.MaterialTheme 
import androidx.compose.runtime.Composable

import androidx.compose.runtime.DisposableEffect

import androidx.compose.ui.Modifier

import androidx.compose.ui.platform.testTag

import com.android.compose.animation.scene.ContentScope

import com.android.compose.animation.scene.ElementKey

import com.android.compose.animation.scene.UserAction
@@ -103,8 +102,6 @@ private fun ContentScope.BouncerScene( 
            viewModel,

            dialogFactory,

            Modifier.element(Bouncer.Elements.Content)

                // TODO(b/393516240): Use the same sysuiResTag() as views instead.

                .testTag(Bouncer.Elements.Content.testTag)

                .overscroll(verticalOverscrollEffect)

                .sysuiResTag(Bouncer.TestTags.Root)

                .fillMaxSize(),

packages/SystemUI/compose/features/src/com/android/systemui/keyguard/ui/composable/LockscreenScene.kt

+1 −6
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ package com.android.systemui.keyguard.ui.composable 
import androidx.compose.foundation.layout.fillMaxSize

import androidx.compose.runtime.Composable

import androidx.compose.ui.Modifier

import androidx.compose.ui.platform.testTag

import com.android.compose.animation.scene.ContentScope

import com.android.compose.animation.scene.UserAction

import com.android.compose.animation.scene.UserActionResult
@@ -56,11 +55,7 @@ constructor( 


    @Composable

    override fun ContentScope.Content(modifier: Modifier) {

        LockscreenScene(

            lockscreenContent = lockscreenContent,

            // TODO(b/393516240): Use the same sysuiResTag() as views instead.

            modifier = modifier.testTag(key.rootElementKey.testTag),

        )

        LockscreenScene(lockscreenContent = lockscreenContent, modifier = modifier)

    }

}

packages/SystemUI/compose/scene/src/com/android/compose/animation/scene/Element.kt

+1 −1
Original line number Diff line number Diff line
@@ -169,7 +169,7 @@ internal fun Modifier.element( 
            Modifier.maybeElevateInContent(layoutImpl, content, key, currentTransitionStates)

        }

        .then(ElementModifier(layoutImpl, currentTransitionStates, content, key))

        .thenIf(layoutImpl.implicitTestTags) { Modifier.testTag(key.testTag) }

        .testTag(key.testTag)

}



/**

packages/SystemUI/compose/scene/src/com/android/compose/animation/scene/SceneTransitionLayout.kt

+4 −7
Original line number Diff line number Diff line
@@ -65,8 +65,6 @@ fun SceneTransitionLayout( 
    swipeSourceDetector: SwipeSourceDetector = DefaultEdgeDetector,

    swipeDetector: SwipeDetector = DefaultSwipeDetector,

    @FloatRange(from = 0.0, to = 0.5) transitionInterceptionThreshold: Float = 0.05f,

    // TODO(b/240432457) Remove this once test utils can access the internal STLForTesting().

    implicitTestTags: Boolean = false,

    builder: SceneTransitionLayoutScope<ContentScope>.() -> Unit,

) {

    SceneTransitionLayoutForTesting(
@@ -75,7 +73,6 @@ fun SceneTransitionLayout( 
        swipeSourceDetector,

        swipeDetector,

        transitionInterceptionThreshold,

        implicitTestTags = implicitTestTags,

        onLayoutImpl = null,

        builder = builder,

    )
@@ -728,8 +725,10 @@ class FixedDistance(private val distance: Dp) : UserActionDistance { 
}



/**

 * An internal version of [SceneTransitionLayout] to be used for tests, that provides access to the

 * internal [SceneTransitionLayoutImpl] and implicitly tags all scenes and elements.

 * An internal version of [SceneTransitionLayout] to be used for tests.

 *

 * Important: You should use this only in tests and if you need to access the underlying

 * [SceneTransitionLayoutImpl]. In other cases, you should use [SceneTransitionLayout].

 */

@Composable

internal fun SceneTransitionLayoutForTesting(
@@ -742,7 +741,6 @@ internal fun SceneTransitionLayoutForTesting( 
    sharedElementMap: MutableMap<ElementKey, Element> = remember { mutableMapOf() },

    ancestors: List<Ancestor> = remember { emptyList() },

    lookaheadScope: LookaheadScope? = null,

    implicitTestTags: Boolean = true,

    builder: SceneTransitionLayoutScope<InternalContentScope>.() -> Unit,

) {

    val density = LocalDensity.current
@@ -767,7 +765,6 @@ internal fun SceneTransitionLayoutForTesting( 
                directionChangeSlop = directionChangeSlop,

                defaultEffectFactory = defaultEffectFactory,

                decayAnimationSpec = decayAnimationSpec,

                implicitTestTags = implicitTestTags,

            )

            .also { onLayoutImpl?.invoke(it) }

    }

packages/SystemUI/compose/scene/src/com/android/compose/animation/scene/SceneTransitionLayoutImpl.kt

+0 −3
Original line number Diff line number Diff line
@@ -122,9 +122,6 @@ internal class SceneTransitionLayoutImpl( 
     * This is used to enable transformations and shared elements across NestedSTLs.

     */

    internal val ancestors: List<Ancestor> = emptyList(),



    /** Whether elements and scene should be tagged using `Modifier.testTag`. */

    internal val implicitTestTags: Boolean = false,

    lookaheadScope: LookaheadScope? = null,

    defaultEffectFactory: OverscrollFactory,

) {