Merge "Add check KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL" into qt-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
1ffae41f4e
@@ -65,8 +65,11 @@ public class ApnPreferenceController extends TelephonyBasePreferenceController i
|
|||||||
final boolean isGsmApn = MobileNetworkUtils.isGsmOptions(mContext, subId)
|
final boolean isGsmApn = MobileNetworkUtils.isGsmOptions(mContext, subId)
|
||||||
&& carrierConfig != null
|
&& carrierConfig != null
|
||||||
&& carrierConfig.getBoolean(CarrierConfigManager.KEY_APN_EXPAND_BOOL);
|
&& carrierConfig.getBoolean(CarrierConfigManager.KEY_APN_EXPAND_BOOL);
|
||||||
|
final boolean hideCarrierNetwork = carrierConfig == null
|
||||||
|
|| carrierConfig.getBoolean(
|
||||||
|
CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL);
|
||||||
|
|
||||||
return isCdmaApn || isGsmApn
|
return !hideCarrierNetwork && (isCdmaApn || isGsmApn)
|
||||||
? AVAILABLE
|
? AVAILABLE
|
||||||
: CONDITIONALLY_UNAVAILABLE;
|
: CONDITIONALLY_UNAVAILABLE;
|
||||||
}
|
}
|
||||||
|
@@ -355,6 +355,8 @@ public class MobileNetworkUtils {
|
|||||||
|| carrierConfig == null
|
|| carrierConfig == null
|
||||||
|| !carrierConfig.getBoolean(
|
|| !carrierConfig.getBoolean(
|
||||||
CarrierConfigManager.KEY_OPERATOR_SELECTION_EXPAND_BOOL)
|
CarrierConfigManager.KEY_OPERATOR_SELECTION_EXPAND_BOOL)
|
||||||
|
|| carrierConfig.getBoolean(
|
||||||
|
CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL)
|
||||||
|| (carrierConfig.getBoolean(CarrierConfigManager.KEY_CSP_ENABLED_BOOL)
|
|| (carrierConfig.getBoolean(CarrierConfigManager.KEY_CSP_ENABLED_BOOL)
|
||||||
&& !telephonyManager.isManualNetworkSelectionAllowed())) {
|
&& !telephonyManager.isManualNetworkSelectionAllowed())) {
|
||||||
return false;
|
return false;
|
||||||
|
@@ -52,7 +52,6 @@ public class AutoSelectPreferenceController extends TelephonyTogglePreferenceCon
|
|||||||
private static final long MINIMUM_DIALOG_TIME_MILLIS = TimeUnit.SECONDS.toMillis(1);
|
private static final long MINIMUM_DIALOG_TIME_MILLIS = TimeUnit.SECONDS.toMillis(1);
|
||||||
|
|
||||||
private final Handler mUiHandler;
|
private final Handler mUiHandler;
|
||||||
private int mSubId;
|
|
||||||
private TelephonyManager mTelephonyManager;
|
private TelephonyManager mTelephonyManager;
|
||||||
private boolean mOnlyAutoSelectInHome;
|
private boolean mOnlyAutoSelectInHome;
|
||||||
private List<OnNetworkSelectModeListener> mListeners;
|
private List<OnNetworkSelectModeListener> mListeners;
|
||||||
|
@@ -42,7 +42,6 @@ public class OpenNetworkSelectPagePreferenceController extends
|
|||||||
TelephonyBasePreferenceController implements
|
TelephonyBasePreferenceController implements
|
||||||
AutoSelectPreferenceController.OnNetworkSelectModeListener {
|
AutoSelectPreferenceController.OnNetworkSelectModeListener {
|
||||||
|
|
||||||
private int mSubId;
|
|
||||||
private TelephonyManager mTelephonyManager;
|
private TelephonyManager mTelephonyManager;
|
||||||
private Preference mPreference;
|
private Preference mPreference;
|
||||||
|
|
||||||
|
@@ -123,6 +123,18 @@ public class ApnPreferenceControllerTest {
|
|||||||
assertThat(mController.getAvailabilityStatus()).isEqualTo(CONDITIONALLY_UNAVAILABLE);
|
assertThat(mController.getAvailabilityStatus()).isEqualTo(CONDITIONALLY_UNAVAILABLE);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void getAvailabilityStatus_hideCarrierNetworkSettings_returnUnavailable() {
|
||||||
|
doReturn(PhoneConstants.PHONE_TYPE_GSM).when(mTelephonyManager).getPhoneType();
|
||||||
|
final PersistableBundle bundle = new PersistableBundle();
|
||||||
|
bundle.putBoolean(CarrierConfigManager.KEY_APN_EXPAND_BOOL, true);
|
||||||
|
bundle.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL, true);
|
||||||
|
doReturn(bundle).when(mCarrierConfigManager).getConfigForSubId(SUB_ID);
|
||||||
|
|
||||||
|
assertThat(mController.getAvailabilityStatus()).isEqualTo(CONDITIONALLY_UNAVAILABLE);
|
||||||
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void handPreferenceTreeClick_fireIntent() {
|
public void handPreferenceTreeClick_fireIntent() {
|
||||||
ArgumentCaptor<Intent> captor = ArgumentCaptor.forClass(Intent.class);
|
ArgumentCaptor<Intent> captor = ArgumentCaptor.forClass(Intent.class);
|
||||||
|
@@ -198,4 +198,25 @@ public class MobileNetworkUtilsTest {
|
|||||||
assertThat(MobileNetworkUtils.getSearchableSubscriptionId(mContext))
|
assertThat(MobileNetworkUtils.getSearchableSubscriptionId(mContext))
|
||||||
.isEqualTo(SubscriptionManager.INVALID_SUBSCRIPTION_ID);
|
.isEqualTo(SubscriptionManager.INVALID_SUBSCRIPTION_ID);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void shouldDisplayNetworkSelectOptions_HideCarrierNetwork_returnFalse() {
|
||||||
|
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL,
|
||||||
|
true);
|
||||||
|
|
||||||
|
assertThat(MobileNetworkUtils.shouldDisplayNetworkSelectOptions(mContext, SUB_ID_1))
|
||||||
|
.isFalse();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void shouldDisplayNetworkSelectOptions_allCheckPass_returnTrue() {
|
||||||
|
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL,
|
||||||
|
false);
|
||||||
|
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_OPERATOR_SELECTION_EXPAND_BOOL, true);
|
||||||
|
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_CSP_ENABLED_BOOL, false);
|
||||||
|
when(mTelephonyManager.getPhoneType()).thenReturn(PhoneConstants.PHONE_TYPE_GSM);
|
||||||
|
|
||||||
|
assertThat(MobileNetworkUtils.shouldDisplayNetworkSelectOptions(mContext, SUB_ID_1))
|
||||||
|
.isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user