diff options
author | Jim Meyering <meyering@redhat.com> | 2008-02-11 20:54:38 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2008-02-11 21:44:54 +0100 |
commit | 6ed90c719e06af86c3515921fab8d07a4c1156bd (patch) | |
tree | d1af1864f98423ccd6a60c89e990e1cb1158de91 | |
parent | 5b954a875681f39d0b51d31475034ef93bd61251 (diff) | |
download | coreutils-6ed90c719e06af86c3515921fab8d07a4c1156bd.tar.xz |
tests: Migrate sourced strace file into a function, require_strace_.
* tests/strace: Remove file.
* tests/test-lib.sh (require_strace_): New function.
* tests/mv/atomic: Use require_strace_, rather than ". strace".
* tests/ls/stat-free-symlinks: Likewise.
Peter Fales reported that stat-free-symlinks failed without strace.
-rwxr-xr-x | tests/ls/stat-free-symlinks | 4 | ||||
-rwxr-xr-x | tests/mv/atomic | 4 | ||||
-rw-r--r-- | tests/strace | 22 | ||||
-rw-r--r-- | tests/test-lib.sh | 10 |
4 files changed, 14 insertions, 26 deletions
diff --git a/tests/ls/stat-free-symlinks b/tests/ls/stat-free-symlinks index 7b36ba601..07fb38e31 100755 --- a/tests/ls/stat-free-symlinks +++ b/tests/ls/stat-free-symlinks @@ -1,7 +1,7 @@ #!/bin/sh # ensure that ls does not stat a symlink in an unusual case -# Copyright (C) 2007 Free Software Foundation, Inc. +# Copyright (C) 2007-2008 Free Software Foundation, Inc. # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -21,8 +21,8 @@ if test "$VERBOSE" = yes; then ls --version fi -. $srcdir/../strace . $srcdir/../test-lib.sh +require_strace_ touch x || framework_failure chmod a+x x || framework_failure diff --git a/tests/mv/atomic b/tests/mv/atomic index ef30398e7..38e98a5a2 100755 --- a/tests/mv/atomic +++ b/tests/mv/atomic @@ -1,7 +1,7 @@ #!/bin/sh # ensure that mv doesn't first unlink its destination in one particular case -# Copyright (C) 2006-2007 Free Software Foundation, Inc. +# Copyright (C) 2006-2008 Free Software Foundation, Inc. # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -21,8 +21,8 @@ if test "$VERBOSE" = yes; then mv --version fi -. $srcdir/../strace . $srcdir/../test-lib.sh +require_strace_ # Before the fix, mv would unnecessarily unlink the destination symlink: # $ rm -rf s[12]; ln -s / s1; ln -s /tmp s2; strace -qe unlink /bin/mv -T s1 s2 diff --git a/tests/strace b/tests/strace deleted file mode 100644 index bc04fb30a..000000000 --- a/tests/strace +++ /dev/null @@ -1,22 +0,0 @@ -# Skip the current test if strace is not available or doesn't work. - -# Copyright (C) 2007 Free Software Foundation, Inc. - -# This program is free software: you can redistribute it and/or modify -# it under the terms of the GNU General Public License as published by -# the Free Software Foundation, either version 3 of the License, or -# (at your option) any later version. - -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. - -# You should have received a copy of the GNU General Public License -# along with this program. If not, see <http://www.gnu.org/licenses/>. - -strace -V < /dev/null > /dev/null 2>&1 || - skip_test_ 'no strace program' - -strace -qe unlink echo > /dev/null 2>&1 || - skip_test_ 'strace does not work' diff --git a/tests/test-lib.sh b/tests/test-lib.sh index b273f1a81..a473dfb31 100644 --- a/tests/test-lib.sh +++ b/tests/test-lib.sh @@ -39,6 +39,16 @@ require_ulimit_() && skip_test_ "this shell lacks ulimit support" } +# Skip the current test if strace is not available or doesn't work. +require_strace_() +{ + strace -V < /dev/null > /dev/null 2>&1 || + skip_test_ 'no strace program' + + strace -qe unlink echo > /dev/null 2>&1 || + skip_test_ 'strace does not work' +} + require_built_() { skip_=no |