Merge "In High usage, hide apps without screen time." into pi-dev am: 6f61ae0adf
am: c6882cf458
Change-Id: Icaf40316eaff1db5dd711c92400179bea4f58650
This commit is contained in:
@@ -17,8 +17,10 @@
|
||||
package com.android.settings.fuelgauge.batterytip.detectors;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import static org.mockito.Mockito.doNothing;
|
||||
import static org.mockito.Mockito.doReturn;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.spy;
|
||||
import static org.mockito.Mockito.when;
|
||||
|
||||
@@ -48,7 +50,8 @@ import java.util.List;
|
||||
|
||||
@RunWith(SettingsRobolectricTestRunner.class)
|
||||
public class HighUsageDetectorTest {
|
||||
private static final int UID = 123;
|
||||
private static final int UID_HIGH = 123;
|
||||
private static final int UID_ZERO = 345;
|
||||
private static final long SCREEN_ON_TIME_MS = DateUtils.HOUR_IN_MILLIS;
|
||||
private Context mContext;
|
||||
@Mock
|
||||
@@ -56,7 +59,9 @@ public class HighUsageDetectorTest {
|
||||
@Mock
|
||||
private BatteryUtils mBatteryUtils;
|
||||
@Mock
|
||||
private BatterySipper mBatterySipper;
|
||||
private BatterySipper mHighBatterySipper;
|
||||
@Mock
|
||||
private BatterySipper mZeroBatterySipper;
|
||||
@Mock
|
||||
private HighUsageDataParser mDataParser;
|
||||
|
||||
@@ -75,14 +80,25 @@ public class HighUsageDetectorTest {
|
||||
mHighUsageDetector.mBatteryUtils = mBatteryUtils;
|
||||
mHighUsageDetector.mDataParser = mDataParser;
|
||||
doNothing().when(mHighUsageDetector).parseBatteryData();
|
||||
doReturn(UID).when(mBatterySipper).getUid();
|
||||
doReturn(UID_HIGH).when(mHighBatterySipper).getUid();
|
||||
mHighBatterySipper.uidObj = mock(BatteryStats.Uid.class);
|
||||
mZeroBatterySipper.uidObj = mock(BatteryStats.Uid.class);
|
||||
doReturn(UID_ZERO).when(mZeroBatterySipper).getUid();
|
||||
mAppInfo = new AppInfo.Builder()
|
||||
.setUid(UID)
|
||||
.setUid(UID_HIGH)
|
||||
.setScreenOnTimeMs(SCREEN_ON_TIME_MS)
|
||||
.build();
|
||||
|
||||
doReturn(SCREEN_ON_TIME_MS).when(mBatteryUtils).getProcessTimeMs(
|
||||
BatteryUtils.StatusType.FOREGROUND, mHighBatterySipper.uidObj,
|
||||
BatteryStats.STATS_SINCE_CHARGED);
|
||||
doReturn(0L).when(mBatteryUtils).getProcessTimeMs(
|
||||
BatteryUtils.StatusType.FOREGROUND, mZeroBatterySipper.uidObj,
|
||||
BatteryStats.STATS_SINCE_CHARGED);
|
||||
|
||||
mUsageList = new ArrayList<>();
|
||||
mUsageList.add(mBatterySipper);
|
||||
mUsageList.add(mHighBatterySipper);
|
||||
when(mBatteryStatsHelper.getUsageList()).thenReturn(mUsageList);
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -95,10 +111,16 @@ public class HighUsageDetectorTest {
|
||||
@Test
|
||||
public void testDetect_containsHighUsageApp_tipVisible() {
|
||||
doReturn(true).when(mDataParser).isDeviceHeavilyUsed();
|
||||
when(mBatteryStatsHelper.getUsageList()).thenReturn(mUsageList);
|
||||
doReturn(SCREEN_ON_TIME_MS).when(mBatteryUtils).getProcessTimeMs(
|
||||
BatteryUtils.StatusType.FOREGROUND, mBatterySipper.uidObj,
|
||||
BatteryStats.STATS_SINCE_CHARGED);
|
||||
|
||||
final HighUsageTip highUsageTip = (HighUsageTip) mHighUsageDetector.detect();
|
||||
assertThat(highUsageTip.isVisible()).isTrue();
|
||||
assertThat(highUsageTip.getHighUsageAppList()).containsExactly(mAppInfo);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testDetect_containsHighUsageApp_removeZeroOne() {
|
||||
doReturn(true).when(mDataParser).isDeviceHeavilyUsed();
|
||||
mUsageList.add(mZeroBatterySipper);
|
||||
|
||||
final HighUsageTip highUsageTip = (HighUsageTip) mHighUsageDetector.detect();
|
||||
assertThat(highUsageTip.isVisible()).isTrue();
|
||||
|
Reference in New Issue
Block a user