| commit | 88c86f73bb27fa00041ca11e03aa2370d40c19d7 | [log] [tgz] |
|---|---|---|
| author | jackqdyulei <jackqdyulei@google.com> | Thu Feb 08 10:12:34 2018 -0800 |
| committer | jackqdyulei <jackqdyulei@google.com> | Thu Feb 08 10:25:24 2018 -0800 |
| tree | f2606bfe62db1c8fd7fa4ac426abe2b7cc3733a4 | |
| parent | b1c7b88f100ed1d9cfca2c5fa17c55be97c57ba8 [diff] |
Don't use debug data in app list controller This is a bug, USE_FAKE_DATA should never be true in build Change-Id: I14059cd243380c754ca0cff09f088d62828ec755 Fixes: 73117246 Test: Build
diff --git a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java index 2095f25..2f93d0a 100644 --- a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java +++ b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java
@@ -62,7 +62,8 @@ */ public class BatteryAppListPreferenceController extends AbstractPreferenceController implements PreferenceControllerMixin, LifecycleObserver, OnPause, OnDestroy { - private static final boolean USE_FAKE_DATA = true; + @VisibleForTesting + static final boolean USE_FAKE_DATA = false; private static final int MAX_ITEMS_TO_LIST = USE_FAKE_DATA ? 30 : 10; private static final int MIN_AVERAGE_POWER_THRESHOLD_MILLI_AMP = 10; private static final int STATS_TYPE = BatteryStats.STATS_SINCE_CHARGED;
diff --git a/tests/robotests/src/com/android/settings/fuelgauge/BatteryAppListPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/fuelgauge/BatteryAppListPreferenceControllerTest.java index a814989..cee84de 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/BatteryAppListPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/BatteryAppListPreferenceControllerTest.java
@@ -220,4 +220,9 @@ assertThat(mPreferenceController.isAvailable()).isFalse(); } + + @Test + public void testNeverUseFakeData() { + assertThat(BatteryAppListPreferenceController.USE_FAKE_DATA).isFalse(); + } }