1 files changed, 30 insertions, 4 deletions
diff --git a/robolectric/src/test/java/org/robolectric/shadows/ShadowSubscriptionManagerTest.java b/robolectric/src/test/java/org/robolectric/shadows/ShadowSubscriptionManagerTest.java
index 9ea0e9a12..4e7fb16ce 100644
--- a/robolectric/src/test/java/org/robolectric/shadows/ShadowSubscriptionManagerTest.java
+++ b/robolectric/src/test/java/org/robolectric/shadows/ShadowSubscriptionManagerTest.java
@@ -3,6 +3,8 @@ package org.robolectric.shadows;
 import static android.content.Context.TELEPHONY_SUBSCRIPTION_SERVICE;
 import static android.os.Build.VERSION_CODES.N;
 import static android.os.Build.VERSION_CODES.P;
+import static android.os.Build.VERSION_CODES.R;
+import static android.os.Build.VERSION_CODES.TIRAMISU;
 import static androidx.test.core.app.ApplicationProvider.getApplicationContext;
 import static com.google.common.truth.Truth.assertThat;
 import static org.junit.Assert.assertThrows;
@@ -31,6 +33,14 @@ public class ShadowSubscriptionManagerTest {
             getApplicationContext().getSystemService(TELEPHONY_SUBSCRIPTION_SERVICE);
   }
 
+  @Config(minSdk = R)
+  @Test
+  public void shouldGiveActiveDataSubscriptionId() {
+    int testId = 42;
+    ShadowSubscriptionManager.setActiveDataSubscriptionId(testId);
+    assertThat(SubscriptionManager.getActiveDataSubscriptionId()).isEqualTo(testId);
+  }
+
   @Test
   public void shouldGiveDefaultSubscriptionId() {
     int testId = 42;
@@ -161,24 +171,24 @@ public class ShadowSubscriptionManagerTest {
 
   @Test
   public void isNetworkRoaming_shouldReturnTrueIfSet() {
-    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /*isNetworkRoaming=*/ true);
+    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /* isNetworkRoaming= */ true);
     assertThat(shadowOf(subscriptionManager).isNetworkRoaming(123)).isTrue();
   }
 
   /** Multi act-asserts are discouraged but here we are testing the set+unset. */
   @Test
   public void isNetworkRoaming_shouldReturnFalseIfUnset() {
-    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /*isNetworkRoaming=*/ true);
+    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /* isNetworkRoaming= */ true);
     assertThat(shadowOf(subscriptionManager).isNetworkRoaming(123)).isTrue();
 
-    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /*isNetworkRoaming=*/ false);
+    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /* isNetworkRoaming= */ false);
     assertThat(shadowOf(subscriptionManager).isNetworkRoaming(123)).isFalse();
   }
 
   /** Multi act-asserts are discouraged but here we are testing the set+clear. */
   @Test
   public void isNetworkRoaming_shouldReturnFalseOnClear() {
-    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /*isNetworkRoaming=*/ true);
+    shadowOf(subscriptionManager).setNetworkRoamingStatus(123, /* isNetworkRoaming= */ true);
     assertThat(shadowOf(subscriptionManager).isNetworkRoaming(123)).isTrue();
 
     shadowOf(subscriptionManager).clearNetworkRoamingStatus();
@@ -305,6 +315,22 @@ public class ShadowSubscriptionManagerTest {
         .isEqualTo(123);
   }
 
+  @Test
+  @Config(minSdk = TIRAMISU)
+  public void getPhoneNumber_phoneNumberNotSet_returnsEmptyString() {
+    assertThat(subscriptionManager.getPhoneNumber(SubscriptionManager.DEFAULT_SUBSCRIPTION_ID))
+        .isEqualTo("");
+  }
+
+  @Test
+  @Config(minSdk = TIRAMISU)
+  public void getPhoneNumber_setPhoneNumber_returnsPhoneNumber() {
+    shadowOf(subscriptionManager)
+        .setPhoneNumber(SubscriptionManager.DEFAULT_SUBSCRIPTION_ID, "123");
+    assertThat(subscriptionManager.getPhoneNumber(SubscriptionManager.DEFAULT_SUBSCRIPTION_ID))
+        .isEqualTo("123");
+  }
+
   private static class DummySubscriptionsChangedListener
       extends SubscriptionManager.OnSubscriptionsChangedListener {
     private int subscriptionChangedCount;