From: Stefano Lattarini Date: Thu, 16 Jun 2011 08:17:30 +0000 (+0200) Subject: Merge branch 'tests-more-shells' into testsuite-work X-Git-Tag: ng-0.5a~89^2~187 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=fef8ee88b48851569de5b05768c892914a72b16d;p=thirdparty%2Fautomake.git Merge branch 'tests-more-shells' into testsuite-work --- fef8ee88b48851569de5b05768c892914a72b16d diff --cc ChangeLog index afca04437,f72eaacc7..2029e92cd --- a/ChangeLog +++ b/ChangeLog @@@ -1,89 -1,44 +1,130 @@@ + 2011-06-13 Stefano Lattarini + + tests: remove 'test_prefer_config_shell' from the environment + Since commit `v1.11-910-g7df1a9b', the once user-overridable + variable `$test_prefer_config_shell' has become an internal + detail, and the test scripts now complain and bail out if it is + set in the environment. + * tests/Makefile.am (AM_TESTS_ENVIRONMENT): Unset the variable + `test_prefer_config_shell' if it is set in the environment. + + 2011-06-13 Stefano Lattarini + + tests: autogenerate list of wrapped tests for `lib/' shell scripts + * tests/gen-config-shell-tests: New script, generates distributed + makefile snippet `tests/config-shell-tests.am' to list all tests + that use the `get_shell_script' function, with names mangled to + use suffix `-w.shtst', in ... + * tests/Makefile.am (config_shell_tests): ... this macro, whose + definition has been consequently removed from Makefile.am. + (EXTRA_DIST): Distribute the new script. + ($(srcdir)/config-shell-tests.am): Generate using the new script. + (include): Include the `config-shell-tests.am' fragment. + * bootstrap: Invoke `tests/gen-config-shell-tests' to generate + `tests/config-shell-tests.am'. + * tests/.gitignore: Ignore `config-shell-tests.am'. + * tests/gen-parallel-tests: Fixlet in heading comments. + + 2011-06-13 Stefano Lattarini + + tests: test mdate-sh with /bin/sh too + * tests/mdate5.test: Fetch the `mdate-sh' script using the + `get_shell_script' function, and run it directly instead of + using `$SHELL'. + * tests/mdate6.test: Likewise. Since we are at it, make checks + on the `mdate-sh' output stricter, remove now unneeded calls to + aclocal and automake and creation/extension of `configure.in', + `Makefile.am' and `textutils.tex' files, and add a trailing `:' + command. + * tests/Makefile.am (config_shell_tests): Add `mdate5-w.shtst' + and `mdate6-w.shtst'. + +2011-06-13 Stefano Lattarini + + tests: extend tests on `--add-missing' and `--copy' a bit + * tests/add-missing.test: Fix typo in heading comments. Try with + another testcase that install many (but not all) the auxiliary + scripts at once, and uses non-standard (but valid and documented) + setups (e.g., defining YACC in Makefile.am instead of calling + AC_PROG_YACC from configure.in). + * tests/copy.test: Reference `add-missing.test' in heading + comments. Try few more test scenarios. + +2011-06-16 Stefano Lattarini + + tests: few fixlets and improvements + * tests/cond31.test ($required): Remove `cc', it's not really + needed. + * tests/confh.test: Call autoheader too. The lack of this call + wasn't causing spurious failures because the automatic remake + rules were somehow invoking it on our behalf (at make time). + * tests/fn99subdir.test: Use $subdirname throughout, instead of + ${subdirname}, for consistency with the rest of the testsuite. + Avoid an unnecessary subshell, which could also cause spurious + passes, being guarded by a trailing `|| Exit 1', which neutralize + the `errexit' flag. Remove an unnecessary `|| Exit 1' guard. + * tests/insh2.test: Rewrite to avoid hackish Makefile.in munging, + and to also run configure and make. + +2011-06-13 Stefano Lattarini + + tests: don't hard-code test name in txinfo21.test + * tests/txinfo21.test: Use `$me' instead of hard-coding the + current testcase name "txinfo21". Add a trailing `:' command + since we are at it. + +2011-06-10 Stefano Lattarini + + tests: new test dedicated to `--add-missing' and `--copy' + * tests/add-missing.test: New test. + * tests/Makefile.am (TESTS): Update. + Suggested by Peter Rosin. + +2011-06-08 Stefano Lattarini + + testsuite: use 'fatal_' and 'framework_failure_' for hard errors + * tests/defs (require_xsi): Use `fatal_', not `framework_failure', + to report an invalid usage. + * tests/remake-gnulib-remove-header.test: Prefer using `fatal_' + with a proper error message over a direct call to `Exit 99'. + * tests/pr8365-remake-timing.test: Likewise. + * tests/cygnus-imply-foreign.test: Likewise. + * tests/missing6.test: Likewise. + * tests/cond8.test: Likewise. + * tests/cond33.test: Likewise. + * tests/python-virtualenv.test: Prefer using `framework_failure_' + with a proper error message over a direct call to `Exit 99'. + * tests/instspc-tests.sh: Prefer using `framework_failure_' and + `fatal_' over direct calls to `Exit 99'. + (fatal_): Define this (which is a simplified version of the one + in `tests/defs') for early uses (i.e., before `tests/defs' + gets sourced). + * tests/depmode-tests.sh: Likewise. Also, simplify the + 'get_depmodes' function and calls to it accordingly. + +2011-06-08 Stefano Lattarini + + self tests: check new 'fatal_' function + * tests/self-check-exit.test: Also check the new 'fatal_' + function. + +2011-06-08 Stefano Lattarini + + test defs: new function 'fatal_', for hard errors + Before this patch, the only way offered by tests/defs to + properly signal a hard error was the `framework_failure_' + function. But the error message issued by that function, + as its name would suggest, refers to a set-up failure in the + testsuite, while hard errors can obviously also be due to + other reasons. The best way to fix this inconsistency is to + introduce a new function with a more general error message. + Inspired by a recent similar change to Gnulib's tests/init.sh. + * tests/defs (fatal_): New function. + * tests/README (Section "Writing test cases" subsection "Do"): + Suggest the use of `fatal_', not of `framework_failure_', for + generic hard errors. The latter should be reserved for "real" + set-up failures. + 2011-06-07 Stefano Lattarini tests: `lib/' shell scripts transparently tested also with $SHELL