From 5d6b8729ed7335c702022f4f3176573f4cf2cf5a Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Sun, 2 Oct 2016 17:39:31 +0200 Subject: hush testsuite: add many tests from ash testsuite Signed-off-by: Denys Vlasenko --- .../ash-quoting/dollar_repl_slash_bash1.tests | 2 +- shell/hush_test/hush-invert/invert.right | 10 ++++++ shell/hush_test/hush-invert/invert.tests | 19 ++++++++++ shell/hush_test/hush-misc/heredoc4.right | 1 + shell/hush_test/hush-misc/heredoc4.tests | 3 ++ shell/hush_test/hush-misc/last_amp.right | 2 ++ shell/hush_test/hush-misc/last_amp.tests | 8 +++++ shell/hush_test/hush-misc/local1.right | 4 +++ shell/hush_test/hush-misc/local1.tests | 11 ++++++ shell/hush_test/hush-misc/nulltick1.right | 3 ++ shell/hush_test/hush-misc/nulltick1.tests | 3 ++ .../hush-quoting/dollar_repl_slash_bash1.right | 10 ++++++ .../hush-quoting/dollar_repl_slash_bash1.tests | 21 +++++++++++ shell/hush_test/hush-read/read_r.tests | 6 ++-- shell/hush_test/hush-signals/reap1.right | 1 + shell/hush_test/hush-signals/reap1.tests | 14 ++++++++ shell/hush_test/hush-signals/sigint1.right | 1 + shell/hush_test/hush-signals/sigint1.tests | 41 ++++++++++++++++++++++ shell/hush_test/hush-signals/signal2.right | 3 ++ shell/hush_test/hush-signals/signal2.tests | 18 ++++++++++ shell/hush_test/hush-signals/signal3.right | 4 +++ shell/hush_test/hush-signals/signal3.tests | 17 +++++++++ shell/hush_test/hush-signals/signal5.right | 12 +++++++ shell/hush_test/hush-signals/signal5.tests | 14 ++++++++ shell/hush_test/hush-signals/signal6.right | 2 ++ shell/hush_test/hush-signals/signal6.tests | 2 ++ shell/hush_test/hush-signals/sigquit_exec.right | 2 ++ shell/hush_test/hush-signals/sigquit_exec.tests | 4 +++ .../hush-standalone/noexec_gets_no_env.right | 4 +++ .../hush-standalone/noexec_gets_no_env.tests | 5 +++ .../hush-standalone/nofork_trashes_getopt.right | 1 + .../hush-standalone/nofork_trashes_getopt.tests | 6 ++++ .../hush-standalone/var_standalone1.right | 1 + .../hush-standalone/var_standalone1.tests | 2 ++ 34 files changed, 254 insertions(+), 3 deletions(-) create mode 100644 shell/hush_test/hush-invert/invert.right create mode 100755 shell/hush_test/hush-invert/invert.tests create mode 100644 shell/hush_test/hush-misc/heredoc4.right create mode 100755 shell/hush_test/hush-misc/heredoc4.tests create mode 100644 shell/hush_test/hush-misc/last_amp.right create mode 100755 shell/hush_test/hush-misc/last_amp.tests create mode 100644 shell/hush_test/hush-misc/local1.right create mode 100755 shell/hush_test/hush-misc/local1.tests create mode 100644 shell/hush_test/hush-misc/nulltick1.right create mode 100755 shell/hush_test/hush-misc/nulltick1.tests create mode 100644 shell/hush_test/hush-quoting/dollar_repl_slash_bash1.right create mode 100755 shell/hush_test/hush-quoting/dollar_repl_slash_bash1.tests create mode 100644 shell/hush_test/hush-signals/reap1.right create mode 100755 shell/hush_test/hush-signals/reap1.tests create mode 100644 shell/hush_test/hush-signals/sigint1.right create mode 100755 shell/hush_test/hush-signals/sigint1.tests create mode 100644 shell/hush_test/hush-signals/signal2.right create mode 100755 shell/hush_test/hush-signals/signal2.tests create mode 100644 shell/hush_test/hush-signals/signal3.right create mode 100755 shell/hush_test/hush-signals/signal3.tests create mode 100644 shell/hush_test/hush-signals/signal5.right create mode 100755 shell/hush_test/hush-signals/signal5.tests create mode 100644 shell/hush_test/hush-signals/signal6.right create mode 100755 shell/hush_test/hush-signals/signal6.tests create mode 100644 shell/hush_test/hush-signals/sigquit_exec.right create mode 100755 shell/hush_test/hush-signals/sigquit_exec.tests create mode 100644 shell/hush_test/hush-standalone/noexec_gets_no_env.right create mode 100755 shell/hush_test/hush-standalone/noexec_gets_no_env.tests create mode 100644 shell/hush_test/hush-standalone/nofork_trashes_getopt.right create mode 100755 shell/hush_test/hush-standalone/nofork_trashes_getopt.tests create mode 100644 shell/hush_test/hush-standalone/var_standalone1.right create mode 100755 shell/hush_test/hush-standalone/var_standalone1.tests diff --git a/shell/ash_test/ash-quoting/dollar_repl_slash_bash1.tests b/shell/ash_test/ash-quoting/dollar_repl_slash_bash1.tests index 3fa2f186d..74dca1cc0 100755 --- a/shell/ash_test/ash-quoting/dollar_repl_slash_bash1.tests +++ b/shell/ash_test/ash-quoting/dollar_repl_slash_bash1.tests @@ -1,5 +1,5 @@ # The bug here was triggered by: -# * performin pathname expansion because we see [ +# * performing pathname expansion because we see [ # * replace operator did not escape \ in replace string IP=192.168.0.1 diff --git a/shell/hush_test/hush-invert/invert.right b/shell/hush_test/hush-invert/invert.right new file mode 100644 index 000000000..5a9239aa6 --- /dev/null +++ b/shell/hush_test/hush-invert/invert.right @@ -0,0 +1,10 @@ +1 +1 +1 +0 +0 +1 +0 +1 +0 +1 diff --git a/shell/hush_test/hush-invert/invert.tests b/shell/hush_test/hush-invert/invert.tests new file mode 100755 index 000000000..8393d95a4 --- /dev/null +++ b/shell/hush_test/hush-invert/invert.tests @@ -0,0 +1,19 @@ +# tests of return value inversion +# placeholder for future expansion + +# user subshells (...) did this wrong in bash versions before 2.04 + +! ( echo hello | grep h >/dev/null 2>&1 ); echo $? +! echo hello | grep h >/dev/null 2>&1 ; echo $? + +! true ; echo $? +! false; echo $? + +! (false) ; echo $? +! (true); echo $? + +! true | false ; echo $? +! false | true ; echo $? + +! (true | false) ; echo $? +! (false | true) ; echo $? diff --git a/shell/hush_test/hush-misc/heredoc4.right b/shell/hush_test/hush-misc/heredoc4.right new file mode 100644 index 000000000..371b092e2 --- /dev/null +++ b/shell/hush_test/hush-misc/heredoc4.right @@ -0,0 +1 @@ +'$' diff --git a/shell/hush_test/hush-misc/heredoc4.tests b/shell/hush_test/hush-misc/heredoc4.tests new file mode 100755 index 000000000..642ddb324 --- /dev/null +++ b/shell/hush_test/hush-misc/heredoc4.tests @@ -0,0 +1,3 @@ +cat <'192.168.0.1[' +echo "${IP//./\\.}" +echo "${IP//./\\.}"'[' # bug was here +echo "${IP//./\\.}[" # bug was here +echo "${IP//./\\\\.}[" # bug was here +echo "192\.168\.0\.1[" + +rm -f '192.168.0.1[' diff --git a/shell/hush_test/hush-read/read_r.tests b/shell/hush_test/hush-read/read_r.tests index 2c4cc6106..1f0a18afc 100755 --- a/shell/hush_test/hush-read/read_r.tests +++ b/shell/hush_test/hush-read/read_r.tests @@ -1,2 +1,4 @@ -echo -e 'test\\\nbest' | (read reply; echo "$reply") -echo -e 'test\\\nbest' | (read -r reply; echo "$reply") +echo 'test\ +best' | (read reply; echo "$reply") +echo 'test\ +best' | (read -r reply; echo "$reply") diff --git a/shell/hush_test/hush-signals/reap1.right b/shell/hush_test/hush-signals/reap1.right new file mode 100644 index 000000000..7326d9603 --- /dev/null +++ b/shell/hush_test/hush-signals/reap1.right @@ -0,0 +1 @@ +Ok diff --git a/shell/hush_test/hush-signals/reap1.tests b/shell/hush_test/hush-signals/reap1.tests new file mode 100755 index 000000000..bf1a1f908 --- /dev/null +++ b/shell/hush_test/hush-signals/reap1.tests @@ -0,0 +1,14 @@ +#!/bin/sh + +# Must not find us alive +{ sleep 2; kill -9 $$; } 2>/dev/null & + +sleep 1 & +PID=$! + +# We must exit the loop in one second. +# We had bug 5304: builtins never waited for exited children +while kill -0 $PID >/dev/null 2>&1; do + true +done +echo Ok diff --git a/shell/hush_test/hush-signals/sigint1.right b/shell/hush_test/hush-signals/sigint1.right new file mode 100644 index 000000000..a9094b056 --- /dev/null +++ b/shell/hush_test/hush-signals/sigint1.right @@ -0,0 +1 @@ +Sending SIGINT to main shell PID diff --git a/shell/hush_test/hush-signals/sigint1.tests b/shell/hush_test/hush-signals/sigint1.tests new file mode 100755 index 000000000..3d483d32a --- /dev/null +++ b/shell/hush_test/hush-signals/sigint1.tests @@ -0,0 +1,41 @@ +# What should happen if non-interactive shell gets SIGINT? + +(sleep 1; echo Sending SIGINT to main shell PID; exec kill -INT $$) & + +# We create a child which exits with 0 even on SIGINT +# (The complex command is necessary only if SIGINT is generated by ^C, +# in this testcase even bare "sleep 2" would do because +# in the testcase we don't send SIGINT *to the child*...) +$THIS_SH -c 'trap "exit 0" SIGINT; sleep 2' + +# In one second, we (main shell) get SIGINT here. +# The question is whether we should, or should not, exit. + +# bash will not stop here. It will execute next command(s). + +# The rationale for this is described here: +# http://www.cons.org/cracauer/sigint.html +# +# Basically, bash will not exit on SIGINT immediately if it waits +# for a child. It will wait for the child to exit. +# If child exits NOT by dying on SIGINT, then bash will not exit. +# +# The idea is that the following script: +# | emacs file.txt +# | more cmds +# User may use ^C to interrupt editor's ops like search. But then +# emacs exits normally. User expects that script doesn't stop. +# +# This is a nice idea, but detecting "did process really exit +# with SIGINT?" is racy. Consider: +# | bash -c 'while true; do /bin/true; done' +# When ^C is pressed while bash waits for /bin/true to exit, +# it may happen that /bin/true exits with exitcode 0 before +# ^C is delivered to it as SIGINT. bash will see SIGINT, then +# it will see that child exited with 0, and bash will NOT EXIT. + +# Therefore we do not implement bash behavior. +# I'd say that emacs need to put itself into a separate pgrp +# to isolate shell from getting stray SIGINTs from ^C. + +echo Next command after SIGINT was executed diff --git a/shell/hush_test/hush-signals/signal2.right b/shell/hush_test/hush-signals/signal2.right new file mode 100644 index 000000000..a2af919c0 --- /dev/null +++ b/shell/hush_test/hush-signals/signal2.right @@ -0,0 +1,3 @@ +child sleeps +child exits as expected +parent exits diff --git a/shell/hush_test/hush-signals/signal2.tests b/shell/hush_test/hush-signals/signal2.tests new file mode 100755 index 000000000..df639ca2c --- /dev/null +++ b/shell/hush_test/hush-signals/signal2.tests @@ -0,0 +1,18 @@ +#!/bin/sh + +$THIS_SH -c ' +cleanup() { + echo "child exits as expected" + exit +} +trap cleanup HUP +echo "child sleeps" +sleep 1 +echo "BAD exit from child!" +' & + +child=$! +sleep 0.1 # let child install handler first +kill -HUP $child +wait +echo "parent exits" diff --git a/shell/hush_test/hush-signals/signal3.right b/shell/hush_test/hush-signals/signal3.right new file mode 100644 index 000000000..3113ba5cf --- /dev/null +++ b/shell/hush_test/hush-signals/signal3.right @@ -0,0 +1,4 @@ +child sleeps +child got HUP +child exits +parent exits diff --git a/shell/hush_test/hush-signals/signal3.tests b/shell/hush_test/hush-signals/signal3.tests new file mode 100755 index 000000000..b56c2d97e --- /dev/null +++ b/shell/hush_test/hush-signals/signal3.tests @@ -0,0 +1,17 @@ +#!/bin/sh + +$THIS_SH -c ' +hup() { + echo "child got HUP" +} +trap hup HUP +echo "child sleeps" +sleep 1 +echo "child exits" +' & + +child=$! +sleep 0.1 # let child install handler first +kill -HUP $child +wait +echo "parent exits" diff --git a/shell/hush_test/hush-signals/signal5.right b/shell/hush_test/hush-signals/signal5.right new file mode 100644 index 000000000..7cfd4110e --- /dev/null +++ b/shell/hush_test/hush-signals/signal5.right @@ -0,0 +1,12 @@ +Sleeping +Sleeping +Waiting +2 sec passed, sending USR1 to parent +USR1 received +Wait exit code: 138 +Waiting +3 sec passed, sending USR1 to parent +USR1 received +Wait exit code: 138 +Waiting +Wait returned 0 diff --git a/shell/hush_test/hush-signals/signal5.tests b/shell/hush_test/hush-signals/signal5.tests new file mode 100755 index 000000000..179bcdd80 --- /dev/null +++ b/shell/hush_test/hush-signals/signal5.tests @@ -0,0 +1,14 @@ +trap "echo USR1 received" USR1 +stub() { + echo "Sleeping" + sleep $1 + echo "$1 sec passed, sending USR1 to parent" + kill -USR1 $$ +} +stub 3 & +stub 2 & +sleep 1 +until { echo "Waiting"; wait; } do + echo "Wait exit code: $?" +done +echo "Wait returned 0" diff --git a/shell/hush_test/hush-signals/signal6.right b/shell/hush_test/hush-signals/signal6.right new file mode 100644 index 000000000..df4d9306a --- /dev/null +++ b/shell/hush_test/hush-signals/signal6.right @@ -0,0 +1,2 @@ +got TERM +Done: 0 diff --git a/shell/hush_test/hush-signals/signal6.tests b/shell/hush_test/hush-signals/signal6.tests new file mode 100755 index 000000000..3ce151060 --- /dev/null +++ b/shell/hush_test/hush-signals/signal6.tests @@ -0,0 +1,2 @@ +{ trap "echo got TERM" TERM; sleep 3; }& sleep 1; kill $!; wait +echo Done: $? diff --git a/shell/hush_test/hush-signals/sigquit_exec.right b/shell/hush_test/hush-signals/sigquit_exec.right new file mode 100644 index 000000000..a8041929a --- /dev/null +++ b/shell/hush_test/hush-signals/sigquit_exec.right @@ -0,0 +1,2 @@ +SigIgn: 0000000000000000 +SigIgn: 0000000000000000 diff --git a/shell/hush_test/hush-signals/sigquit_exec.tests b/shell/hush_test/hush-signals/sigquit_exec.tests new file mode 100755 index 000000000..24bda6921 --- /dev/null +++ b/shell/hush_test/hush-signals/sigquit_exec.tests @@ -0,0 +1,4 @@ +# Should show no masked signals in both cases. +# We had a bug where SIGQUIT was masked on exec. +grep SigIgn: /proc/self/status +exec grep SigIgn: /proc/self/status diff --git a/shell/hush_test/hush-standalone/noexec_gets_no_env.right b/shell/hush_test/hush-standalone/noexec_gets_no_env.right new file mode 100644 index 000000000..8522dff31 --- /dev/null +++ b/shell/hush_test/hush-standalone/noexec_gets_no_env.right @@ -0,0 +1,4 @@ +VAR7=VAL +0 +VAR8=VAL +0 diff --git a/shell/hush_test/hush-standalone/noexec_gets_no_env.tests b/shell/hush_test/hush-standalone/noexec_gets_no_env.tests new file mode 100755 index 000000000..0d347bdcd --- /dev/null +++ b/shell/hush_test/hush-standalone/noexec_gets_no_env.tests @@ -0,0 +1,5 @@ +export VAR7=VAL +env | grep ^VAR7= +echo $? +VAR8=VAL env | grep ^VAR8= +echo $? diff --git a/shell/hush_test/hush-standalone/nofork_trashes_getopt.right b/shell/hush_test/hush-standalone/nofork_trashes_getopt.right new file mode 100644 index 000000000..573541ac9 --- /dev/null +++ b/shell/hush_test/hush-standalone/nofork_trashes_getopt.right @@ -0,0 +1 @@ +0 diff --git a/shell/hush_test/hush-standalone/nofork_trashes_getopt.tests b/shell/hush_test/hush-standalone/nofork_trashes_getopt.tests new file mode 100755 index 000000000..f42c50730 --- /dev/null +++ b/shell/hush_test/hush-standalone/nofork_trashes_getopt.tests @@ -0,0 +1,6 @@ +# In this test, rm is NOFORK and it modifies getopt internal state +rm -f non_existent_file +# Subsequent hexdump is run as NOEXEC, and thus still uses this state +hexdump