summaryrefslogtreecommitdiff
path: root/framework-s/java
diff options
context:
space:
mode:
author Scarlett Song <scarlettsong@google.com> 2024-11-20 21:12:54 +0000
committer Yi-an Chen <theianchen@google.com> 2024-11-21 21:35:31 +0000
commit5ac164984e99736a02978f2ae5c7ec78afcfb1f9 (patch)
tree1b2b58ea3361f0278f585ec0256456c8fced71ef /framework-s/java
parent51409ed0da3a587fb2d20b83fc65fc963648e380 (diff)
Add splitPermissionsSystemTest for READ_HEART_RATE
Before this CL, fail with assertion mismatch ``` [1/1] android.permission.cts.SplitPermissionsSystemTest#validateAndroidSystem: FAILED (29ms) STACKTRACE: value of: iterable.onlyElement() expected: android.permission.BODY_SENSORS_BACKGROUND but was : android.permission.health.READ_HEART_RATE at android.permission.cts.SplitPermissionsSystemTest.assertSplit(SplitPermissionsSystemTest.java:167) at android.permission.cts.SplitPermissionsSystemTest.validateAndroidSystem(SplitPermissionsSystemTest.java:153) ``` [Alternative considered] Why the other approach ag/30505597 is not recommended: 1. Accessing flag value directly in test runtime instead of @RequiresFlagsEnabled is against decisions in https://docs.google.com/document/d/1fsQG6rU6iB_NfmUz0F2WRJGrpYQjwfy0k81K70sDHxQ/edit?resourcekey=0-7gsaqK_uag0zCzyrvRxbrQ&tab=t.0 2. Runtime error flag not found Test 1. ABTD https://android-build.corp.google.com/builds/abtd/run/L92400030007911708 2. With this CL, locally modify ApiLevelUtil.isAtLeast(36) with .isAtLeast(35) so that it can run. ``` x86_64 CtsPermissionTestCases ----------------------------- android.permission.cts (3 Tests) [1/3] android.permission.cts.SplitPermissionsSystemTest#validateBodySensors_beforeGranularHealthPermissions_splitToBodySensorsBackground: ASSUMPTION_FAILED (130ms) [2/3] android.permission.cts.SplitPermissionsSystemTest#validateBodySensors_afterGranularHealthPermissions_splitToReadHeartRate: PASSED (26ms) [3/3] android.permission.cts.SplitPermissionsSystemTest#validateAndroidSystemExceptBodySensors: PASSED (26ms) ``` After change back to 36, also skip with assumption failed. LOW_COVERAGE_REASON=TEST_ONLY Bug: 364638912 Bug: 380299737 Flag: android.permission.flags.replace_body_sensor_permission_enabled Relnote: Granular health permission Change-Id: I8806d7ff8d9dcada7a348aba567e26a1516fcfef
Diffstat (limited to 'framework-s/java')
0 files changed, 0 insertions, 0 deletions