diff --git a/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryController.java b/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryController.java index 614ff6a695f..b404801898f 100644 --- a/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryController.java +++ b/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryController.java @@ -99,12 +99,19 @@ public class SatelliteSettingsPreferenceCategoryController extends if (!com.android.internal.telephony.flags.Flags.carrierEnabledSatelliteFlag()) { return UNSUPPORTED_ON_DEVICE; } + final PersistableBundle carrierConfig = mCarrierConfigCache.getConfigForSubId(subId); - if (!mIsSatelliteSupported.get()) { + boolean isSatelliteConnectedTypeIsAuto = + CARRIER_ROAMING_NTN_CONNECT_AUTOMATIC == carrierConfig.getInt( + KEY_CARRIER_ROAMING_NTN_CONNECT_TYPE_INT, + CARRIER_ROAMING_NTN_CONNECT_AUTOMATIC); + + // SatelliteManager#requestIsSupported is only supported for manual connection type, so + // if type is auto, this check shall be skipped. + if (!isSatelliteConnectedTypeIsAuto && !mIsSatelliteSupported.get()) { return UNSUPPORTED_ON_DEVICE; } - final PersistableBundle carrierConfig = mCarrierConfigCache.getConfigForSubId(subId); boolean isSatelliteSosSupported = false; if (Flags.satelliteOemSettingsUxMigration()) { isSatelliteSosSupported = carrierConfig.getBoolean(KEY_SATELLITE_ESOS_SUPPORTED_BOOL); @@ -118,8 +125,7 @@ public class SatelliteSettingsPreferenceCategoryController extends return AVAILABLE_UNSEARCHABLE; } - if (CARRIER_ROAMING_NTN_CONNECT_AUTOMATIC == carrierConfig.getInt( - KEY_CARRIER_ROAMING_NTN_CONNECT_TYPE_INT, CARRIER_ROAMING_NTN_CONNECT_AUTOMATIC)) { + if (isSatelliteConnectedTypeIsAuto) { return AVAILABLE_UNSEARCHABLE; } else { return mCarrierRoamingNtnModeCallback.isSatelliteSmsAvailable() diff --git a/tests/unit/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryControllerTest.java b/tests/unit/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryControllerTest.java index ef44b928afe..563dddf9875 100644 --- a/tests/unit/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryControllerTest.java +++ b/tests/unit/src/com/android/settings/network/telephony/satellite/SatelliteSettingsPreferenceCategoryControllerTest.java @@ -77,6 +77,8 @@ public class SatelliteSettingsPreferenceCategoryControllerTest { CarrierConfigCache.setTestInstance(mContext, mCarrierConfigCache); mController = new SatelliteSettingsPreferenceCategoryController(mContext, KEY); when(mCarrierConfigCache.getConfigForSubId(TEST_SUB_ID)).thenReturn(mPersistableBundle); + mPersistableBundle.putInt(KEY_CARRIER_ROAMING_NTN_CONNECT_TYPE_INT, + CARRIER_ROAMING_NTN_CONNECT_AUTOMATIC); when(mContext.getSystemService(SatelliteManager.class)).thenReturn(satelliteManager); mController.mIsSatelliteSupported.set(true); } @@ -95,7 +97,10 @@ public class SatelliteSettingsPreferenceCategoryControllerTest { @Test @EnableFlags(Flags.FLAG_CARRIER_ENABLED_SATELLITE_FLAG) public void getAvailabilityStatus_deviceUnsupported_returnUnsupported() { + mPersistableBundle.putInt(KEY_CARRIER_ROAMING_NTN_CONNECT_TYPE_INT, + CARRIER_ROAMING_NTN_CONNECT_MANUAL); mController.mIsSatelliteSupported.set(false); + mController.init(TEST_SUB_ID); int result = mController.getAvailabilityStatus(TEST_SUB_ID);