From 57351996415477b20543e0e670c59c739c5285a3 Mon Sep 17 00:00:00 2001 From: Yiwei Zhang Date: Fri, 11 Jan 2019 04:40:04 +0800 Subject: [PATCH] GUP: Fixed some typos and update some values Bug: 119221883 Test: make RunSettingsRoboTests Change-Id: I7b1e42cd3a823a72bcd6d61b26c4954c237ff6d0 --- res/values/strings.xml | 24 +++++++++---------- .../gup/GupPreferenceController.java | 15 ++++++------ .../gup/GupPreferenceControllerTest.java | 22 ++++++++++------- 3 files changed, 33 insertions(+), 28 deletions(-) diff --git a/res/values/strings.xml b/res/values/strings.xml index 88fe4eb6a53..fddcdc8f85b 100644 --- a/res/values/strings.xml +++ b/res/values/strings.xml @@ -10055,23 +10055,23 @@ ANGLE enabled application: %1$s - - Game Update Packages Preferences - - Modify Game Update Packages settings - + + Game Update Package Preferences + + Modify Game Update Package settings + Select Graphics Driver - + Default - - Game Update Packages - - Native Graphics Driver - + + Game Update Package + + System Graphics Driver + @string/gup_app_preference_default @string/gup_app_preference_gup - @string/gup_app_preference_native + @string/gup_app_preference_system diff --git a/src/com/android/settings/development/gup/GupPreferenceController.java b/src/com/android/settings/development/gup/GupPreferenceController.java index 762314474ef..d4cd2f1284e 100644 --- a/src/com/android/settings/development/gup/GupPreferenceController.java +++ b/src/com/android/settings/development/gup/GupPreferenceController.java @@ -48,7 +48,7 @@ public class GupPreferenceController private final String mPreferenceTitle; private final String mPreferenceDefault; private final String mPreferenceGup; - private final String mPreferenceNative; + private final String mPreferenceSystem; private final List mAppInfos; private final Set mDevOptInApps; @@ -62,7 +62,7 @@ public class GupPreferenceController mPreferenceTitle = resources.getString(R.string.gup_app_preference_title); mPreferenceDefault = resources.getString(R.string.gup_app_preference_default); mPreferenceGup = resources.getString(R.string.gup_app_preference_gup); - mPreferenceNative = resources.getString(R.string.gup_app_preference_native); + mPreferenceSystem = resources.getString(R.string.gup_app_preference_system); // TODO: Move this task to background if there's potential ANR/Jank. // Update the UI when all the app infos are ready. @@ -105,19 +105,18 @@ public class GupPreferenceController // When user choose a new preference, update both Sets for // opt-in and opt-out apps. Then set the new summary text. - if (value.equals(mPreferenceNative)) { + if (value.equals(mPreferenceSystem)) { mDevOptInApps.remove(packageName); mDevOptOutApps.add(packageName); - listPref.setSummary(mPreferenceNative); } else if (value.equals(mPreferenceGup)) { mDevOptInApps.add(packageName); mDevOptOutApps.remove(packageName); - listPref.setSummary(mPreferenceGup); } else { mDevOptInApps.remove(packageName); mDevOptOutApps.remove(packageName); - listPref.setSummary(mPreferenceDefault); } + listPref.setValue(value); + listPref.setSummary(value); // Push the updated Sets for opt-in and opt-out apps to // corresponding Settings.Global.GUP_DEV_OPT_(IN|OUT)_APPS @@ -189,8 +188,8 @@ public class GupPreferenceController // Initialize preference default and summary with the opt in/out choices // from Settings.Global.GUP_DEV_OPT_(IN|OUT)_APPS if (mDevOptOutApps.contains(packageName)) { - listPreference.setValue(mPreferenceNative); - listPreference.setSummary(mPreferenceNative); + listPreference.setValue(mPreferenceSystem); + listPreference.setSummary(mPreferenceSystem); } else if (mDevOptInApps.contains(packageName)) { listPreference.setValue(mPreferenceGup); listPreference.setSummary(mPreferenceGup); diff --git a/tests/robotests/src/com/android/settings/development/gup/GupPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/development/gup/GupPreferenceControllerTest.java index 62e3475bd25..d5e7a852273 100644 --- a/tests/robotests/src/com/android/settings/development/gup/GupPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/development/gup/GupPreferenceControllerTest.java @@ -52,7 +52,7 @@ import org.robolectric.RuntimeEnvironment; public class GupPreferenceControllerTest { private static final int DEFAULT = 0; private static final int GUP = 1; - private static final int NATIVE = 2; + private static final int SYSTEM = 2; private static final String TEST_APP_NAME = "testApp"; private static final String TEST_PKG_NAME = "testPkg"; @@ -143,7 +143,7 @@ public class GupPreferenceControllerTest { } @Test - public void createPreference_configNative_shouldSetNativeAttributes() { + public void createPreference_configSystem_shouldSetSystemAttributes() { loadConfig("", TEST_PKG_NAME); final ListPreference preference = mController.createListPreference(TEST_PKG_NAME, TEST_APP_NAME); @@ -153,9 +153,9 @@ public class GupPreferenceControllerTest { assertThat(preference.getDialogTitle()).isEqualTo(mDialogTitle); assertThat(preference.getEntries()).isEqualTo(mValueList); assertThat(preference.getEntryValues()).isEqualTo(mValueList); - assertThat(preference.getEntry()).isEqualTo(mValueList[NATIVE]); - assertThat(preference.getValue()).isEqualTo(mValueList[NATIVE]); - assertThat(preference.getSummary()).isEqualTo(mValueList[NATIVE]); + assertThat(preference.getEntry()).isEqualTo(mValueList[SYSTEM]); + assertThat(preference.getValue()).isEqualTo(mValueList[SYSTEM]); + assertThat(preference.getSummary()).isEqualTo(mValueList[SYSTEM]); } @Test @@ -165,6 +165,8 @@ public class GupPreferenceControllerTest { mController.createListPreference(TEST_PKG_NAME, TEST_APP_NAME); mController.onPreferenceChange(preference, mValueList[DEFAULT]); + assertThat(preference.getEntry()).isEqualTo(mValueList[DEFAULT]); + assertThat(preference.getValue()).isEqualTo(mValueList[DEFAULT]); assertThat(preference.getSummary()).isEqualTo(mValueList[DEFAULT]); assertThat(Settings.Global.getString(mResolver, Settings.Global.GUP_DEV_OPT_IN_APPS)) .isEqualTo(""); @@ -179,6 +181,8 @@ public class GupPreferenceControllerTest { mController.createListPreference(TEST_PKG_NAME, TEST_APP_NAME); mController.onPreferenceChange(preference, mValueList[GUP]); + assertThat(preference.getEntry()).isEqualTo(mValueList[GUP]); + assertThat(preference.getValue()).isEqualTo(mValueList[GUP]); assertThat(preference.getSummary()).isEqualTo(mValueList[GUP]); assertThat(Settings.Global.getString(mResolver, Settings.Global.GUP_DEV_OPT_IN_APPS)) .isEqualTo(TEST_PKG_NAME); @@ -187,13 +191,15 @@ public class GupPreferenceControllerTest { } @Test - public void onPreferenceChange_selectNative_shouldUpdateAttributesAndSettingsGlobal() { + public void onPreferenceChange_selectSystem_shouldUpdateAttributesAndSettingsGlobal() { loadDefaultConfig(); final ListPreference preference = mController.createListPreference(TEST_PKG_NAME, TEST_APP_NAME); - mController.onPreferenceChange(preference, mValueList[NATIVE]); + mController.onPreferenceChange(preference, mValueList[SYSTEM]); - assertThat(preference.getSummary()).isEqualTo(mValueList[NATIVE]); + assertThat(preference.getEntry()).isEqualTo(mValueList[SYSTEM]); + assertThat(preference.getValue()).isEqualTo(mValueList[SYSTEM]); + assertThat(preference.getSummary()).isEqualTo(mValueList[SYSTEM]); assertThat(Settings.Global.getString(mResolver, Settings.Global.GUP_DEV_OPT_IN_APPS)) .isEqualTo(""); assertThat(Settings.Global.getString(mResolver, Settings.Global.GUP_DEV_OPT_OUT_APPS))