tests: never source test-defs.sh directly, source test-lib.sh instead

After the recent re-organization, sourcing 'test-defs.sh' directly might
not work well and cause spurious failures or other unexpected behaviours.
We should source 'test-lib.sh' instead, which contains not more direct
code execution (only definition of shell variables/functions, or sourcing
of other '*.sh' with the same property), is protected against multiple
inclusions, and sources 'test-defs.sh' automatically in in a proper way.

* t/testsuite-summary-count.sh, t/tap-summary.sh, t/tap-summary-color.sh,
t/testsuite-summary-color.sh: Source 'test-lib.sh', not 'test-defs.sh'.
* gen-testsuite-part: Likewise, in the generated wrapper scripts.

Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>

diff --git a/gen-testsuite-part b/gen-testsuite-part
index d646b27..21c26c4 100755 (executable)
--- a/gen-testsuite-part
+++ b/gen-testsuite-part
@@ -96,7 +96,7 @@ sub write_wrapper_script ($$$)
   print $file_handle unindent <<EOF;
     #! /bin/sh
     # This file has been automatically generated.  DO NOT EDIT BY HAND!
-    . test-defs.sh
+    . test-lib.sh
     $shell_setup_code
     # In the spirit of VPATH, we prefer a test in the build tree
     # over one in the source tree.

diff --git a/t/tap-summary-color.sh b/t/tap-summary-color.sh
index c84e848..e9b516b 100755 (executable)
--- a/t/tap-summary-color.sh
+++ b/t/tap-summary-color.sh
@@ -17,7 +17,7 @@
 # TAP support:
 #  - colorized testsuite summary
 
-. test-defs.sh
+. test-lib.sh
 
 use_colors=yes
 . tap-summary-aux.sh

diff --git a/t/tap-summary.sh b/t/tap-summary.sh
index 6e0e798..c4bd739 100755 (executable)
--- a/t/tap-summary.sh
+++ b/t/tap-summary.sh
@@ -17,7 +17,7 @@
 # TAP support:
 #  - colorized testsuite summary
 
-. test-defs.sh
+. test-lib.sh
 
 use_colors=no
 . tap-summary-aux.sh

diff --git a/t/testsuite-summary-color.sh b/t/testsuite-summary-color.sh
index a0effd6..7278a2f 100755 (executable)
--- a/t/testsuite-summary-color.sh
+++ b/t/testsuite-summary-color.sh
@@ -16,11 +16,9 @@
 
 # Check coloring of the testsuite summary.
 
-. test-defs.sh
-
-use_colors=yes
-use_vpath=no
+. test-lib.sh
 
+use_colors=yes; use_vpath=no
 . testsuite-summary-checks.sh
 
 ./configure

diff --git a/t/testsuite-summary-count.sh b/t/testsuite-summary-count.sh
index 47bb663..25ecf27 100755 (executable)
--- a/t/testsuite-summary-count.sh
+++ b/t/testsuite-summary-count.sh
@@ -16,11 +16,9 @@
 
 # Check test counts in the testsuite summary.
 
-. test-defs.sh
-
-use_colors=no
-use_vpath=no
+. test-lib.sh
 
+use_colors=no; use_vpath=no
 . testsuite-summary-checks.sh
 
 ./configure