coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests: skip some parts of tests/misc/tee.sh if run as root


From: Pádraig Brady
Subject: Re: [PATCH] tests: skip some parts of tests/misc/tee.sh if run as root
Date: Tue, 21 Mar 2023 16:16:33 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Thunderbird/109.0

On 21/03/2023 12:54, Nick Alcock wrote:
Similarly to the fix to tests/rmdir/ignore.sh in c0e5f8c59,
tee should not be expected to fail when run with read-only outputs
when run as root.

* tests/rmdir/ignore.sh: Add uid_is_privileged_ guard around test for
read-only outputs.
---
  tests/misc/tee.sh | 8 +++++---
  1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/tests/misc/tee.sh b/tests/misc/tee.sh
index e01c27ecc..9f5eace8a 100755
--- a/tests/misc/tee.sh
+++ b/tests/misc/tee.sh
@@ -79,9 +79,11 @@ test -f tee.exited || fail=1 ;;
  esac
# Test with unwriteable files
-touch file.ro || framework_failure_
-chmod a-w file.ro || framework_failure_
-returns_ 1 tee -p </dev/null file.ro || fail=1
+if ! uid_is_privileged_; then  # root does not get EPERM.
+    touch file.ro || framework_failure_
+    chmod a-w file.ro || framework_failure_
+    returns_ 1 tee -p </dev/null file.ro || fail=1
+fi

Applied.

I'll update release procedure to do a test run as root
(which is unusual / ill-advised TBH).

thanks,
Pádraig




reply via email to

[Prev in Thread] Current Thread [Next in Thread]