Merge "Revert "Fix InstallOverlayTests fail to install overlay"" (87c28df0) · Commits · e / os / android_frameworks_base

core/tests/overlaytests/host/Android.bp

+1 −1

Original line number	Diff line number	Diff line
		@@ -16,7 +16,7 @@ java_test_host {
		name: "OverlayHostTests",
		srcs: ["src/*/.java"],
		libs: ["tradefed"],
		test_suites: ["device-tests"],
		test_suites: ["general-tests"],
		target_required: [
		"OverlayHostTests_NonPlatformSignatureOverlay",
		"OverlayHostTests_PlatformSignatureStaticOverlay",

core/tests/overlaytests/host/src/com/android/server/om/hosttest/InstallOverlayTests.java

+8 −7

Original line number	Diff line number	Diff line
		@@ -78,9 +78,14 @@ public class InstallOverlayTests extends BaseHostJUnit4Test {
		}

		@Test
		public void installedIsStaticOverlayIsMutable() throws Exception {
		public void failToInstallPlatformSignedStaticOverlay() throws Exception {
		try {
		installPackage("OverlayHostTests_PlatformSignatureStaticOverlay.apk");
		assertTrue(isOverlayMutable(SIG_OVERLAY_PACKAGE_NAME));
		fail("installed a static overlay");
		} catch (Exception e) {
		// Expected.
		}
		assertFalse(overlayManagerContainsPackage(SIG_OVERLAY_PACKAGE_NAME));
		}

		@Test
		@@ -224,10 +229,6 @@ public class InstallOverlayTests extends BaseHostJUnit4Test {
		return shell("cmd overlay list").contains(pkg);
		}

		private boolean isOverlayMutable(String pkg) throws Exception {
		return shell("cmd overlay dump ismutable " + pkg).contains("true");
		}

		private String shell(final String cmd) throws Exception {
		return getDevice().executeShellCommand(cmd);
		}

core/tests/overlaytests/host/test-apps/SignatureOverlay/Android.mk

+3 −4

Original line number	Diff line number	Diff line
		@@ -20,7 +20,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_NonPlatformSignatureOverlay
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_bad
		include $(BUILD_PACKAGE)

		@@ -28,8 +28,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_PlatformSignatureStaticOverlay
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_CERTIFICATE := platform
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_MANIFEST_FILE := static/AndroidManifest.xml
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_static
		include $(BUILD_PACKAGE)
		@@ -38,7 +37,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_PlatformSignatureOverlay
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_CERTIFICATE := platform
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_v1
		LOCAL_AAPT_FLAGS += --version-code 1 --version-name v1

core/tests/overlaytests/host/test-apps/UpdateOverlay/Android.mk

+5 −5

Original line number	Diff line number	Diff line
		@@ -19,7 +19,7 @@ LOCAL_MODULE_TAGS := tests
		LOCAL_SRC_FILES := $(call all-java-files-under,src)
		LOCAL_PACKAGE_NAME := OverlayHostTests_UpdateOverlay
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_STATIC_JAVA_LIBRARIES := androidx.test.rules
		LOCAL_USE_AAPT2 := true
		LOCAL_AAPT_FLAGS := --no-resource-removal
		@@ -31,7 +31,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_FrameworkOverlayV1
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_CERTIFICATE := platform
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_v1
		LOCAL_AAPT_FLAGS += --version-code 1 --version-name v1
		@@ -43,7 +43,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_FrameworkOverlayV2
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_CERTIFICATE := platform
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_v2
		LOCAL_AAPT_FLAGS += --version-code 2 --version-name v2
		@@ -57,7 +57,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_AppOverlayV1
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_v1
		LOCAL_AAPT_FLAGS += --version-code 1 --version-name v1
		LOCAL_RESOURCE_DIR := $(LOCAL_PATH)/app/v1/res
		@@ -68,7 +68,7 @@ include $(CLEAR_VARS)
		LOCAL_MODULE_TAGS := tests
		LOCAL_PACKAGE_NAME := OverlayHostTests_AppOverlayV2
		LOCAL_SDK_VERSION := current
		LOCAL_COMPATIBILITY_SUITE := device-tests
		LOCAL_COMPATIBILITY_SUITE := general-tests
		LOCAL_AAPT_FLAGS := --custom-package $(my_package_prefix)_v2
		LOCAL_AAPT_FLAGS += --version-code 2 --version-name v2
		LOCAL_RESOURCE_DIR := $(LOCAL_PATH)/app/v2/res

services/tests/servicestests/src/com/android/server/om/OverlayManagerServiceImplTestsBase.java

+2 −1

Original line number	Diff line number	Diff line
		@@ -377,7 +377,8 @@ class OverlayManagerServiceImplTestsBase {
		return false;
		}
		final String key = createKey(overlayPackage.packageName, userId);
		return mIdmapFiles.add(key);
		mIdmapFiles.add(key);
		return true;
		}

		@Override