Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2d6c7df8 authored by Matthew Fritze's avatar Matthew Fritze Committed by android-build-merger
Browse files

Merge "Revert "Replace dynamic summary text for SettingPref"" into oc-dev am: bcb2565e

am: f8b8eceb

Change-Id: I4ae1d981c928e1b61d555d129723b7dabafd439c
parents b900846e f8b8eceb
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -22,6 +22,6 @@
    <DropDownPreference
            android:key="turn_on_automatically"
            android:title="@string/battery_saver_turn_on_automatically_title"
            android:summary="@string/summary_placeholder" />
            android:summary="%s" />

</PreferenceScreen>
+2 −2
Original line number Diff line number Diff line
@@ -124,7 +124,7 @@
        <DropDownPreference
          android:key="dock_audio_media"
          android:title="@string/dock_audio_media_title"
          android:summary="@string/summary_placeholder" />
          android:summary="%s" />

        <!-- Boot sounds -->
        <SwitchPreference
@@ -135,7 +135,7 @@
        <DropDownPreference
          android:key="emergency_tone"
          android:title="@string/emergency_tone_title"
          android:summary="@string/summary_placeholder" />
          android:summary="%s" />

        <com.android.settingslib.RestrictedPreference
          android:key="cell_broadcast_settings"
+1 −0
Original line number Diff line number Diff line
@@ -99,6 +99,7 @@ public class BatterySaverSettings extends SettingsPreferenceFragment
            }
        };
        mTriggerPref.init(this);

        mPowerManager = (PowerManager) mContext.getSystemService(Context.POWER_SERVICE);
    }

+0 −13
Original line number Diff line number Diff line
@@ -121,19 +121,6 @@ public class SettingPref {
        if (mTwoState != null) {
            mTwoState.setChecked(val != 0);
        } else if (mDropDown != null) {
            if (mValues != null) {
                int index = 0;
                for (int len = mValues.length; index < len; index++) {
                    if (mValues[index] == val) {
                        break;
                    }
                }

                if (index < mValues.length) {
                    CharSequence entry = mDropDown.getEntries()[index];
                    mDropDown.setSummary(entry);
                }
            }
            mDropDown.setValue(Integer.toString(val));
        }
    }
+0 −67
Original line number Diff line number Diff line
package com.android.settings.notification;

import android.content.res.Resources;

import android.content.Context;
import android.provider.Settings;
import android.support.v7.preference.DropDownPreference;
import com.android.settings.SettingsRobolectricTestRunner;
import com.android.settings.TestConfig;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.robolectric.RuntimeEnvironment;
import org.robolectric.annotation.Config;

import static com.google.common.truth.Truth.assertThat;
import static org.mockito.Mockito.spy;

@RunWith(SettingsRobolectricTestRunner.class)
@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION)
public class SettingPrefTest {

    @Test
    public void update_setsDropDownSummaryText() {
        Context context = RuntimeEnvironment.application;
        String testSetting = "test_setting";
        int[] values = new int[] {1,2,3};
        String[] entries = new String[] {"one", "two", "three"};
        SettingPref settingPref =
                spy(new SettingPref(SettingPref.TYPE_GLOBAL, "key", testSetting, 0, values) {
                    @Override
                    protected String getCaption(Resources res, int value) {
                        return "temp";
                    }
                });
        DropDownPreference dropdownPref = spy(new DropDownPreference(context));
        dropdownPref.setEntries(entries);
        settingPref.mDropDown = dropdownPref;
        Settings.Global.putInt(context.getContentResolver(), testSetting, values[2]);

        settingPref.update(context);

        assertThat(settingPref.mDropDown.getSummary()).isEqualTo(entries[2]);
    }

    @Test
    public void update_setsDropDownSummaryText_noMatch_noError() {
        Context context = RuntimeEnvironment.application;
        String testSetting = "test_setting";
        int[] values = new int[] {1,2,3};
        String[] entries = new String[] {"one", "two", "three"};
        SettingPref settingPref =
                spy(new SettingPref(SettingPref.TYPE_GLOBAL, "key", testSetting, 0, values) {
                    @Override
                    protected String getCaption(Resources res, int value) {
                        return "temp";
                    }
                });
        DropDownPreference dropdownPref = spy(new DropDownPreference(context));
        dropdownPref.setEntries(entries);
        settingPref.mDropDown = dropdownPref;
        Settings.Global.putInt(context.getContentResolver(), testSetting, -1);

        settingPref.update(context);

        assertThat(settingPref.mDropDown.getSummary()).isNull();
    }
}