summaryrefslogtreecommitdiff
path: root/java
diff options
context:
space:
mode:
author Colin Cross <ccross@android.com> 2023-08-09 18:34:17 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2023-08-09 18:34:17 +0000
commita0c4477fdbbae9857ead667a5227c3d93d9d3167 (patch)
treead0882d883f532b41439348bf322467770ceea7c /java
parente30dfb628ae89f4adb5e88e8bd6027175845eefe (diff)
parentb12fe0a69e3e25138de99ef9ab33f426282d29f1 (diff)
Fix kotlin nullable errors in IntentResolver am: b12fe0a69e
Original change: https://android-review.googlesource.com/c/platform/packages/modules/IntentResolver/+/2696253 Change-Id: I94cf34a704ec7726068a1107cabdd9968364c044 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Diffstat (limited to 'java')
-rw-r--r--java/tests/src/com/android/intentresolver/chooser/ImmutableTargetInfoTest.kt34
-rw-r--r--java/tests/src/com/android/intentresolver/chooser/TargetInfoTest.kt10
2 files changed, 22 insertions, 22 deletions
diff --git a/java/tests/src/com/android/intentresolver/chooser/ImmutableTargetInfoTest.kt b/java/tests/src/com/android/intentresolver/chooser/ImmutableTargetInfoTest.kt
index e9c755d3..dff1e5ae 100644
--- a/java/tests/src/com/android/intentresolver/chooser/ImmutableTargetInfoTest.kt
+++ b/java/tests/src/com/android/intentresolver/chooser/ImmutableTargetInfoTest.kt
@@ -218,8 +218,8 @@ class ImmutableTargetInfoTest {
.build()
val info = originalInfo.tryToCloneWithAppliedRefinement(refinementIntent)
- assertThat(info.baseIntentToSend.getBooleanExtra("ORIGINAL", false)).isTrue()
- assertThat(info.baseIntentToSend.getBooleanExtra("REFINEMENT", false)).isTrue()
+ assertThat(info?.baseIntentToSend?.getBooleanExtra("ORIGINAL", false)).isTrue()
+ assertThat(info?.baseIntentToSend?.getBooleanExtra("REFINEMENT", false)).isTrue()
}
@Test
@@ -241,9 +241,9 @@ class ImmutableTargetInfoTest {
val info = infoWithReferrerFillIn.tryToCloneWithAppliedRefinement(refinementIntent)
- assertThat(info.baseIntentToSend.getPackage()).isEqualTo("original") // Set all along.
- assertThat(info.baseIntentToSend.action).isEqualTo("REFINE_ME") // Refinement wins.
- assertThat(info.baseIntentToSend.type).isEqualTo("test/referrer") // Left for referrer.
+ assertThat(info?.baseIntentToSend?.getPackage()).isEqualTo("original") // Set all along.
+ assertThat(info?.baseIntentToSend?.action).isEqualTo("REFINE_ME") // Refinement wins.
+ assertThat(info?.baseIntentToSend?.type).isEqualTo("test/referrer") // Left for referrer.
}
@Test
@@ -262,18 +262,18 @@ class ImmutableTargetInfoTest {
.build()
val refined1 = originalInfo.tryToCloneWithAppliedRefinement(refinementIntent1)
- val refined2 = refined1.tryToCloneWithAppliedRefinement(refinementIntent2) // Cloned clone.
+ val refined2 = refined1?.tryToCloneWithAppliedRefinement(refinementIntent2) // Cloned clone.
// Both clones get the same values filled in from the referrer intent.
- assertThat(refined1.baseIntentToSend.getStringExtra("TEST")).isEqualTo("REFERRER")
- assertThat(refined2.baseIntentToSend.getStringExtra("TEST")).isEqualTo("REFERRER")
+ assertThat(refined1?.baseIntentToSend?.getStringExtra("TEST")).isEqualTo("REFERRER")
+ assertThat(refined2?.baseIntentToSend?.getStringExtra("TEST")).isEqualTo("REFERRER")
// Each clone has the respective value that was set in their own refinement request.
- assertThat(refined1.baseIntentToSend.getStringExtra("TEST1")).isEqualTo("1")
- assertThat(refined2.baseIntentToSend.getStringExtra("TEST2")).isEqualTo("2")
+ assertThat(refined1?.baseIntentToSend?.getStringExtra("TEST1")).isEqualTo("1")
+ assertThat(refined2?.baseIntentToSend?.getStringExtra("TEST2")).isEqualTo("2")
// The clones don't have the data from each other's refinements, even though the intent
// field is empty (thus able to be populated by filling-in).
- assertThat(refined1.baseIntentToSend.getStringExtra("TEST2")).isNull()
- assertThat(refined2.baseIntentToSend.getStringExtra("TEST1")).isNull()
+ assertThat(refined1?.baseIntentToSend?.getStringExtra("TEST2")).isNull()
+ assertThat(refined2?.baseIntentToSend?.getStringExtra("TEST1")).isNull()
}
@Test
@@ -297,15 +297,15 @@ class ImmutableTargetInfoTest {
refinement.putExtra("refinement", true)
val refinedResult = originalInfo.tryToCloneWithAppliedRefinement(refinement)
- assertThat(refinedResult.baseIntentToSend.getBooleanExtra("refinement", false)).isTrue()
- assertThat(refinedResult.baseIntentToSend.getBooleanExtra("targetAlternate", false))
+ assertThat(refinedResult?.baseIntentToSend?.getBooleanExtra("refinement", false)).isTrue()
+ assertThat(refinedResult?.baseIntentToSend?.getBooleanExtra("targetAlternate", false))
.isTrue()
// None of the other source intents got merged in (not even the later one that matched):
- assertThat(refinedResult.baseIntentToSend.getBooleanExtra("originalIntent", false))
+ assertThat(refinedResult?.baseIntentToSend?.getBooleanExtra("originalIntent", false))
.isFalse()
- assertThat(refinedResult.baseIntentToSend.getBooleanExtra("mismatchedAlternate", false))
+ assertThat(refinedResult?.baseIntentToSend?.getBooleanExtra("mismatchedAlternate", false))
.isFalse()
- assertThat(refinedResult.baseIntentToSend.getBooleanExtra("extraMatch", false)).isFalse()
+ assertThat(refinedResult?.baseIntentToSend?.getBooleanExtra("extraMatch", false)).isFalse()
}
@Test
diff --git a/java/tests/src/com/android/intentresolver/chooser/TargetInfoTest.kt b/java/tests/src/com/android/intentresolver/chooser/TargetInfoTest.kt
index dddbcccb..3d832cc9 100644
--- a/java/tests/src/com/android/intentresolver/chooser/TargetInfoTest.kt
+++ b/java/tests/src/com/android/intentresolver/chooser/TargetInfoTest.kt
@@ -231,15 +231,15 @@ class TargetInfoTest {
val refinedResult = originalInfo.tryToCloneWithAppliedRefinement(refinement)
// Note `DisplayResolveInfo` targets merge refinements directly into their `resolvedIntent`.
- assertThat(refinedResult.resolvedIntent.getBooleanExtra("refinement", false)).isTrue()
- assertThat(refinedResult.resolvedIntent.getBooleanExtra("targetAlternate", false))
+ assertThat(refinedResult?.resolvedIntent?.getBooleanExtra("refinement", false)).isTrue()
+ assertThat(refinedResult?.resolvedIntent?.getBooleanExtra("targetAlternate", false))
.isTrue()
// None of the other source intents got merged in (not even the later one that matched):
- assertThat(refinedResult.resolvedIntent.getBooleanExtra("originalIntent", false))
+ assertThat(refinedResult?.resolvedIntent?.getBooleanExtra("originalIntent", false))
.isFalse()
- assertThat(refinedResult.resolvedIntent.getBooleanExtra("mismatchedAlternate", false))
+ assertThat(refinedResult?.resolvedIntent?.getBooleanExtra("mismatchedAlternate", false))
.isFalse()
- assertThat(refinedResult.resolvedIntent.getBooleanExtra("extraMatch", false)).isFalse()
+ assertThat(refinedResult?.resolvedIntent?.getBooleanExtra("extraMatch", false)).isFalse()
}
@Test