test_terminal: ensure redirections work reliably

[thirdparty/git.git] / t / lib-terminal.sh

diff --git a/t/lib-terminal.sh b/t/lib-terminal.sh
index 5e7ee9a5c7ea33a5911006d6a19de0f3cbe121b9..c383b57ed9d995f530004923962c45ab38c7fc8b 100644 (file)
--- a/t/lib-terminal.sh
+++ b/t/lib-terminal.sh
@@ -1,37 +1,19 @@
 #!/bin/sh
 
 test_expect_success 'set up terminal for tests' '
-       if test -t 1 && test -t 2
-       then
-               >have_tty
-       elif
+       if
                test_have_prereq PERL &&
                "$PERL_PATH" "$TEST_DIRECTORY"/test-terminal.perl \
                        sh -c "test -t 1 && test -t 2"
        then
-               >test_terminal_works
+               test_set_prereq TTY &&
+               test_terminal () {
+                       if ! test_declared_prereq TTY
+                       then
+                               echo >&4 "test_terminal: need to declare TTY prerequisite"
+                               return 127
+                       fi
+                       "$PERL_PATH" "$TEST_DIRECTORY"/test-terminal.perl "$@"
+               }
        fi
 '
-
-if test -e have_tty
-then
-       test_terminal_() { "$@"; }
-       test_set_prereq TTY
-elif test -e test_terminal_works
-then
-       test_terminal_() {
-               "$PERL_PATH" "$TEST_DIRECTORY"/test-terminal.perl "$@"
-       }
-       test_set_prereq TTY
-else
-       say "# no usable terminal, so skipping some tests"
-fi
-
-test_terminal () {
-       if ! test_declared_prereq TTY
-       then
-               echo >&2 'test_terminal: need to declare TTY prerequisite'
-               return 127
-       fi
-       test_terminal_ "$@"
-}