Merge "Default to timing out to dock user in 1 minute" into tm-qpr-dev

This commit is contained in:
William Xiao
2022-12-15 20:04:15 +00:00
committed by Android (Google) Code Review
3 changed files with 14 additions and 4 deletions

View File

@@ -37,7 +37,8 @@ import java.util.List;
*/
public class TimeoutToDockUserSettings extends RadioButtonPickerFragment {
// Index of the default key of the timeout setting if it hasn't been changed by the user.
public static final int DEFAULT_TIMEOUT_SETTING_VALUE_INDEX = 0;
// Default to the smallest non-zero option (which is currently 1 minute).
public static final int DEFAULT_TIMEOUT_SETTING_VALUE_INDEX = 1;
// Labels of the options, for example, "never", "after 5 minutes".
private String[] mEntries;

View File

@@ -124,7 +124,8 @@ public class TimeoutToDockUserPreferenceControllerTest {
Settings.Secure.putStringForUser(mContext.getContentResolver(), TIMEOUT_TO_DOCK_USER,
null, UserHandle.myUserId());
assertThat(mController.getSummary().toString()).isEqualTo(mEntries[0]);
assertThat(mController.getSummary().toString()).isEqualTo(
mEntries[TimeoutToDockUserSettings.DEFAULT_TIMEOUT_SETTING_VALUE_INDEX]);
}
@Test

View File

@@ -78,8 +78,9 @@ public class TimeoutToDockUserSettingsTest {
}
@Test
public void defaultKey_settingNotSet_shouldReturnFirstValueAsDefault() {
assertThat(mSettings.getDefaultKey()).isEqualTo(mValues[0]);
public void defaultKey_settingNotSet_shouldReturnSecondValueAsDefault() {
assertThat(mSettings.getDefaultKey()).isEqualTo(
mValues[TimeoutToDockUserSettings.DEFAULT_TIMEOUT_SETTING_VALUE_INDEX]);
}
@Test
@@ -95,4 +96,11 @@ public class TimeoutToDockUserSettingsTest {
mSettings.setDefaultKey(expectedKey);
assertThat(mSettings.getDefaultKey()).isEqualTo(expectedKey);
}
@Test
public void defaultKey_setToThirdValue_shouldSaveToSettings() {
final String expectedKey = mValues[2];
mSettings.setDefaultKey(expectedKey);
assertThat(mSettings.getDefaultKey()).isEqualTo(expectedKey);
}
}