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

Commit 51a2da4b authored by Ling Ma's avatar Ling Ma Committed by Android (Google) Code Review
Browse files

Merge "Fix disableGroup" into main

parents 14407ecb 2c45cdc9
Loading
Loading
Loading
Loading
+2 −1
Original line number Diff line number Diff line
@@ -4278,9 +4278,10 @@ public class SubscriptionManagerService extends ISub.Stub {
     */
    @VisibleForTesting
    public void updateGroupDisabled() {
        List<SubscriptionInfoInternal> activeSubscriptions = mSubscriptionDatabaseManager
        List<SubscriptionInfo> activeSubscriptions = mSubscriptionDatabaseManager
                .getAllSubscriptions().stream()
                .filter(SubscriptionInfoInternal::isActive)
                .map(SubscriptionInfoInternal::toSubscriptionInfo)
                .collect(Collectors.toList());
        for (SubscriptionInfo oppSubInfo : getOpportunisticSubscriptions(
                mContext.getOpPackageName(), mContext.getFeatureId())) {
+13 −1
Original line number Diff line number Diff line
@@ -2957,7 +2957,6 @@ public class SubscriptionManagerServiceTest extends TelephonyTest {
        assertThat(subInfo.getNativeAccessRules()).isEqualTo(new byte[]{});
    }


    @Test
    public void testGetActiveSubscriptionInfoListNoSecurityException() {
        // Grant MODIFY_PHONE_STATE permission for insertion.
@@ -2983,4 +2982,17 @@ public class SubscriptionManagerServiceTest extends TelephonyTest {
        assertThat(mSubscriptionManagerServiceUT.getActiveSubscriptionInfoList(
                CALLING_PACKAGE, CALLING_FEATURE, true)).isEmpty();
    }

    @Test
    public void testUpdateGroupDisabled() {
        insertSubscription(FAKE_SUBSCRIPTION_INFO1);
        insertSubscription(new SubscriptionInfoInternal
                .Builder(FAKE_SUBSCRIPTION_INFO2).setGroupUuid(FAKE_UUID1).build());

        mSubscriptionManagerServiceUT.updateGroupDisabled();

        SubscriptionInfoInternal subInfo = mSubscriptionManagerServiceUT
                .getSubscriptionInfoInternal(2);
        assertThat(subInfo.isGroupDisabled()).isFalse();
    }
}