More debugging.

[nmh] / test / install-mh / test-version-check

diff --git a/test/install-mh/test-version-check b/test/install-mh/test-version-check
index 3dfef0430c667615c1c19fc5961f79f6d6e18563..8085a44579d7b5755f639de2459bf70e42a86571 100755 (executable)
--- a/test/install-mh/test-version-check
+++ b/test/install-mh/test-version-check
@@ -6,6 +6,7 @@
 #
 ######################################################
 
 #
 ######################################################
 

+set -x # ???? debug on buildbot

 set -e
 
 if test -z "${MH_OBJ_DIR}"; then
 set -e
 
 if test -z "${MH_OBJ_DIR}"; then


@@ -18,41 +19,103 @@ fi
 setup_test
 
 require_prog script
 setup_test
 
 require_prog script

+#### FreeBSD script, e.g., doesn't use -c to identify a command to run.
+if script -c 'echo OK' /dev/null 2>&1 | egrep 'OK' >/dev/null; then
+    script_command_opt='-c'
+else
+    script_command_opt=''
+fi
+
+#### Run a command but don't wait for user input.  script(1) seems to do
+#### what we want by not waiting when run in the background.
+run_without_input() {
+    echo MHCONTEXT = $MHCONTEXT # ????
+    if [ "$script_command_opt" = -c ]; then
+        #### -c takes single argument with command + arguments.
+        script -q -f -c "$*" "$actual" >/dev/null &
+    else
+        #### E.g., FreeBSD.  Don't combine command arguments.
+        script -q -t 0 "$actual" "$@" >/dev/null &
+    fi
+
+    wait $!
+}

 
 actual="$MH_TEST_DIR/test-version-check$$.actual"
 context="${MH_TEST_DIR}"/Mail/context
 
 actual="$MH_TEST_DIR/test-version-check$$.actual"
 context="${MH_TEST_DIR}"/Mail/context

+version="Version: nmh-${MH_VERSION}"

 
 

-# Removing Version will trigger the welcome message.
+# Removing Version will trigger the welcome message.  (setup_test
+# inserted it so that other tests wouldn't show it.)

 grep -v Version "$context" > "$context.NEW"
 mv -f "$context.NEW" "$context"
 
 
 start_test 'mhparam skips the welcome message'
 grep -v Version "$context" > "$context.NEW"
 mv -f "$context.NEW" "$context"
 
 
 start_test 'mhparam skips the welcome message'

-# The command will wait for the user to hit return.  script seems to do what
-# we want by not waiting when run in the background.
-script -c 'mhparam path last' -f -q "$actual" >/dev/null &
-wait $!
+run_without_input mhparam path

 grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false
 grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false

+# Make sure that version wasn't added to context.
+grep "^${version}$" "${MH_TEST_DIR}/Mail/context" >/dev/null  &&  false
+rm "$actual"
+
+
+start_test 'Welcome: disable in profile skips the welcome message'
+cp "${MH}" "${MH}-welcome"
+printf 'Welcome: disable\n' >> "${MH}-welcome"
+(MH="${MH}-welcome" run_without_input pick last)
+grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false
+# Make sure that version wasn't added to context.
+grep "^${version}$" "${MH_TEST_DIR}/Mail/context" >/dev/null  &&  false

 rm "$actual"
 rm "$actual"

-finish_test

 
 
 start_test 'with welcome message'
 
 
 start_test 'with welcome message'

-script -c 'pick last' -f -q "$actual" >/dev/null &
-wait $!
+run_without_input pick last
+cat "${MH_TEST_DIR}/Mail/context"; cat "$actual" # ????

 grep 'Welcome to nmh version ' "$actual" >/dev/null
 grep 'Welcome to nmh version ' "$actual" >/dev/null

+# Make sure that version was added to context.
+grep "^${version}$" "${MH_TEST_DIR}/Mail/context" >/dev/null

 rm "$actual"
 rm "$actual"

-finish_test

 
 
 start_test 'without welcome message'
 # After running the previous test, this one should not have
 # the welcome message.
 
 
 start_test 'without welcome message'
 # After running the previous test, this one should not have
 # the welcome message.

-script -c 'pick last' -f -q "$actual" >/dev/null &
-wait $!
+run_without_input pick last

 grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false
 grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false

+# Make sure that version is still in context.
+grep "^${version}$" "${MH_TEST_DIR}/Mail/context" >/dev/null

 rm "$actual"
 rm "$actual"

-finish_test

 
 
 
 

+start_test 'with MHCONTEXT, welcome only if older'
+MHCONTEXT="${MH_TEST_DIR}/Mail/context2"; export MHCONTEXT
+printf 'Version: nmh-1.5\n' >"${MHCONTEXT}"
+cat "$MHCONTEXT" # ????
+run_without_input pick last
+grep 'Welcome to nmh version ' "$actual" >/dev/null
+# And make sure that version did get updated in context.
+grep "^${version}$" "${MHCONTEXT}" >/dev/null
+rm "$actual"
+
+
+start_test "with MHCONTEXT doesn't welcome if newer"
+printf 'Version: nmh-10000.0\n' >"${MHCONTEXT}"
+run_without_input pick last
+grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false
+# And make sure that version didn't get updated in context.
+grep '^Version: nmh-10000.0$' "${MHCONTEXT}" >/dev/null
+rm "$actual"
+
+
+start_test 'with MHCONTEXT but no version, no welcome and update'
+printf '' >"${MHCONTEXT}"
+run_without_input pick last
+grep 'Welcome to nmh version ' "$actual" >/dev/null  &&  false
+# And make sure that version did get updated in context.
+grep "^${version}$" "${MHCONTEXT}" >/dev/null
+rm "$actual"
+
+
+finish_test

 exit $failed
 exit $failed