PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Init BatteryUtils in constructor

Browse Source 
In this case we won't have the NPE.

Bug: 63526637
Test: RunSettingsRoboTests
Change-Id: I20375e535ba07deddacf74052f31476e0cb28e56
This commit is contained in:
jackqdyulei
2017-07-10 11:00:05 -07:00
parent 8dc795d64b
commit 116fec5b25
2 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/com/android/settings/fuelgauge/anomaly/action/BackgroundCheckAction.java
View File

@@ -38,6 +38,7 @@ public class BackgroundCheckAction extends AnomalyAction {
super(context); super(context);
mAppOpsManager = (AppOpsManager) context.getSystemService(Context.APP_OPS_SERVICE); mAppOpsManager = (AppOpsManager) context.getSystemService(Context.APP_OPS_SERVICE);
mActionMetricKey = MetricsProto.MetricsEvent.ACTION_APP_BACKGROUND_CHECK; mActionMetricKey = MetricsProto.MetricsEvent.ACTION_APP_BACKGROUND_CHECK;
mBatteryUtils = BatteryUtils.getInstance(context);
} }
@Override @Override

7
tests/robotests/src/com/android/settings/fuelgauge/anomaly/action/BackgroundCheckActionTest.java
View File

@@ -94,4 +94,11 @@ public class BackgroundCheckActionTest {
assertThat(mBackgroundCheckAction.isActionActive(mAnomaly)).isFalse(); assertThat(mBackgroundCheckAction.isActionActive(mAnomaly)).isFalse();
} }
@Test
public void testConstructor_batteryUtilsNotNull() {
mBackgroundCheckAction = new BackgroundCheckAction(mContext);
assertThat(mBackgroundCheckAction.mBatteryUtils).isNotNull();
}
} }