Merge cherrypicks of ['googleplex-android-review.googlesource.com/28121446'] into 24Q3-release.
Change-Id: I062a7bf67678f78f60f360df0590c9b12b9c75a5
This commit is contained in:
@@ -23,11 +23,16 @@ import android.os.Bundle;
|
||||
import android.util.ArrayMap;
|
||||
import android.util.SparseIntArray;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryDiffData;
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryEvent;
|
||||
import com.android.settings.fuelgauge.batteryusage.DetectRequestSourceType;
|
||||
import com.android.settings.fuelgauge.batteryusage.PowerAnomalyEventList;
|
||||
import com.android.settingslib.fuelgauge.Estimate;
|
||||
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.Set;
|
||||
|
||||
/** Feature Provider used in power usage */
|
||||
@@ -154,4 +159,14 @@ public interface PowerUsageFeatureProvider {
|
||||
|
||||
/** Whether the device is under the battery defender mode */
|
||||
boolean isBatteryDefend(BatteryInfo info);
|
||||
|
||||
/** Whether the battery usage reattribute is eabled or not. */
|
||||
boolean isBatteryUsageReattributeEnabled();
|
||||
|
||||
/** Collect and process battery reattribute data if needed. */
|
||||
boolean processBatteryReattributeData(
|
||||
@NonNull Context context,
|
||||
@NonNull Map<Long, BatteryDiffData> batteryDiffDataMap,
|
||||
@NonNull List<BatteryEvent> batteryEventList,
|
||||
final boolean isFromPeriodJob);
|
||||
}
|
||||
|
@@ -18,7 +18,6 @@ package com.android.settings.fuelgauge;
|
||||
|
||||
import static com.android.settings.Utils.SYSTEMUI_PACKAGE_NAME;
|
||||
|
||||
import android.annotation.Nullable;
|
||||
import android.content.Context;
|
||||
import android.content.Intent;
|
||||
import android.content.pm.PackageManager;
|
||||
@@ -27,13 +26,19 @@ import android.util.ArrayMap;
|
||||
import android.util.ArraySet;
|
||||
import android.util.SparseIntArray;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
import androidx.annotation.Nullable;
|
||||
|
||||
import com.android.internal.util.ArrayUtils;
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryDiffData;
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryEvent;
|
||||
import com.android.settings.fuelgauge.batteryusage.DetectRequestSourceType;
|
||||
import com.android.settings.fuelgauge.batteryusage.PowerAnomalyEventList;
|
||||
import com.android.settingslib.fuelgauge.Estimate;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.Set;
|
||||
|
||||
/** Implementation of {@code PowerUsageFeatureProvider} */
|
||||
@@ -240,4 +245,18 @@ public class PowerUsageFeatureProviderImpl implements PowerUsageFeatureProvider
|
||||
public boolean isBatteryDefend(BatteryInfo info) {
|
||||
return info.isBatteryDefender && !isExtraDefend();
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean isBatteryUsageReattributeEnabled() {
|
||||
return false;
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean processBatteryReattributeData(
|
||||
@NonNull Context context,
|
||||
@NonNull Map<Long, BatteryDiffData> batteryDiffDataMap,
|
||||
@NonNull List<BatteryEvent> batteryEventList,
|
||||
final boolean isFromPeriodJob) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
@@ -77,11 +77,13 @@ public class BatteryDiffData {
|
||||
processAndSortEntries(mSystemEntries);
|
||||
}
|
||||
|
||||
long getStartTimestamp() {
|
||||
/** Gets the start timestamp. */
|
||||
public long getStartTimestamp() {
|
||||
return mStartTimestamp;
|
||||
}
|
||||
|
||||
long getEndTimestamp() {
|
||||
/** Gets the end timestamp. */
|
||||
public long getEndTimestamp() {
|
||||
return mEndTimestamp;
|
||||
}
|
||||
|
||||
@@ -97,7 +99,8 @@ public class BatteryDiffData {
|
||||
return mScreenOnTime;
|
||||
}
|
||||
|
||||
List<BatteryDiffEntry> getAppDiffEntryList() {
|
||||
/** Gets the {@link BatteryDiffEntry} list for apps. */
|
||||
public List<BatteryDiffEntry> getAppDiffEntryList() {
|
||||
return mAppEntries;
|
||||
}
|
||||
|
||||
@@ -296,8 +299,7 @@ public class BatteryDiffData {
|
||||
* Sets total consume power, and adjusts the percentages to ensure the total round percentage
|
||||
* could be 100%, and then sorts entries based on the sorting key.
|
||||
*/
|
||||
@VisibleForTesting
|
||||
static void processAndSortEntries(final List<BatteryDiffEntry> batteryDiffEntries) {
|
||||
public static void processAndSortEntries(final List<BatteryDiffEntry> batteryDiffEntries) {
|
||||
if (batteryDiffEntries.isEmpty()) {
|
||||
return;
|
||||
}
|
||||
|
@@ -201,6 +201,20 @@ public final class ConvertUtils {
|
||||
return defaultInstance;
|
||||
}
|
||||
|
||||
/** Gets the encoded string from {@link BatteryReattribute} instance. */
|
||||
@NonNull
|
||||
public static String encodeBatteryReattribute(
|
||||
@NonNull BatteryReattribute batteryReattribute) {
|
||||
return Base64.encodeToString(batteryReattribute.toByteArray(), Base64.DEFAULT);
|
||||
}
|
||||
|
||||
/** Gets the decoded {@link BatteryReattribute} instance from string. */
|
||||
@NonNull
|
||||
public static BatteryReattribute decodeBatteryReattribute(@NonNull String content) {
|
||||
return BatteryUtils.parseProtoFromString(
|
||||
content, BatteryReattribute.getDefaultInstance());
|
||||
}
|
||||
|
||||
/** Converts to {@link BatteryHistEntry} */
|
||||
public static BatteryHistEntry convertToBatteryHistEntry(
|
||||
BatteryEntry entry, BatteryUsageStats batteryUsageStats) {
|
||||
|
@@ -28,6 +28,8 @@ import androidx.annotation.NonNull;
|
||||
import androidx.annotation.Nullable;
|
||||
|
||||
import com.android.internal.annotations.VisibleForTesting;
|
||||
import com.android.settings.fuelgauge.PowerUsageFeatureProvider;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.Calendar;
|
||||
@@ -78,6 +80,7 @@ public class DataProcessManager {
|
||||
// Raw start timestamp with round to the nearest hour.
|
||||
private final long mRawStartTimestamp;
|
||||
private final long mLastFullChargeTimestamp;
|
||||
private final boolean mIsFromPeriodJob;
|
||||
private final Context mContext;
|
||||
private final Handler mHandler;
|
||||
private final UserIdsSeries mUserIdsSeries;
|
||||
@@ -122,6 +125,7 @@ public class DataProcessManager {
|
||||
Context context,
|
||||
Handler handler,
|
||||
final UserIdsSeries userIdsSeries,
|
||||
final boolean isFromPeriodJob,
|
||||
final long rawStartTimestamp,
|
||||
final long lastFullChargeTimestamp,
|
||||
@NonNull final OnBatteryDiffDataMapLoadedListener callbackFunction,
|
||||
@@ -130,6 +134,7 @@ public class DataProcessManager {
|
||||
mContext = context.getApplicationContext();
|
||||
mHandler = handler;
|
||||
mUserIdsSeries = userIdsSeries;
|
||||
mIsFromPeriodJob = isFromPeriodJob;
|
||||
mRawStartTimestamp = rawStartTimestamp;
|
||||
mLastFullChargeTimestamp = lastFullChargeTimestamp;
|
||||
mCallbackFunction = callbackFunction;
|
||||
@@ -147,6 +152,7 @@ public class DataProcessManager {
|
||||
mHandler = handler;
|
||||
mUserIdsSeries = userIdsSeries;
|
||||
mCallbackFunction = callbackFunction;
|
||||
mIsFromPeriodJob = false;
|
||||
mRawStartTimestamp = 0L;
|
||||
mLastFullChargeTimestamp = 0L;
|
||||
mHourlyBatteryLevelsPerDay = null;
|
||||
@@ -158,14 +164,9 @@ public class DataProcessManager {
|
||||
|
||||
/** Starts the async tasks to load battery history data and app usage data. */
|
||||
public void start() {
|
||||
start(/* isFromPeriodJob= */ false);
|
||||
}
|
||||
|
||||
/** Starts the async tasks to load battery history data and app usage data. */
|
||||
public void start(boolean isFromPeriodJob) {
|
||||
// If we have battery level data, load the battery history map and app usage simultaneously.
|
||||
if (mHourlyBatteryLevelsPerDay != null) {
|
||||
if (isFromPeriodJob) {
|
||||
if (mIsFromPeriodJob) {
|
||||
mIsCurrentBatteryHistoryLoaded = true;
|
||||
mIsCurrentAppUsageLoaded = true;
|
||||
mIsBatteryUsageSlotLoaded = true;
|
||||
@@ -514,6 +515,14 @@ public class DataProcessManager {
|
||||
mAppUsagePeriodMap,
|
||||
getSystemAppsPackageNames(),
|
||||
getSystemAppsUids()));
|
||||
// Process the reattributate data for the following two cases:
|
||||
// 1) the latest slot for the timestamp "until now"
|
||||
// 2) walkthrough all BatteryDiffData again to handle "re-compute" case
|
||||
final PowerUsageFeatureProvider featureProvider =
|
||||
FeatureFactory.getFeatureFactory()
|
||||
.getPowerUsageFeatureProvider();
|
||||
featureProvider.processBatteryReattributeData(
|
||||
mContext, batteryDiffDataMap, mBatteryEventList, mIsFromPeriodJob);
|
||||
|
||||
Log.d(
|
||||
TAG,
|
||||
@@ -683,12 +692,13 @@ public class DataProcessManager {
|
||||
context,
|
||||
handler,
|
||||
userIdsSeries,
|
||||
isFromPeriodJob,
|
||||
startTimestamp,
|
||||
lastFullChargeTime,
|
||||
onBatteryDiffDataMapLoadedListener,
|
||||
batteryLevelData.getHourlyBatteryLevelsPerDay(),
|
||||
processedBatteryHistoryMap)
|
||||
.start(isFromPeriodJob);
|
||||
.start();
|
||||
|
||||
return batteryLevelData;
|
||||
}
|
||||
|
@@ -429,6 +429,7 @@ public final class DatabaseUtils {
|
||||
database.batteryEventDao().clearAll();
|
||||
database.batteryStateDao().clearAll();
|
||||
database.batteryUsageSlotDao().clearAll();
|
||||
database.batteryReattributeDao().clearAll();
|
||||
} catch (RuntimeException e) {
|
||||
Log.e(TAG, "clearAll() failed", e);
|
||||
}
|
||||
@@ -466,6 +467,7 @@ public final class DatabaseUtils {
|
||||
database.batteryEventDao().clearAllBefore(earliestTimestamp);
|
||||
database.batteryStateDao().clearAllBefore(earliestTimestamp);
|
||||
database.batteryUsageSlotDao().clearAllBefore(earliestTimestamp);
|
||||
database.batteryReattributeDao().clearAllBefore(earliestTimestamp);
|
||||
} catch (RuntimeException e) {
|
||||
Log.e(TAG, "clearAllBefore() failed", e);
|
||||
}
|
||||
|
@@ -55,6 +55,7 @@ public final class BugReportContentProvider extends ContentProvider {
|
||||
}
|
||||
writer.println("dump BatteryUsage and AppUsage states:");
|
||||
LogUtils.dumpAppOptimizationModeEventHist(context, writer);
|
||||
LogUtils.dumpBatteryReattributeDatabaseHist(context, writer);
|
||||
LogUtils.dumpBatteryUsageDatabaseHist(context, writer);
|
||||
LogUtils.dumpAppUsageDatabaseHist(context, writer);
|
||||
LogUtils.dumpBatteryUsageSlotDatabaseHist(context, writer);
|
||||
|
@@ -19,6 +19,8 @@ package com.android.settings.fuelgauge.batteryusage.bugreport;
|
||||
import android.content.Context;
|
||||
import android.util.Log;
|
||||
|
||||
import androidx.annotation.VisibleForTesting;
|
||||
|
||||
import com.android.settings.fuelgauge.BatteryUtils;
|
||||
import com.android.settings.fuelgauge.batteryusage.AppOptModeSharedPreferencesUtils;
|
||||
import com.android.settings.fuelgauge.batteryusage.AppOptimizationModeEvent;
|
||||
@@ -29,11 +31,14 @@ import com.android.settings.fuelgauge.batteryusage.db.AppUsageEventDao;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.AppUsageEventEntity;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryEventDao;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryEventEntity;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryReattributeDao;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryReattributeEntity;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryState;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryStateDao;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryStateDatabase;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryUsageSlotDao;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryUsageSlotEntity;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
|
||||
import java.io.PrintWriter;
|
||||
import java.time.Clock;
|
||||
@@ -127,6 +132,33 @@ public final class LogUtils {
|
||||
dumpListItems(writer, entities, entity -> entity);
|
||||
}
|
||||
|
||||
static void dumpBatteryReattributeDatabaseHist(Context context, PrintWriter writer) {
|
||||
try {
|
||||
dumpBatteryReattributeDatabaseHist(
|
||||
BatteryStateDatabase.getInstance(context).batteryReattributeDao(),
|
||||
writer);
|
||||
} catch (Exception e) {
|
||||
Log.e(TAG, "failed to run dumpBatteryReattributeDatabaseHist()", e);
|
||||
}
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
static void dumpBatteryReattributeDatabaseHist(
|
||||
BatteryReattributeDao batteryReattributeDao, PrintWriter writer) {
|
||||
if (!FeatureFactory.getFeatureFactory().getPowerUsageFeatureProvider()
|
||||
.isBatteryUsageReattributeEnabled()) {
|
||||
writer.println("\n\tBatteryReattribute is disabled!");
|
||||
return;
|
||||
}
|
||||
writer.println("\n\tBatteryReattribute DatabaseHistory:");
|
||||
final List<BatteryReattributeEntity> entities =
|
||||
batteryReattributeDao.getAllAfter(
|
||||
Clock.systemUTC().millis() - DUMP_TIME_OFFSET.toMillis());
|
||||
if (entities != null && !entities.isEmpty()) {
|
||||
dumpListItems(writer, entities, entity -> entity);
|
||||
}
|
||||
}
|
||||
|
||||
private static <T, S> void dumpListItems(
|
||||
PrintWriter writer, List<T> itemList, Function<T, S> itemConverter) {
|
||||
final AtomicInteger counter = new AtomicInteger(0);
|
||||
|
@@ -0,0 +1,50 @@
|
||||
/*
|
||||
* Copyright (C) 2024 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package com.android.settings.fuelgauge.batteryusage.db;
|
||||
|
||||
import androidx.room.Dao;
|
||||
import androidx.room.Insert;
|
||||
import androidx.room.OnConflictStrategy;
|
||||
import androidx.room.Query;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
/** DAO for accessing {@link BatteryReattributeEntity} in the database. */
|
||||
@Dao
|
||||
public interface BatteryReattributeDao {
|
||||
|
||||
/** Inserts a {@link BatteryReattributeEntity} data into the database. */
|
||||
@Insert(onConflict = OnConflictStrategy.REPLACE)
|
||||
fun insert(event: BatteryReattributeEntity)
|
||||
|
||||
/** Gets all recorded data after a specific timestamp. */
|
||||
@Query(
|
||||
"SELECT * FROM BatteryReattributeEntity WHERE "
|
||||
+ "timestampStart >= :timestampStart ORDER BY timestampStart DESC")
|
||||
fun getAllAfter(timestampStart: Long): List<BatteryReattributeEntity>
|
||||
|
||||
/** Deletes all recorded data before a specific timestamp. */
|
||||
@Query("DELETE FROM BatteryReattributeEntity WHERE timestampStart <= :timestampStart")
|
||||
fun clearAllBefore(timestampStart: Long)
|
||||
|
||||
/** Deletes all recorded data after a specific timestamp. */
|
||||
@Query("DELETE FROM BatteryReattributeEntity WHERE timestampStart >= :timestampStart")
|
||||
fun clearAllAfter(timestampStart: Long)
|
||||
|
||||
/** Clears all recorded data in the database. */
|
||||
@Query("DELETE FROM BatteryReattributeEntity") fun clearAll()
|
||||
}
|
@@ -0,0 +1,73 @@
|
||||
/*
|
||||
* Copyright (C) 2024 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package com.android.settings.fuelgauge.batteryusage.db;
|
||||
|
||||
import static com.android.settings.fuelgauge.batteryusage.ConvertUtils.utcToLocalTimeForLogging;
|
||||
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryReattribute;
|
||||
import com.android.settings.fuelgauge.batteryusage.ConvertUtils;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
import androidx.annotation.VisibleForTesting;
|
||||
import androidx.room.Entity;
|
||||
import androidx.room.PrimaryKey;
|
||||
|
||||
/** A {@link Entity} for battery usage reattribution data in the database. */
|
||||
@Entity
|
||||
public class BatteryReattributeEntity {
|
||||
|
||||
/** The start timestamp of this record data. */
|
||||
@PrimaryKey
|
||||
public final long timestampStart;
|
||||
|
||||
/** The end timestamp of this record data. */
|
||||
public final long timestampEnd;
|
||||
|
||||
/** The battery usage reattribution data for corresponding uids. */
|
||||
public final String reattributeData;
|
||||
|
||||
public BatteryReattributeEntity(@NonNull BatteryReattribute batteryReattribute) {
|
||||
this(
|
||||
batteryReattribute.getTimestampStart(),
|
||||
batteryReattribute.getTimestampEnd(),
|
||||
ConvertUtils.encodeBatteryReattribute(batteryReattribute));
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
BatteryReattributeEntity(
|
||||
long timestampStart, long timestampEnd, @NonNull String reattributeData) {
|
||||
this.timestampStart = timestampStart;
|
||||
this.timestampEnd = timestampEnd;
|
||||
this.reattributeData = reattributeData;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
@Override
|
||||
public String toString() {
|
||||
final BatteryReattribute batteryReattribute =
|
||||
ConvertUtils.decodeBatteryReattribute(reattributeData);
|
||||
final StringBuilder builder = new StringBuilder()
|
||||
.append("\nBatteryReattributeEntity{")
|
||||
.append("\n\t" + utcToLocalTimeForLogging(timestampStart))
|
||||
.append("\n\t" + utcToLocalTimeForLogging(timestampEnd))
|
||||
.append("\n\t" + batteryReattribute);
|
||||
if (batteryReattribute != null) {
|
||||
builder.append("\n\t" + batteryReattribute.getReattributeDataMap());
|
||||
}
|
||||
return builder.append("\n}").toString();
|
||||
}
|
||||
}
|
@@ -19,6 +19,7 @@ package com.android.settings.fuelgauge.batteryusage.db;
|
||||
import android.content.Context;
|
||||
import android.util.Log;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
import androidx.room.Database;
|
||||
import androidx.room.Room;
|
||||
import androidx.room.RoomDatabase;
|
||||
@@ -29,11 +30,13 @@ import androidx.room.RoomDatabase;
|
||||
AppUsageEventEntity.class,
|
||||
BatteryEventEntity.class,
|
||||
BatteryState.class,
|
||||
BatteryUsageSlotEntity.class
|
||||
BatteryUsageSlotEntity.class,
|
||||
BatteryReattributeEntity.class
|
||||
},
|
||||
version = 1)
|
||||
version = 2)
|
||||
public abstract class BatteryStateDatabase extends RoomDatabase {
|
||||
private static final String TAG = "BatteryStateDatabase";
|
||||
private static final String DB_FILE_NAME = "battery-usage-db-v10";
|
||||
|
||||
private static BatteryStateDatabase sBatteryStateDatabase;
|
||||
|
||||
@@ -49,11 +52,15 @@ public abstract class BatteryStateDatabase extends RoomDatabase {
|
||||
/** Provides DAO for battery usage slot table. */
|
||||
public abstract BatteryUsageSlotDao batteryUsageSlotDao();
|
||||
|
||||
/** Provides DAO for battery reattribution table. */
|
||||
@NonNull
|
||||
public abstract BatteryReattributeDao batteryReattributeDao();
|
||||
|
||||
/** Gets or creates an instance of {@link RoomDatabase}. */
|
||||
public static BatteryStateDatabase getInstance(Context context) {
|
||||
if (sBatteryStateDatabase == null) {
|
||||
sBatteryStateDatabase =
|
||||
Room.databaseBuilder(context, BatteryStateDatabase.class, "battery-usage-db-v9")
|
||||
Room.databaseBuilder(context, BatteryStateDatabase.class, DB_FILE_NAME)
|
||||
// Allows accessing data in the main thread for dumping bugreport.
|
||||
.allowMainThreadQueries()
|
||||
.fallbackToDestructiveMigration()
|
||||
|
@@ -0,0 +1,13 @@
|
||||
syntax = "proto3";
|
||||
|
||||
option java_multiple_files = true;
|
||||
option java_package = "com.android.settings.fuelgauge.batteryusage";
|
||||
option java_outer_classname = "BatteryReaatributeProto";
|
||||
|
||||
// Battery usage reattribute data for a specific timestamp slot.
|
||||
message BatteryReattribute {
|
||||
optional int64 timestamp_start = 1;
|
||||
optional int64 timestamp_end = 2;
|
||||
// Battery reattribute data for uid and its corresponding ratio.
|
||||
map<int32, float> reattribute_data = 3;
|
||||
}
|
@@ -703,4 +703,26 @@ public final class ConvertUtilsTest {
|
||||
/* taskRootPackageName= */ ""))
|
||||
.isEqualTo(packageName);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void decodeBatteryReattribute_returnExpectedResult() {
|
||||
final BatteryReattribute batteryReattribute =
|
||||
BatteryReattribute.newBuilder()
|
||||
.setTimestampStart(100L)
|
||||
.setTimestampEnd(200L)
|
||||
.putReattributeData(1001, 0.2f)
|
||||
.putReattributeData(2001, 0.8f)
|
||||
.build();
|
||||
|
||||
final BatteryReattribute decodeResult = ConvertUtils.decodeBatteryReattribute(
|
||||
ConvertUtils.encodeBatteryReattribute(batteryReattribute));
|
||||
|
||||
assertThat(decodeResult.getTimestampStart()).isEqualTo(100L);
|
||||
assertThat(decodeResult.getTimestampEnd()).isEqualTo(200L);
|
||||
final Map<Integer, Float> reattributeDataMap = decodeResult.getReattributeDataMap();
|
||||
// Verify the reattribute data in the map.
|
||||
assertThat(reattributeDataMap).hasSize(2);
|
||||
assertThat(reattributeDataMap.get(1001)).isEqualTo(0.2f);
|
||||
assertThat(reattributeDataMap.get(2001)).isEqualTo(0.8f);
|
||||
}
|
||||
}
|
||||
|
@@ -112,6 +112,7 @@ public final class DataProcessManagerTest {
|
||||
mContext,
|
||||
/* handler= */ null,
|
||||
mUserIdsSeries,
|
||||
/* isFromPeriodJob= */ false,
|
||||
/* rawStartTimestamp= */ 0L,
|
||||
/* lastFullChargeTimestamp= */ 0L,
|
||||
/* callbackFunction= */ null,
|
||||
@@ -258,6 +259,7 @@ public final class DataProcessManagerTest {
|
||||
mContext,
|
||||
/* handler= */ null,
|
||||
mUserIdsSeries,
|
||||
/* isFromPeriodJob= */ false,
|
||||
/* rawStartTimestamp= */ 2L,
|
||||
/* lastFullChargeTimestamp= */ 1L,
|
||||
/* callbackFunction= */ null,
|
||||
|
@@ -0,0 +1,122 @@
|
||||
/*
|
||||
* Copyright (C) 2024 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package com.android.settings.fuelgauge.batteryusage.bugreport;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import static org.mockito.Mockito.when;
|
||||
|
||||
import android.content.Context;
|
||||
|
||||
import androidx.test.core.app.ApplicationProvider;
|
||||
|
||||
import com.android.settings.fuelgauge.PowerUsageFeatureProvider;
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryReattribute;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryReattributeDao;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryReattributeEntity;
|
||||
import com.android.settings.fuelgauge.batteryusage.db.BatteryStateDatabase;
|
||||
import com.android.settings.testutils.BatteryTestUtils;
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
|
||||
import org.junit.After;
|
||||
import org.junit.Before;
|
||||
import org.junit.FixMethodOrder;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.junit.runners.MethodSorters;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
|
||||
import java.io.PrintWriter;
|
||||
import java.io.StringWriter;
|
||||
|
||||
@FixMethodOrder(MethodSorters.NAME_ASCENDING)
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public final class LogUtilsTest {
|
||||
|
||||
private StringWriter mTestStringWriter;
|
||||
private PrintWriter mTestPrintWriter;
|
||||
private Context mContext;
|
||||
private BatteryStateDatabase mDatabase;
|
||||
private BatteryReattributeDao mBatteryReattributeDao;
|
||||
private PowerUsageFeatureProvider mPowerUsageFeatureProvider;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
mContext = ApplicationProvider.getApplicationContext();
|
||||
mTestStringWriter = new StringWriter();
|
||||
mTestPrintWriter = new PrintWriter(mTestStringWriter);
|
||||
mDatabase = BatteryTestUtils.setUpBatteryStateDatabase(mContext);
|
||||
mBatteryReattributeDao = mDatabase.batteryReattributeDao();
|
||||
mPowerUsageFeatureProvider = FakeFeatureFactory.setupForTest().powerUsageFeatureProvider;
|
||||
when(mPowerUsageFeatureProvider.isBatteryUsageReattributeEnabled()).thenReturn(true);
|
||||
}
|
||||
|
||||
@After
|
||||
public void cleanUp() {
|
||||
mBatteryReattributeDao.clearAll();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void dumpBatteryReattributeDatabaseHist_noData_printExpectedResult() {
|
||||
LogUtils.dumpBatteryReattributeDatabaseHist(mBatteryReattributeDao, mTestPrintWriter);
|
||||
|
||||
assertThat(mTestStringWriter.toString())
|
||||
.contains("BatteryReattribute DatabaseHistory:");
|
||||
}
|
||||
|
||||
@Test
|
||||
public void dumpBatteryReattributeDatabaseHist_printExpectedResult() {
|
||||
final long currentTimeMillis = System.currentTimeMillis();
|
||||
// Insert the first testing data.
|
||||
final BatteryReattribute batteryReattribute1 =
|
||||
BatteryReattribute.newBuilder()
|
||||
.setTimestampStart(currentTimeMillis - 20000)
|
||||
.setTimestampEnd(currentTimeMillis - 10000)
|
||||
.putReattributeData(1001, 0.1f)
|
||||
.putReattributeData(1002, 0.99f)
|
||||
.build();
|
||||
mBatteryReattributeDao.insert(new BatteryReattributeEntity(batteryReattribute1));
|
||||
// Insert the second testing data.
|
||||
final BatteryReattribute batteryReattribute2 =
|
||||
BatteryReattribute.newBuilder()
|
||||
.setTimestampStart(currentTimeMillis - 40000)
|
||||
.setTimestampEnd(currentTimeMillis - 20000)
|
||||
.putReattributeData(1003, 1f)
|
||||
.build();
|
||||
mBatteryReattributeDao.insert(new BatteryReattributeEntity(batteryReattribute2));
|
||||
|
||||
LogUtils.dumpBatteryReattributeDatabaseHist(mBatteryReattributeDao, mTestPrintWriter);
|
||||
|
||||
final String result = mTestStringWriter.toString();
|
||||
assertThat(result).contains("BatteryReattribute DatabaseHistory:");
|
||||
assertThat(result).contains(batteryReattribute1.toString());
|
||||
assertThat(result).contains(batteryReattribute2.toString());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void dumpBatteryReattributeDatabaseHist_featureDisable_notPrintData() {
|
||||
mBatteryReattributeDao.insert(new BatteryReattributeEntity(
|
||||
BatteryReattribute.getDefaultInstance()));
|
||||
when(mPowerUsageFeatureProvider.isBatteryUsageReattributeEnabled()).thenReturn(false);
|
||||
|
||||
LogUtils.dumpBatteryReattributeDatabaseHist(mBatteryReattributeDao, mTestPrintWriter);
|
||||
|
||||
final String result = mTestStringWriter.toString();
|
||||
assertThat(result).contains("BatteryReattribute is disabled!");
|
||||
assertThat(result.contains("BatteryReattribute DatabaseHistory:")).isFalse();
|
||||
}
|
||||
}
|
@@ -0,0 +1,118 @@
|
||||
/*
|
||||
* Copyright (C) 2024 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package com.android.settings.fuelgauge.batteryusage.db;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import android.content.Context;
|
||||
import androidx.test.core.app.ApplicationProvider;
|
||||
|
||||
import com.android.settings.testutils.BatteryTestUtils;
|
||||
|
||||
import org.junit.After;
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
/** Tests for {@link BatteryReattributeDao}. */
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public final class BatteryReattributeDaoTest {
|
||||
|
||||
private Context mContext;
|
||||
private BatteryStateDatabase mDatabase;
|
||||
private BatteryReattributeDao mBatteryReattributeDao;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
mContext = ApplicationProvider.getApplicationContext();
|
||||
mDatabase = BatteryTestUtils.setUpBatteryStateDatabase(mContext);
|
||||
mBatteryReattributeDao = mDatabase.batteryReattributeDao();
|
||||
insert(100L, 200L, "reattributeData1");
|
||||
insert(300L, 400L, "reattributeData3");
|
||||
insert(200L, 300L, "reattributeData2");
|
||||
insert(400L, 500L, "reattributeData4");
|
||||
// Ensure there was data inserted into the database.
|
||||
assertThat(getAllEntityData()).isNotEmpty();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getAllAfter_returnExpectedEntityData() {
|
||||
final List<BatteryReattributeEntity> entityDataList =
|
||||
mBatteryReattributeDao.getAllAfter(/* timestampStart= */ 300L);
|
||||
|
||||
assertThat(entityDataList).hasSize(2);
|
||||
assertEntity(entityDataList.get(0), 400L, 500L, "reattributeData4");
|
||||
assertEntity(entityDataList.get(1), 300L, 400L, "reattributeData3");
|
||||
}
|
||||
|
||||
@Test
|
||||
public void clearAll_clearAllData() {
|
||||
mBatteryReattributeDao.clearAll();
|
||||
|
||||
assertThat(getAllEntityData()).isEmpty();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void clearAllBefore_clearAllExpectedData() {
|
||||
mBatteryReattributeDao.clearAllBefore(/* timestampStart= */ 300L);
|
||||
|
||||
final List<BatteryReattributeEntity> entityDataList = getAllEntityData();
|
||||
assertThat(entityDataList).hasSize(1);
|
||||
assertEntity(entityDataList.get(0), 400L, 500L, "reattributeData4");
|
||||
}
|
||||
|
||||
@Test
|
||||
public void clearAllAfter_clearAllExpectedData() {
|
||||
mBatteryReattributeDao.clearAllAfter(/* timestampStart= */ 300L);
|
||||
|
||||
final List<BatteryReattributeEntity> entityDataList = getAllEntityData();
|
||||
assertThat(entityDataList).hasSize(2);
|
||||
assertEntity(entityDataList.get(0), 200L, 300L, "reattributeData2");
|
||||
assertEntity(entityDataList.get(1), 100L, 200L, "reattributeData1");
|
||||
}
|
||||
|
||||
@Test
|
||||
public void insert_samePrimaryKeyEntityData_replaceIntoNewEntityData() {
|
||||
// Verify the original data before update.
|
||||
assertEntity(getAllEntityData().get(0), 400L, 500L, "reattributeData4");
|
||||
|
||||
insert(400L, 600L, "reattribute4Update");
|
||||
|
||||
// Verify the new update entity data.
|
||||
assertEntity(getAllEntityData().get(0), 400L, 600L, "reattribute4Update");
|
||||
}
|
||||
|
||||
private void insert(long timestampStart, long timestampEnd, String reattributeData) {
|
||||
mBatteryReattributeDao.insert(
|
||||
new BatteryReattributeEntity(
|
||||
timestampStart, timestampEnd, reattributeData));
|
||||
}
|
||||
|
||||
private List<BatteryReattributeEntity> getAllEntityData() {
|
||||
return mBatteryReattributeDao.getAllAfter(/* timestampStart= */ 0L);
|
||||
}
|
||||
|
||||
private static void assertEntity(BatteryReattributeEntity entity, long timestampStart,
|
||||
long timestampEnd, String reattributeData) {
|
||||
assertThat(entity.timestampStart).isEqualTo(timestampStart);
|
||||
assertThat(entity.timestampEnd).isEqualTo(timestampEnd);
|
||||
assertThat(entity.reattributeData).isEqualTo(reattributeData);
|
||||
}
|
||||
}
|
@@ -0,0 +1,54 @@
|
||||
/*
|
||||
* Copyright (C) 2024 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package com.android.settings.fuelgauge.batteryusage.db;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import com.android.settings.fuelgauge.batteryusage.BatteryReattribute;
|
||||
import com.android.settings.fuelgauge.batteryusage.ConvertUtils;
|
||||
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
|
||||
/** Tests for {@link BatteryReattributeEntity}. */
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public final class BatteryReattributeEntityTest {
|
||||
|
||||
@Test
|
||||
public void constructor_createExpectedData() {
|
||||
final BatteryReattribute batteryReattribute =
|
||||
BatteryReattribute.newBuilder()
|
||||
.setTimestampStart(100L)
|
||||
.setTimestampEnd(200L)
|
||||
.putReattributeData(1001, 0.2f)
|
||||
.putReattributeData(2001, 0.8f)
|
||||
.build();
|
||||
|
||||
final BatteryReattributeEntity batteryReattributeEntity =
|
||||
new BatteryReattributeEntity(batteryReattribute);
|
||||
|
||||
assertThat(batteryReattributeEntity.timestampStart)
|
||||
.isEqualTo(batteryReattribute.getTimestampStart());
|
||||
assertThat(batteryReattributeEntity.timestampEnd)
|
||||
.isEqualTo(batteryReattribute.getTimestampEnd());
|
||||
// Verify the BatteryReattribute data.
|
||||
final BatteryReattribute decodeResult =
|
||||
ConvertUtils.decodeBatteryReattribute(batteryReattributeEntity.reattributeData);
|
||||
assertThat(decodeResult).isEqualTo(batteryReattribute);
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user