From c9cb8e5bcbfca744d5aa57c0ef89e65fa87948b1 Mon Sep 17 00:00:00 2001 From: Nikhil Kumar Date: Wed, 22 Feb 2023 11:14:40 +0000 Subject: [PATCH] Removed isWorkProfile_withSystemUser_returnFalse test Only the UM.isManagedProfile property should be used to determine whether a user is a managed prfile. There is no need to perform any additional check on top of UM.isManagedProfile to make sure the user is a managed profile. After removing the additional check as part of ag/21512484 DataBaseUtilsTest.isWorkProfile_withSystemUser_returnFalse test is redundant. Test: atest DataBaseUtilsTest -c Bug: 270283369 Change-Id: Ic4fd7eef2ac24786e2a59db7dabc9daf4a7ce028 --- .../fuelgauge/batteryusage/DatabaseUtilsTest.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/DatabaseUtilsTest.java b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/DatabaseUtilsTest.java index 078aafa9973..722f869bd7f 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/DatabaseUtilsTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/DatabaseUtilsTest.java @@ -87,14 +87,6 @@ public final class DatabaseUtilsTest { assertThat(DatabaseUtils.isWorkProfile(mContext)).isTrue(); } - @Test - public void isWorkProfile_withSystemUser_returnFalse() { - BatteryTestUtils.setWorkProfile(mContext); - Shadows.shadowOf(mContext.getSystemService(UserManager.class)).setIsSystemUser(true); - - assertThat(DatabaseUtils.isWorkProfile(mContext)).isFalse(); - } - @Test public void sendAppUsageEventData_returnsExpectedList() { // Configures the testing AppUsageEvent data.