Merge "Remove READ_DEVICE_CONFIG CTS tests." into android14-tests-dev

Author: Ted Bauer

tests/tests/deviceconfig/src/android/deviceconfig/cts/DeviceConfigApiPermissionTests.java

@@ -68,8 +68,8 @@ public void dropShellPermissionIdentityAfterTest() {
                 .dropShellPermissionIdentity();
     }
     /**
-     * Checks that when application does not have READ_DEVICE_CONFIG or WRITE_DEVICE_CONFIG
-     * permissions it cannot access any of DeviceConfig API methods
+     * Checks that when application does not have WRITE_DEVICE_CONFIG
+     * permissions it cannot access any write DeviceConfig API methods
      * @throws Exception
      */
     @Test
@@ -81,10 +81,6 @@ public void testDeviceConfigWithoutPermissions() {
         trySetPropertiesWithoutWritePermission(violations);
         tryDeletePropertyWithoutWritePermission(violations);
 
-        // getters without read permission
-        tryGetPropertyWithoutReadPermission(violations);
-        tryGetPropertiesWithoutReadPermission(violations);
-
         // Bail if we found any violations
         if (violations.length() > 0) {
             fail(violations.toString());
@@ -107,10 +103,6 @@ public void testDeviceConfigWithWritePermission() {
         trySetPropertyWithWritePermission(violations);
         trySetPropertiesWithWritePermission(violations);
 
-        // getters without read permission
-        tryGetPropertyWithoutReadPermission(violations);
-        tryGetPropertiesWithoutReadPermission(violations);
-
         // Bail if we found any violations
         if (violations.length() > 0) {
             fail(violations.toString());
@@ -304,24 +296,6 @@ private void tryDeletePropertyWithoutWritePermission(StringBuilder violations) {
         }
     }
 
-    private void tryGetPropertyWithoutReadPermission(StringBuilder violations) {
-        try {
-            DeviceConfig.getProperty(NAMESPACE, KEY);
-            violations.append("DeviceConfig.getProperty() must not be accessible without "
-                    + "READ_DEVICE_CONFIG permission.\n");
-        } catch (SecurityException e) {
-        }
-    }
-
-    private void tryGetPropertiesWithoutReadPermission(StringBuilder violations) {
-        try {
-            DeviceConfig.getProperties(NAMESPACE2);
-            violations.append("DeviceConfig.getProperties() must not be accessible without "
-                    + "READ_DEVICE_CONFIG permission.\n");
-        } catch (SecurityException e) {
-        }
-    }
-
     private void trySetPropertyWithWritePermission(StringBuilder violations) {
         try {
             DeviceConfig.setProperty(NAMESPACE, KEY, VALUE, /*makeDefault=*/ false);