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

Commit 74d8d9c3 authored by Antony Sargent's avatar Antony Sargent
Browse files

Implement off state for SIM details page

When a subscription is disabled, we need to hide most things on the
details page. The UX mocks called for leaving in a few things such as
the data usage header and some carrier information, but in tests I've
done it looks like the telephony APIs don't have any of that available
when the subscription is disabled. So for now the only thing left on the
page in the disabled case is the on/off toggle at the very top.

Fixes: 129665956
Test: make RunSettingsRoboTests
Change-Id: I98f4590fca38d03b1e5ff2cf0d2b4876d2adf936
parent 172d03c7
Loading
Loading
Loading
Loading
+154 −146
Original line number Original line Diff line number Diff line
@@ -24,6 +24,12 @@
        android:layout="@layout/styled_switch_bar"
        android:layout="@layout/styled_switch_bar"
        settings:controller="com.android.settings.network.telephony.MobileNetworkSwitchController"/>
        settings:controller="com.android.settings.network.telephony.MobileNetworkSwitchController"/>


    <PreferenceCategory
        android:key="enabled_state_container"
        android:title="@string/summary_placeholder"
        settings:controller="com.android.settings.network.telephony.DisabledSubscriptionController"
        android:layout="@layout/preference_category_no_label">

        <com.android.settings.datausage.DataUsageSummaryPreference
        <com.android.settings.datausage.DataUsageSummaryPreference
            android:key="status_header"
            android:key="status_header"
            android:visibility="gone"
            android:visibility="gone"
@@ -182,4 +188,6 @@
            settings:controller="com.android.settings.network.telephony.CarrierPreferenceController">
            settings:controller="com.android.settings.network.telephony.CarrierPreferenceController">
        </Preference>
        </Preference>


    </PreferenceCategory>

</PreferenceScreen>
</PreferenceScreen>
+91 −0
Original line number Original line Diff line number Diff line
/*
 * Copyright (C) 2019 The Android Open Source Project
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License
 */

package com.android.settings.network.telephony;

import static androidx.lifecycle.Lifecycle.Event.ON_PAUSE;
import static androidx.lifecycle.Lifecycle.Event.ON_RESUME;

import android.content.Context;
import android.telephony.SubscriptionManager;

import androidx.lifecycle.Lifecycle;
import androidx.lifecycle.LifecycleObserver;
import androidx.lifecycle.OnLifecycleEvent;
import androidx.preference.PreferenceCategory;
import androidx.preference.PreferenceScreen;

import com.android.settings.core.BasePreferenceController;
import com.android.settings.network.SubscriptionsChangeListener;

public class DisabledSubscriptionController extends BasePreferenceController implements
        SubscriptionsChangeListener.SubscriptionsChangeListenerClient, LifecycleObserver {
    private PreferenceCategory mCategory;
    private int mSubId;
    private SubscriptionsChangeListener mChangeListener;
    private SubscriptionManager mSubscriptionManager;

    public DisabledSubscriptionController(Context context, String preferenceKey) {
        super(context, preferenceKey);
        mSubId = SubscriptionManager.INVALID_SUBSCRIPTION_ID;
        mSubscriptionManager = mContext.getSystemService(SubscriptionManager.class);
        mChangeListener = new SubscriptionsChangeListener(context, this);
    }

    public void init(Lifecycle lifecycle, int subId) {
        lifecycle.addObserver(this);
        mSubId = subId;
    }

    @OnLifecycleEvent(ON_RESUME)
    public void onResume() {
        mChangeListener.start();
        update();
    }

    @OnLifecycleEvent(ON_PAUSE)
    public void onPause() {
        mChangeListener.stop();
    }

    @Override
    public void displayPreference(PreferenceScreen screen) {
        super.displayPreference(screen);
        mCategory = screen.findPreference(getPreferenceKey());
        update();
    }

    private void update() {
        if (mCategory == null || mSubId ==  SubscriptionManager.INVALID_SUBSCRIPTION_ID) {
            return;
        }
        mCategory.setVisible(mSubscriptionManager.isSubscriptionEnabled(mSubId));
    }

    @Override
    public int getAvailabilityStatus() {
        return AVAILABLE_UNSEARCHABLE;
    }

    @Override
    public void onAirplaneModeChanged(boolean airplaneModeEnabled) {
    }

    @Override
    public void onSubscriptionsChanged() {
        update();
    }
}
+7 −6
Original line number Original line Diff line number Diff line
@@ -137,6 +137,7 @@ public class MobileNetworkSettings extends RestrictedDashboardFragment {
            use(CarrierSettingsVersionPreferenceController.class).init(mSubId);
            use(CarrierSettingsVersionPreferenceController.class).init(mSubId);
            use(BillingCyclePreferenceController.class).init(mSubId);
            use(BillingCyclePreferenceController.class).init(mSubId);
            use(MmsMessagePreferenceController.class).init(mSubId);
            use(MmsMessagePreferenceController.class).init(mSubId);
            use(DisabledSubscriptionController.class).init(getLifecycle(), mSubId);
        }
        }
        use(MobileDataPreferenceController.class).init(getFragmentManager(), mSubId);
        use(MobileDataPreferenceController.class).init(getFragmentManager(), mSubId);
        use(RoamingPreferenceController.class).init(getFragmentManager(), mSubId);
        use(RoamingPreferenceController.class).init(getFragmentManager(), mSubId);
