Fix tests for getfattr/setfattr

The test system may have extended attributes set by default (for instance, security.selinux on Android). Use the -n option to limit the output.
zonurofai83 · Feb 20, 2024 · 2c96060 · 2c96060
1 parent 43c9ee3
commit 2c96060
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/tests/getfattr.test b/tests/getfattr.test
@@ -8,18 +8,14 @@ mkdir attrs
 touch attrs/file
 setfattr -n user.empty attrs/file
 setfattr -n user.data -v hello attrs/file
-setfattr -n user.more -v world attrs/file
 
-testing "" "getfattr attrs/file" \
-    "# file: attrs/file\nuser.data\nuser.empty\nuser.more\n\n" "" ""
-testing "-d" "getfattr -d attrs/file" \
-    "# file: attrs/file\nuser.data=\"hello\"\nuser.empty\nuser.more=\"world\"\n\n" "" ""
+# The filesystem may include some extended attributes by default (for
+# instance, security.selinux). Use the -n option to limit the output.
+
 testing "-n" "getfattr -n user.empty attrs/file" \
     "# file: attrs/file\nuser.empty\n\n" "" ""
 testing "-d -n" "getfattr -d -n user.data attrs/file" \
     "# file: attrs/file\nuser.data=\"hello\"\n\n" "" ""
-testing "--only-values" "getfattr --only-values attrs/file" \
-    "helloworld" "" ""
 testing "--only-values -n" "getfattr --only-values -n user.data attrs/file" \
     "hello" "" ""
 

diff --git a/tests/setfattr.test b/tests/setfattr.test
@@ -10,12 +10,15 @@ setfattr -n user.empty attrs/file
 setfattr -n user.data -v hello attrs/file
 setfattr -n user.delete-me -v hello attrs/file
 
+# The filesystem may include some extended attributes by default (for
+# instance, security.selinux). Use the -n option to limit the output.
+
 testing "-x" \
-    "setfattr -x user.delete-me attrs/file && getfattr attrs/file" \
-    "# file: attrs/file\nuser.data\nuser.empty\n\n" "" ""
-testing "-n" "setfattr -n user.new attrs/file && getfattr -d attrs/file" \
-    "# file: attrs/file\nuser.data=\"hello\"\nuser.empty\nuser.new\n\n" "" ""
-testing "-n -v" "setfattr -n user.new -v data attrs/file && getfattr -d attrs/file" \
-    "# file: attrs/file\nuser.data=\"hello\"\nuser.empty\nuser.new=\"data\"\n\n" "" ""
+    "setfattr -x user.delete-me attrs/file && getfattr -n user.delete-me attrs/file" \
+    "# file: attrs/file\n\n" "" ""
+testing "-n" "setfattr -n user.new attrs/file && getfattr -n user.new -d attrs/file" \
+    "# file: attrs/file\nuser.new\n\n" "" ""
+testing "-n -v" "setfattr -n user.new -v data attrs/file && getfattr -n user.new -d attrs/file" \
+    "# file: attrs/file\nuser.new=\"data\"\n\n" "" ""
 
 rm -rf attrs