diff options
author | 2024-11-20 21:12:54 +0000 | |
---|---|---|
committer | 2024-11-21 21:35:31 +0000 | |
commit | 5ac164984e99736a02978f2ae5c7ec78afcfb1f9 (patch) | |
tree | 1b2b58ea3361f0278f585ec0256456c8fced71ef /framework-s/java | |
parent | 51409ed0da3a587fb2d20b83fc65fc963648e380 (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