diff options
author | 2021-02-23 22:28:53 +0000 | |
---|---|---|
committer | 2022-07-12 20:34:31 +0000 | |
commit | 85585d9e19f12f0e43126979a50391902cbd24ef (patch) | |
tree | 68af437355e0d6c770e34587225f3b021c215814 | |
parent | 7cc1ac3771d446f96c539da760142d1525bea2c3 (diff) |
Add mainline presubmit to test mapping for permission.
Also adjust presubmit tests according to mts-permission-tests-list.
Bug: 181197796
Test: presubmit
Change-Id: I3a314a87d61e1fcca41318770ad0a406b1871ed5
15 files changed, 315 insertions, 3 deletions
diff --git a/PermissionController/TEST_MAPPING b/PermissionController/TEST_MAPPING index 0911e0d91..78e7e4a42 100644 --- a/PermissionController/TEST_MAPPING +++ b/PermissionController/TEST_MAPPING @@ -13,6 +13,24 @@ "file_patterns": ["res/xml/roles\\.xml"] } ], + "mainline-presubmit": [ + { + "name": "CtsRoleTestCases[com.google.android.permission.apex]", + "options": [ + // TODO(b/238677748): These two tests currently fails on R base image + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#openDefaultAppListThenIsNotDefaultAppInList" + }, + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#removeSmsRoleHolderThenPermissionIsRevoked" + }, + { + "exclude-annotation": "androidx.test.filters.FlakyTest" + } + ], + "file_patterns": ["res/xml/roles\\.xml"] + } + ], "imports": [ { "path": "vendor/xts/gts-tests/hostsidetests/permissioncontroller" diff --git a/PermissionController/src/com/android/permissioncontroller/permission/TEST_MAPPING b/PermissionController/src/com/android/permissioncontroller/permission/TEST_MAPPING index 6b89bc2f7..23ce155ed 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/TEST_MAPPING +++ b/PermissionController/src/com/android/permissioncontroller/permission/TEST_MAPPING @@ -5,6 +5,48 @@ "options": [ { "include-filter": "android.permission.cts.BackgroundPermissionsTest" + }, + { + "include-filter": "android.permission.cts.LocationAccessCheckTest" + }, + { + "include-filter": "android.permission.cts.NotificationListenerCheckTest" + }, + { + "include-filter": "android.permission.cts.OneTimePermissionTest" + }, + { + "include-filter": "android.permission.cts.PermissionControllerTest" + }, + { + "include-filter": "android.permission.cts.PlatformPermissionGroupMappingTest" + }, + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + }, + { + "name": "PermissionUiTestCases", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + }, + // TODO(b/238773220): These tests currently fails on R base image + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsGranted" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsRevoked" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenAppGetsInstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenDefinerGetsUninstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenUserGetsUninstalled" } ] }, @@ -13,6 +55,12 @@ "options": [ { "include-filter": "android.os.cts.AutoRevokeTest" + }, + { + "include-filter": "android.os.cts.AppHibernationIntegrationTest" + }, + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" } ] } @@ -26,5 +74,83 @@ } ] } + ], + "mainline-presubmit": [ + { + "name": "CtsPermissionTestCases[com.google.android.permission.apex]", + "options": [ + { + "include-filter": "android.permission.cts.BackgroundPermissionsTest" + }, + { + "include-filter": "android.permission.cts.LocationAccessCheckTest" + }, + { + "include-filter": "android.permission.cts.NotificationListenerCheckTest" + }, + { + "include-filter": "android.permission.cts.OneTimePermissionTest" + }, + { + "include-filter": "android.permission.cts.PermissionControllerTest" + }, + { + "include-filter": "android.permission.cts.PlatformPermissionGroupMappingTest" + }, + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + }, + { + "name": "CtsPermission3TestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + }, + { + "name": "PermissionUiTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + }, + // TODO(b/238773220): These tests currently fails on R base image + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsGranted" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsRevoked" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenAppGetsInstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenDefinerGetsUninstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenUserGetsUninstalled" + } + ] + }, + { + "name": "CtsOsTestCases[com.google.android.permission.apex]", + "options": [ + { + "include-filter": "android.os.cts.AutoRevokeTest" + }, + { + "include-filter": "android.os.cts.AppHibernationIntegrationTest" + }, + // TODO(b/238677038): This test currently fails on R base image + { + "exclude-filter": "android.os.cts.AutoRevokeTest#testUnusedApp_uninstallApp" + }, + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } ] } diff --git a/PermissionController/src/com/android/permissioncontroller/privacysources/TEST_MAPPING b/PermissionController/src/com/android/permissioncontroller/privacysources/TEST_MAPPING index c58b45dc2..a36d3534f 100644 --- a/PermissionController/src/com/android/permissioncontroller/privacysources/TEST_MAPPING +++ b/PermissionController/src/com/android/permissioncontroller/privacysources/TEST_MAPPING @@ -16,6 +16,16 @@ "name": "CtsSafetyCenterTestCases" } ], + "mainline-presubmit": [ + { + "name": "CtsSafetyCenterTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } + ], "postsubmit": [ { "name": "CtsPermissionTestCases", diff --git a/PermissionController/src/com/android/permissioncontroller/role/TEST_MAPPING b/PermissionController/src/com/android/permissioncontroller/role/TEST_MAPPING index 43d15a49e..d7718a2f2 100644 --- a/PermissionController/src/com/android/permissioncontroller/role/TEST_MAPPING +++ b/PermissionController/src/com/android/permissioncontroller/role/TEST_MAPPING @@ -8,5 +8,22 @@ } ] } + ], + "mainline-presubmit": [ + { + "name": "CtsRoleTestCases[com.google.android.permission.apex]", + "options": [ + // TODO(b/238677748): These two tests currently fails on R base image + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#openDefaultAppListThenIsNotDefaultAppInList" + }, + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#removeSmsRoleHolderThenPermissionIsRevoked" + }, + { + "exclude-annotation": "androidx.test.filters.FlakyTest" + } + ] + } ] } diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/TEST_MAPPING b/PermissionController/src/com/android/permissioncontroller/safetycenter/TEST_MAPPING index 3985e563c..e8c210a5d 100644 --- a/PermissionController/src/com/android/permissioncontroller/safetycenter/TEST_MAPPING +++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/TEST_MAPPING @@ -3,5 +3,15 @@ { "name": "CtsSafetyCenterTestCases" } + ], + "mainline-presubmit": [ + { + "name": "CtsSafetyCenterTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } ] } diff --git a/PermissionController/tests/permissionui/Android.bp b/PermissionController/tests/permissionui/Android.bp index 6d5c91043..1cfeb423c 100644 --- a/PermissionController/tests/permissionui/Android.bp +++ b/PermissionController/tests/permissionui/Android.bp @@ -52,6 +52,7 @@ android_test { test_suites: [ "device-tests", + "general-tests", "mts-permission", ], diff --git a/PermissionController/tests/permissionui/AndroidTest.xml b/PermissionController/tests/permissionui/AndroidTest.xml index 34a56286e..946f97e30 100644 --- a/PermissionController/tests/permissionui/AndroidTest.xml +++ b/PermissionController/tests/permissionui/AndroidTest.xml @@ -21,6 +21,7 @@ <option name="test-suite-tag" value="apct-instrumentation" /> <option name="test-tag" value="PermissionUiTestCases" /> <object type="module_controller" class="com.android.tradefed.testtype.suite.module.Sdk30ModuleController" /> + <option name="config-descriptor:metadata" key="mainline-param" value="com.google.android.permission.apex" /> <!-- Install test --> <target_preparer class="com.android.tradefed.targetprep.TestAppInstallSetup"> diff --git a/PermissionController/tests/permissionui/TEST_MAPPING b/PermissionController/tests/permissionui/TEST_MAPPING new file mode 100644 index 000000000..6a71e5ec0 --- /dev/null +++ b/PermissionController/tests/permissionui/TEST_MAPPING @@ -0,0 +1,54 @@ +{ + "presubmit": [ + { + "name": "PermissionUiTestCases", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + }, + // TODO(b/238773220): These tests currently fails on R base image + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsGranted" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsRevoked" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenAppGetsInstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenDefinerGetsUninstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenUserGetsUninstalled" + } + ] + } + ], + "mainline-presubmit": [ + { + "name": "PermissionUiTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + }, + // TODO(b/238773220): These tests currently fails on R base image + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsGranted" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageCustomPermissionsFragmentTest#groupSummaryGetsUpdatedWhenPermissionGetsRevoked" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenAppGetsInstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenDefinerGetsUninstalled" + }, + { + "exclude-filter": "com.android.permissioncontroller.permissionui.ui.handheld.ManageStandardPermissionsFragmentTest#additionalPermissionSummaryGetUpdateWhenUserGetsUninstalled" + } + ] + } + ] +} diff --git a/SafetyCenter/TEST_MAPPING b/SafetyCenter/TEST_MAPPING index 3985e563c..e8c210a5d 100644 --- a/SafetyCenter/TEST_MAPPING +++ b/SafetyCenter/TEST_MAPPING @@ -3,5 +3,15 @@ { "name": "CtsSafetyCenterTestCases" } + ], + "mainline-presubmit": [ + { + "name": "CtsSafetyCenterTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } ] } diff --git a/framework-s/java/android/app/role/TEST_MAPPING b/framework-s/java/android/app/role/TEST_MAPPING index f8f140dd7..ce53dca05 100644 --- a/framework-s/java/android/app/role/TEST_MAPPING +++ b/framework-s/java/android/app/role/TEST_MAPPING @@ -4,7 +4,24 @@ "name": "CtsRoleTestCases", "options": [ { - "include-filter": "android.app.role.cts.RoleManagerTest" + "exclude-annotation": "androidx.test.filters.FlakyTest" + } + ] + } + ], + "mainline-presubmit": [ + { + "name": "CtsRoleTestCases[com.google.android.permission.apex]", + "options": [ + // TODO(b/238677748): These two tests currently fails on R base image + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#openDefaultAppListThenIsNotDefaultAppInList" + }, + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#removeSmsRoleHolderThenPermissionIsRevoked" + }, + { + "exclude-annotation": "androidx.test.filters.FlakyTest" } ] } diff --git a/framework-s/java/android/safetycenter/TEST_MAPPING b/framework-s/java/android/safetycenter/TEST_MAPPING index 3985e563c..e8c210a5d 100644 --- a/framework-s/java/android/safetycenter/TEST_MAPPING +++ b/framework-s/java/android/safetycenter/TEST_MAPPING @@ -3,5 +3,15 @@ { "name": "CtsSafetyCenterTestCases" } + ], + "mainline-presubmit": [ + { + "name": "CtsSafetyCenterTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } ] } diff --git a/service/java/com/android/role/TEST_MAPPING b/service/java/com/android/role/TEST_MAPPING index 0d7bc1476..15173a9da 100644 --- a/service/java/com/android/role/TEST_MAPPING +++ b/service/java/com/android/role/TEST_MAPPING @@ -1,10 +1,10 @@ { "presubmit": [ { - "name": "CtsStatsdHostTestCases", + "name": "CtsAppSecurityHostTestCases", "options": [ { - "include-filter": "android.cts.statsd.atom.UidAtomTests#testRoleHolder" + "include-filter": "android.appsecurity.cts.StatsdAppSecurityAtomTest#testRoleHolder" } ] }, @@ -16,5 +16,22 @@ } ] } + ], + "mainline-presubmit": [ + { + "name": "CtsRoleTestCases[com.google.android.permission.apex]", + "options": [ + // TODO(b/238677748): These two tests currently fails on R base image + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#openDefaultAppListThenIsNotDefaultAppInList" + }, + { + "exclude-filter": "android.app.role.cts.RoleManagerTest#removeSmsRoleHolderThenPermissionIsRevoked" + }, + { + "exclude-annotation": "androidx.test.filters.FlakyTest" + } + ] + } ] } diff --git a/service/java/com/android/safetycenter/TEST_MAPPING b/service/java/com/android/safetycenter/TEST_MAPPING index 3985e563c..e8c210a5d 100644 --- a/service/java/com/android/safetycenter/TEST_MAPPING +++ b/service/java/com/android/safetycenter/TEST_MAPPING @@ -3,5 +3,15 @@ { "name": "CtsSafetyCenterTestCases" } + ], + "mainline-presubmit": [ + { + "name": "CtsSafetyCenterTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } ] } diff --git a/tests/cts/safetycenter/AndroidTest.xml b/tests/cts/safetycenter/AndroidTest.xml index 2e10bad48..7b614982c 100644 --- a/tests/cts/safetycenter/AndroidTest.xml +++ b/tests/cts/safetycenter/AndroidTest.xml @@ -28,6 +28,7 @@ <!-- Multi-user code is tested separately using Bedstead. See SafetyCenterManagedDeviceTest. --> <option name="config-descriptor:metadata" key="parameter" value="not_secondary_user"/> + <option name="config-descriptor:metadata" key="mainline-param" value="com.google.android.permission.apex" /> <option name="test-suite-tag" value="cts"/> diff --git a/tests/cts/safetycenter/TEST_MAPPING b/tests/cts/safetycenter/TEST_MAPPING index 3985e563c..e8c210a5d 100644 --- a/tests/cts/safetycenter/TEST_MAPPING +++ b/tests/cts/safetycenter/TEST_MAPPING @@ -3,5 +3,15 @@ { "name": "CtsSafetyCenterTestCases" } + ], + "mainline-presubmit": [ + { + "name": "CtsSafetyCenterTestCases[com.google.android.permission.apex]", + "options": [ + { + "exclude-annotation": "android.platform.test.annotations.FlakyTest" + } + ] + } ] } |