diff options
author | 2024-06-13 17:21:42 +0000 | |
---|---|---|
committer | 2024-06-13 17:21:42 +0000 | |
commit | f43c33083038780ebe47ddb2d162ad8635cede4a (patch) | |
tree | 099177e80a17e71ae666f05dce71d8b5a4d88330 | |
parent | b86d3349618d33657ca6b38e72ff08dc227676ed (diff) | |
parent | 68206f356b11e94183a7e95f36214730b4b90df0 (diff) |
Merge "Add comments to interaction handler tests" into main
2 files changed, 28 insertions, 24 deletions
diff --git a/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/smartspace/SmartspaceInteractionHandlerTest.kt b/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/smartspace/SmartspaceInteractionHandlerTest.kt index 9e24ef840a5b..0cd3fb28e299 100644 --- a/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/smartspace/SmartspaceInteractionHandlerTest.kt +++ b/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/smartspace/SmartspaceInteractionHandlerTest.kt @@ -64,14 +64,15 @@ class SmartspaceInteractionHandlerTest : SysuiTestCase() { underTest.onInteraction(view, testIntent, testResponse) + // Verify that we pass in a non-null animation controller verify(activityStarter) .startPendingIntentWithoutDismissing( - eq(testIntent), - eq(false), - isNull(), - notNull(), - refEq(fillInIntent), - refEq(activityOptions.toBundle()), + /* intent = */ eq(testIntent), + /* dismissShade = */ eq(false), + /* intentSentUiThreadCallback = */ isNull(), + /* animationController = */ notNull(), + /* fillInIntent = */ refEq(fillInIntent), + /* extraOptions = */ refEq(activityOptions.toBundle()), ) } @@ -84,14 +85,15 @@ class SmartspaceInteractionHandlerTest : SysuiTestCase() { underTest.onInteraction(view, testIntent, testResponse) + // Verify null is used as the animation controller verify(activityStarter) .startPendingIntentWithoutDismissing( - eq(testIntent), - eq(false), - isNull(), - isNull(), - refEq(fillInIntent), - refEq(activityOptions.toBundle()), + /* intent = */ eq(testIntent), + /* dismissShade = */ eq(false), + /* intentSentUiThreadCallback = */ isNull(), + /* animationController = */ isNull(), + /* fillInIntent = */ refEq(fillInIntent), + /* extraOptions = */ refEq(activityOptions.toBundle()), ) } } diff --git a/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/widgets/WidgetInteractionHandlerTest.kt b/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/widgets/WidgetInteractionHandlerTest.kt index bf6a2b084a9b..7b7d03b2024a 100644 --- a/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/widgets/WidgetInteractionHandlerTest.kt +++ b/packages/SystemUI/multivalentTests/src/com/android/systemui/communal/widgets/WidgetInteractionHandlerTest.kt @@ -63,14 +63,15 @@ class WidgetInteractionHandlerTest : SysuiTestCase() { underTest.onInteraction(view, testIntent, testResponse) + // Verify that we pass in a non-null animation controller verify(activityStarter) .startPendingIntentMaybeDismissingKeyguard( - eq(testIntent), - eq(false), - isNull(), - notNull(), - refEq(fillInIntent), - refEq(activityOptions.toBundle()), + /* intent = */ eq(testIntent), + /* dismissShade = */ eq(false), + /* intentSentUiThreadCallback = */ isNull(), + /* animationController = */ notNull(), + /* fillInIntent = */ refEq(fillInIntent), + /* extraOptions = */ refEq(activityOptions.toBundle()), ) } @@ -83,14 +84,15 @@ class WidgetInteractionHandlerTest : SysuiTestCase() { underTest.onInteraction(view, testIntent, testResponse) + // Verify null is used as the animation controller verify(activityStarter) .startPendingIntentMaybeDismissingKeyguard( - eq(testIntent), - eq(false), - isNull(), - isNull(), - refEq(fillInIntent), - refEq(activityOptions.toBundle()), + /* intent = */ eq(testIntent), + /* dismissShade = */ eq(false), + /* intentSentUiThreadCallback = */ isNull(), + /* animationController = */ isNull(), + /* fillInIntent = */ refEq(fillInIntent), + /* extraOptions = */ refEq(activityOptions.toBundle()), ) } } |