+101 −0
Original line number Original line Diff line number Diff line
/*
 * Copyright (C) 2019 The Android Open Source Project
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License
 */

package com.android.settings.network.telephony;

import static com.google.common.truth.Truth.assertThat;

import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.spy;

import android.content.Context;
import android.telephony.SubscriptionManager;

import androidx.lifecycle.LifecycleOwner;
import androidx.preference.PreferenceCategory;
import androidx.preference.PreferenceScreen;

import com.android.settingslib.core.lifecycle.Lifecycle;

import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
import org.robolectric.RobolectricTestRunner;
import org.robolectric.RuntimeEnvironment;

@RunWith(RobolectricTestRunner.class)
public class DisabledSubscriptionControllerTest {

    private static final String KEY = "disabled_subscription_category";
    private static final int SUB_ID = 111;

    @Mock
    private SubscriptionManager mSubscriptionManager;
    @Mock
    private PreferenceScreen mScreen;

    private PreferenceCategory mCategory;
    private Context mContext;
    private Lifecycle mLifecycle;
    private DisabledSubscriptionController mController;

    @Before
    public void setUp() {
        MockitoAnnotations.initMocks(this);
        mContext = spy(RuntimeEnvironment.application);
        LifecycleOwner lifecycleOwner = () -> mLifecycle;
        mLifecycle = new Lifecycle(lifecycleOwner);
        doReturn(mSubscriptionManager).when(mContext).getSystemService(SubscriptionManager.class);
        mCategory = new PreferenceCategory(mContext);
        doReturn(mCategory).when(mScreen).findPreference(KEY);
        mController = new DisabledSubscriptionController(mContext, KEY);
        mController.init(mLifecycle, SUB_ID);
    }

    @Test
    public void displayPreference_subscriptionEnabled_categoryIsVisible() {
        doReturn(true).when(mSubscriptionManager).isSubscriptionEnabled(SUB_ID);
        mController.displayPreference(mScreen);
        assertThat(mCategory.isVisible()).isTrue();
    }

    @Test
    public void displayPreference_subscriptionDisabled_categoryIsNotVisible() {
        doReturn(false).when(mSubscriptionManager).isSubscriptionEnabled(SUB_ID);
        mController.displayPreference(mScreen);
        assertThat(mCategory.isVisible()).isFalse();
    }

    @Test
    public void onSubscriptionsChanged_subscriptionBecomesDisabled_categoryIsNotVisible() {
        doReturn(true).when(mSubscriptionManager).isSubscriptionEnabled(SUB_ID);
        mController.displayPreference(mScreen);
        doReturn(false).when(mSubscriptionManager).isSubscriptionEnabled(SUB_ID);
        mController.onSubscriptionsChanged();
        assertThat(mCategory.isVisible()).isFalse();
    }

    @Test
    public void onSubscriptionsChanged_subscriptionBecomesEnabled_categoryIsVisible() {
        doReturn(false).when(mSubscriptionManager).isSubscriptionEnabled(SUB_ID);
        mController.displayPreference(mScreen);
        doReturn(true).when(mSubscriptionManager).isSubscriptionEnabled(SUB_ID);
        mController.onSubscriptionsChanged();
        assertThat(mCategory.isVisible()).isTrue();
    }
}