diff --git a/src/com/android/settings/network/SubscriptionsPreferenceController.java b/src/com/android/settings/network/SubscriptionsPreferenceController.java index 82df4aa25c8..9fe6566d61e 100644 --- a/src/com/android/settings/network/SubscriptionsPreferenceController.java +++ b/src/com/android/settings/network/SubscriptionsPreferenceController.java @@ -170,7 +170,7 @@ public class SubscriptionsPreferenceController extends AbstractPreferenceControl // Avoid from showing subscription(SIM)s which has been marked as hidden // For example, only one subscription will be shown when there're multiple // subscriptions with same group UUID. - if (!isSubscriptionCanBeDisplayed(mContext, subId)) { + if (!canSubscriptionBeDisplayed(mContext, subId)) { continue; } activeSubIds.add(subId); @@ -308,7 +308,7 @@ public class SubscriptionsPreferenceController extends AbstractPreferenceControl // For example, only one subscription will be shown when there're multiple // subscriptions with same group UUID. .filter(subInfo -> - isSubscriptionCanBeDisplayed(mContext, subInfo.getSubscriptionId())) + canSubscriptionBeDisplayed(mContext, subInfo.getSubscriptionId())) .count() >= 2; } @@ -349,7 +349,7 @@ public class SubscriptionsPreferenceController extends AbstractPreferenceControl } @VisibleForTesting - boolean isSubscriptionCanBeDisplayed(Context context, int subId) { + boolean canSubscriptionBeDisplayed(Context context, int subId) { return (SubscriptionUtil.getAvailableSubscription(context, ProxySubscriptionManager.getInstance(context), subId) != null); } diff --git a/tests/robotests/src/com/android/settings/network/SubscriptionsPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/network/SubscriptionsPreferenceControllerTest.java index ea2212c31f6..57b2e2fd6cb 100644 --- a/tests/robotests/src/com/android/settings/network/SubscriptionsPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/network/SubscriptionsPreferenceControllerTest.java @@ -122,7 +122,7 @@ public class SubscriptionsPreferenceControllerTest { mController = spy( new SubscriptionsPreferenceController(mContext, mLifecycle, mUpdateListener, KEY, 5)); - doReturn(true).when(mController).isSubscriptionCanBeDisplayed(any(), anyInt()); + doReturn(true).when(mController).canSubscriptionBeDisplayed(any(), anyInt()); doReturn(mSignalStrengthIcon).when(mController).getIcon(anyInt(), anyInt(), anyBoolean()); } @@ -458,7 +458,7 @@ public class SubscriptionsPreferenceControllerTest { @Test public void displayPreference_subscriptionsWithSameGroupUUID_onlyOneWillBeSeen() { - doReturn(false).when(mController).isSubscriptionCanBeDisplayed(any(), eq(3)); + doReturn(false).when(mController).canSubscriptionBeDisplayed(any(), eq(3)); final List subs = setupMockSubscriptions(3); SubscriptionUtil.setActiveSubscriptionsForTesting(subs.subList(0, 3));