From: Stefano Lattarini <stefano.lattarini@gmail.com>
Date: Tue, 9 Aug 2011 10:13:53 +0000 (+0200)
Subject: test defs: few more sanity checks
X-Git-Tag: v1.11b~257^2~115
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=376293a9ecaf9cc15041260d11e595af867608c8;p=platform%2Fupstream%2Fautomake.git

test defs: few more sanity checks

* tests/defs: Catch some more (very unlikely) set-up or internal
errors, and diagnose them explicitly.  This is intended mostly
to be a theoretic-only improvement.
---

diff --git a/ChangeLog b/ChangeLog
index 0cbdc9e..3ba884f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2011-08-09  Stefano Lattarini  <stefano.lattarini@gmail.com>
 
+	test defs: few more sanity checks
+	* tests/defs: Catch some more (very unlikely) set-up or internal
+	errors, and diagnose them explicitly.  This is intended mostly
+	to be a theoretic-only improvement.
+
+2011-08-09  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
 	test defs: really make them "set -e" clean
 	Motivated by a spurious failure of test `instsh3-w.shtst' on
 	Solaris 10.
diff --git a/tests/defs b/tests/defs
index cf9b86a..5e4c2c2 100644
--- a/tests/defs
+++ b/tests/defs
@@ -188,7 +188,10 @@ else
 fi
 
 if test -f "$testsrcdir/$funcs_file_"; then
-  . "$testsrcdir/$funcs_file_"
+  . "$testsrcdir/$funcs_file_" || {
+    echo "$me: error sourcing $testsrcdir/$funcs_file_" >&2
+    Exit 99
+  }
 else
   echo "$me: $testsrcdir/$funcs_file_ not found, check \$testsrcdir" >&2
   Exit 99
@@ -875,9 +878,12 @@ else
   # by the cleanup trap below if the test passes.  If the test doesn't pass,
   # this directory will be kept, to facilitate debugging.
   testSubDir=$me.dir
-  test ! -d $testSubDir || rm_rf_ $testSubDir
-  mkdir $testSubDir || framework_failure_ "creating test subdirectory"
-  cd ./$testSubDir
+  test ! -d $testSubDir || rm_rf_ $testSubDir \
+    || framework_failure_ "removing old test subdirectory"
+  mkdir $testSubDir \
+    || framework_failure_ "creating test subdirectory"
+  cd ./$testSubDir \
+    || framework_failure_ "cannot chdir into test subdirectory"
   if test x"$am_create_testdir" != x"empty"; then
     cp "$top_testsrcdir"/lib/install-sh "$top_testsrcdir"/lib/missing \
        "$top_testsrcdir"/lib/depcomp . \
@@ -895,7 +901,7 @@ else
         echo "AM_INIT_AUTOMAKE"
       fi
       echo "AC_CONFIG_FILES([Makefile])"
-    } > configure.in
+    } >configure.in || framework_failure_ "creating configure.in skeleton"
   fi
 fi