From 8881b110cd5084f503a70e4b9d178041c810746a Mon Sep 17 00:00:00 2001 From: Blake Kragten Date: Mon, 16 Mar 2020 16:46:03 -0700 Subject: [PATCH] Enhanced Connectivity Enabled Default Changed to enable enhanced connectivity by default Test: On Device Test: make RunSettingsRoboTests ROBOTEST_FILTER=EnhancedConnectivityPreferenceControllerTest Test: On-device using developer option menu Bug: 151633345 Change-Id: I8706f3a15af02214f4516503e685ff2b8f348a3a --- .../EnhancedConnectivityPreferenceController.java | 8 ++++---- ...EnhancedConnectivityPreferenceControllerTest.java | 12 ++++++------ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/com/android/settings/development/EnhancedConnectivityPreferenceController.java b/src/com/android/settings/development/EnhancedConnectivityPreferenceController.java index 2dbd7d1ee2b..dc6597384d2 100644 --- a/src/com/android/settings/development/EnhancedConnectivityPreferenceController.java +++ b/src/com/android/settings/development/EnhancedConnectivityPreferenceController.java @@ -37,7 +37,7 @@ public class EnhancedConnectivityPreferenceController extends @VisibleForTesting static final int ENHANCED_CONNECTIVITY_ON = 1; - // default is enhanced connectivity disabled. + // default is enhanced connectivity enabled. @VisibleForTesting static final int ENHANCED_CONNECTIVITY_OFF = 0; @@ -65,7 +65,7 @@ public class EnhancedConnectivityPreferenceController extends public void updateState(Preference preference) { final int enhancedConnectivityEnabled = Settings.Global.getInt( mContext.getContentResolver(), Settings.Global.ENHANCED_CONNECTIVITY_ENABLED, - ENHANCED_CONNECTIVITY_OFF); + ENHANCED_CONNECTIVITY_ON); ((SwitchPreference) mPreference).setChecked( enhancedConnectivityEnabled == ENHANCED_CONNECTIVITY_ON); } @@ -75,7 +75,7 @@ public class EnhancedConnectivityPreferenceController extends super.onDeveloperOptionsSwitchDisabled(); Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.ENHANCED_CONNECTIVITY_ENABLED, - ENHANCED_CONNECTIVITY_OFF); - ((SwitchPreference) mPreference).setChecked(false); + ENHANCED_CONNECTIVITY_ON); + ((SwitchPreference) mPreference).setChecked(true); } } diff --git a/tests/robotests/src/com/android/settings/development/EnhancedConnectivityPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/development/EnhancedConnectivityPreferenceControllerTest.java index 6fd6f553850..171fc6bc5de 100644 --- a/tests/robotests/src/com/android/settings/development/EnhancedConnectivityPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/development/EnhancedConnectivityPreferenceControllerTest.java @@ -88,20 +88,20 @@ public class EnhancedConnectivityPreferenceControllerTest { } @Test - public void onDeveloperOptionsDisabled_shouldDisablePreference() { - mController.onDeveloperOptionsDisabled(); + public void onDeveloperOptionsDisabled_shouldEnablePreference() { + mController.onDeveloperOptionsSwitchDisabled(); verify(mPreference).setEnabled(false); - verify(mPreference).setChecked(false); + verify(mPreference).setChecked(true); - assertThat(isSettingEnabled()).isFalse(); + assertThat(isSettingEnabled()).isTrue(); } private boolean isSettingEnabled() { return Settings.Global.getInt(mContext.getContentResolver(), Settings.Global.ENHANCED_CONNECTIVITY_ENABLED, - EnhancedConnectivityPreferenceController.ENHANCED_CONNECTIVITY_OFF - /* default off */) + EnhancedConnectivityPreferenceController.ENHANCED_CONNECTIVITY_ON + /* default on */) == EnhancedConnectivityPreferenceController.ENHANCED_CONNECTIVITY_ON; } }