commit 618bf94d723c07ade5cc3bf85969eccb912a9590
author Svetoslav <svetoslavganov@google.com> Thu Apr 16 17:05:49 2015 -0700
committer Svetoslav Ganov <svetoslavganov@google.com> Sat Apr 18 00:17:40 2015 +0000
tree 39fe7914ff9ec69e94f47119a306b4068f3d9fbc
parent 6fcd30df206301049ea15729d534d5c7e108c1e4

Remove unnecessary WRITE_SMS permission - cts

Change-Id: I8c08414eb8ff72a191a568d912e282e1617449ed

tests/tests/content/src/android/content/cts/ContentProviderTest.java

diff --git a/tests/tests/content/src/android/content/cts/ContentProviderTest.java b/tests/tests/content/src/android/content/cts/ContentProviderTest.java
index 0b4d9aa..5fee379 100644
--- a/tests/tests/content/src/android/content/cts/ContentProviderTest.java
+++ b/tests/tests/content/src/android/content/cts/ContentProviderTest.java
@@ -71,7 +71,7 @@
 
         ProviderInfo info1 = new ProviderInfo();
         info1.readPermission = "android.permission.READ_SMS";
-        info1.writePermission = "android.permission.WRITE_SMS";
+        info1.writePermission = null; // Guarded by an app op not a permission.
         mockContentProvider.attachInfo(getContext(), info1);
         assertSame(getContext(), mockContentProvider.getContext());
         assertEquals(info1.readPermission, mockContentProvider.getReadPermission());
@@ -156,10 +156,6 @@
         mockContentProvider.setWritePermissionWrapper(expected);
         assertEquals(expected, mockContentProvider.getWritePermission());
 
-        expected = "android.permission.WRITE_SMS";
-        mockContentProvider.setWritePermissionWrapper(expected);
-        assertEquals(expected, mockContentProvider.getWritePermission());
-
         mockContentProvider.setWritePermissionWrapper(null);
         assertNull(mockContentProvider.getWritePermission());
     }