Remove setattr permission test for external files path.
Requests for these paths (Android/data) go directly to lower filesystem,
so the check was failing on some devices (e.g. crosshatch). we keep the
assertion for media paths, since these requests go through FuseDaemon.
BUG: 155239141
Test: atest ScopedStorageTest:android.scopedstorage.cts.host.ScopedStorageHostTest#testCantSetAttrOtherAppsFile
Change-Id: Id22affe5ec0841b148abb7262b6cf8455e70d360
diff --git a/hostsidetests/scopedstorage/src/android/scopedstorage/cts/ScopedStorageTest.java b/hostsidetests/scopedstorage/src/android/scopedstorage/cts/ScopedStorageTest.java
index d602fa9..2546433 100644
--- a/hostsidetests/scopedstorage/src/android/scopedstorage/cts/ScopedStorageTest.java
+++ b/hostsidetests/scopedstorage/src/android/scopedstorage/cts/ScopedStorageTest.java
@@ -2556,16 +2556,11 @@
*/
@Test
public void testCantSetAttrOtherAppsFile() throws Exception {
- // This path's permission is checked in FuseDaemon (directory/external files dir).
- final File externalFilesPath = new File(getExternalFilesDir(), VIDEO_FILE_NAME);
// This path's permission is checked in MediaProvider (directory/external media dir)
final File externalMediaPath = new File(getExternalMediaDir(), VIDEO_FILE_NAME);
try {
// Create the files
- if (!externalFilesPath.exists()) {
- assertThat(externalFilesPath.createNewFile()).isTrue();
- }
if (!externalMediaPath.exists()) {
assertThat(externalMediaPath.createNewFile()).isTrue();
}
@@ -2575,15 +2570,10 @@
// TEST_APP_A should not be able to setattr to other app's files.
assertWithMessage(
- "setattr on directory/external files path [%s]", externalFilesPath.getPath())
- .that(setAttrAs(TEST_APP_A, externalFilesPath.getPath()))
- .isFalse();
- assertWithMessage(
"setattr on directory/external media path [%s]", externalMediaPath.getPath())
.that(setAttrAs(TEST_APP_A, externalMediaPath.getPath()))
.isFalse();
} finally {
- externalFilesPath.delete();
externalMediaPath.delete();
uninstallAppNoThrow(TEST_APP_A);
}
