summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Michael Wright <michaelwr@google.com> 2021-06-21 10:20:24 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2021-06-21 10:20:24 +0000
commit03a3eb4b3dbd5bc6f65846ff50939c53c96d3024 (patch)
treeae76202bc3f33de7c9afd083ee91949ce7fe4a49
parent2d6933bbae310ed919ce6dfa2ce1c84b3eb60899 (diff)
parent8d16e1b7de9c9faf1b11045d130b6df2a8203ca1 (diff)
Merge "Fix failed tests for ApplicationPackageManagerTest"
-rw-r--r--core/tests/coretests/src/android/app/ApplicationPackageManagerTest.java14
1 files changed, 12 insertions, 2 deletions
diff --git a/core/tests/coretests/src/android/app/ApplicationPackageManagerTest.java b/core/tests/coretests/src/android/app/ApplicationPackageManagerTest.java
index 95da532045ac..d72a400c0078 100644
--- a/core/tests/coretests/src/android/app/ApplicationPackageManagerTest.java
+++ b/core/tests/coretests/src/android/app/ApplicationPackageManagerTest.java
@@ -26,8 +26,11 @@ import android.content.pm.PackageInfo;
import android.os.storage.StorageManager;
import android.os.storage.VolumeInfo;
+import androidx.annotation.NonNull;
import androidx.test.filters.LargeTest;
+import com.android.internal.annotations.VisibleForTesting;
+
import junit.framework.TestCase;
import org.mockito.Mockito;
@@ -110,6 +113,13 @@ public class ApplicationPackageManagerTest extends TestCase {
public boolean isAllow3rdPartyOnInternal(Context context) {
return mAllow3rdPartyOnInternal;
}
+
+ @Override
+ @VisibleForTesting
+ protected @NonNull List<VolumeInfo> getPackageCandidateVolumes(ApplicationInfo app,
+ StorageManager storageManager, IPackageManager pm) {
+ return super.getPackageCandidateVolumes(app, storageManager, pm);
+ }
}
private StorageManager getMockedStorageManager() {
@@ -223,7 +233,7 @@ public class ApplicationPackageManagerTest extends TestCase {
appInfo.flags = 0;
appInfo.volumeUuid = sInternalVolUuid;
- Mockito.when(pm.isPackageDeviceAdminOnAnyUser(Mockito.anyString())).thenReturn(false);
+ Mockito.when(pm.isPackageDeviceAdminOnAnyUser(appInfo.packageName)).thenReturn(false);
appPkgMgr.setAllow3rdPartyOnInternal(true);
List<VolumeInfo> candidates = appPkgMgr.getPackageCandidateVolumes(
appInfo, storageManager, pm);
@@ -231,7 +241,7 @@ public class ApplicationPackageManagerTest extends TestCase {
appInfo.volumeUuid = sInternalVolUuid;
appPkgMgr.setAllow3rdPartyOnInternal(true);
- Mockito.when(pm.isPackageDeviceAdminOnAnyUser(Mockito.anyString())).thenReturn(true);
+ Mockito.when(pm.isPackageDeviceAdminOnAnyUser(appInfo.packageName)).thenReturn(true);
candidates = appPkgMgr.getPackageCandidateVolumes(appInfo, storageManager, pm);
verifyReturnedVolumes(candidates, sInternalVol);