From 374211e79dd51b6ee0aad4c0fbd47f7c27f4fec4 Mon Sep 17 00:00:00 2001
From: Allen Byrne <50328838+byrnHDF@users.noreply.github.com>
Date: Tue, 2 Feb 2021 23:50:44 -0600
Subject: Test script 112 (#301)

* OESS-98 fix tools test for plugins

* sync fork

* Merge of changes from dev

* Move problem option to bottom of the list until fixed

* HDFFV-11106 - fix parsing optional args

* HDFFV-11106 add note

* grammer fix

* Whitespace after clang formatting

* Undo format version 11 changes

* Update check to working version

* Merge workflow and minor changes from develop

* test testing script merge from develop
---
 test/testerror.sh.in        |  6 ++++++
 test/testexternal_env.sh.in |  8 ++++----
 test/testflushrefresh.sh.in | 16 ++++++++++++++--
 test/testlinks_env.sh.in    |  4 ++--
 test/testvds_env.sh.in      |  4 ++--
 5 files changed, 28 insertions(+), 10 deletions(-)

diff --git a/test/testerror.sh.in b/test/testerror.sh.in
index b5ec853..e4c6339 100644
--- a/test/testerror.sh.in
+++ b/test/testerror.sh.in
@@ -64,6 +64,12 @@ TEST() {
       # Skip the plugin for testing missing filter.
       $ENVCMD $RUNSERIAL $TEST_ERR_BIN
    ) >$actual 2>$actual_err
+
+   # Check for core dump
+   if [ $? != 0 ]; then
+       nerrors=`expr $nerrors + 1`
+   fi
+
    # Extract file name, line number, version and thread IDs because they may be different
    sed -e 's/thread [0-9]*/thread (IDs)/' -e 's/: .*\.c /: (file name) /' \
 	-e 's/line [0-9]*/line (number)/' \
diff --git a/test/testexternal_env.sh.in b/test/testexternal_env.sh.in
index 3cc140d..1327c7d 100644
--- a/test/testexternal_env.sh.in
+++ b/test/testexternal_env.sh.in
@@ -34,9 +34,9 @@ ENVCMD="env HDF5_EXTFILE_PREFIX=\${ORIGIN}"    # The environment variable & valu
 $ENVCMD $RUNSERIAL $TEST_BIN
 exitcode=$?
 if [ $exitcode -eq 0 ]; then
-        echo "Test prefix for HDF5_EXTFILE_PREFIX PASSED"
-    else
- 	nerrors="`expr $nerrors + 1`"
-	echo "***Error encountered for HDF5_EXTFILE_PREFIX test***"
+    echo "Test prefix for HDF5_EXTFILE_PREFIX PASSED"
+else
+    nerrors="`expr $nerrors + 1`"
+    echo "***Error encountered for HDF5_EXTFILE_PREFIX test***"
 fi
 exit $nerrors
diff --git a/test/testflushrefresh.sh.in b/test/testflushrefresh.sh.in
index 3cdf10f..be44a13 100644
--- a/test/testflushrefresh.sh.in
+++ b/test/testflushrefresh.sh.in
@@ -82,7 +82,7 @@ fi
 # different, occasionally the wrong file is deleted, interrupting the flow of
 # the test.  Running each of these tests in its own directory should eliminate
 # the problem.
-mkdir flushrefresh_test
+mkdir -p flushrefresh_test
 cp flushrefresh flushrefresh_test
 
 # With the --disable-shared option, flushrefresh is built in the test directory,
@@ -90,7 +90,7 @@ cp flushrefresh flushrefresh_test
 # the test directory.  test/flushrefresh should always be copied,
 # .libs/flushrefresh should be copied only if it exists.
 if [ -f .libs/flushrefresh ]; then
-    mkdir flushrefresh_test/.libs
+    mkdir -p flushrefresh_test/.libs
     for FILE in .libs/flushrefresh*; do
         case "$FILE" in
             *.o) continue ;;    ## don't copy the .o files
@@ -156,6 +156,12 @@ until [ $verification_done -eq 1 ]; do
     echo "all flush verification complete" > $endsignal
   else
     ./flushrefresh $param1 $param2
+
+    # Check for core dump
+    if [ $? -gt 0 ]; then
+        nerrors=`expr $nerrors + 1`
+    fi
+
     echo "verification flush process done" > $endsignal
   fi
 
@@ -195,6 +201,12 @@ if [ $timedout -eq 0 ]; then
       echo "all refresh verification complete" > $endsignal
     else
       ./flushrefresh $param1
+
+      # Check for core dump
+      if [ $? -gt 0 ]; then
+          nerrors=`expr $nerrors + 1`
+      fi
+
       echo "refresh verifiction process done" > $endsignal
     fi
 
diff --git a/test/testlinks_env.sh.in b/test/testlinks_env.sh.in
index 3d6b3ce..e61e057 100644
--- a/test/testlinks_env.sh.in
+++ b/test/testlinks_env.sh.in
@@ -34,8 +34,8 @@ echo "$ENVCMD $RUNSERIAL $TEST_BIN"
 $ENVCMD $RUNSERIAL $TEST_BIN
 exitcode=$?
 if [ $exitcode -eq 0 ]; then
-        echo "Test for HDF5_EXT_PREFIX PASSED"
-    else
+    echo "Test for HDF5_EXT_PREFIX PASSED"
+else
     nerrors="`expr $nerrors + 1`"
     echo "***Error encountered for HDF5_EXT_PREFIX test***"
 fi
diff --git a/test/testvds_env.sh.in b/test/testvds_env.sh.in
index e9a27da..9577d7d 100644
--- a/test/testvds_env.sh.in
+++ b/test/testvds_env.sh.in
@@ -35,8 +35,8 @@ UNENVCMD="unset HDF5_VDS_PREFIX"                # Unset the environment variable
 $ENVCMD $RUNSERIAL $TEST_BIN
 exitcode=$?
 if [ $exitcode -eq 0 ]; then
-        echo "Test prefix for HDF5_VDS_PREFIX PASSED"
-    else
+    echo "Test prefix for HDF5_VDS_PREFIX PASSED"
+else
     nerrors="`expr $nerrors + 1`"
     echo "***Error encountered for HDF5_VDS_PREFIX test***"
 fi
-- 
cgit v0.12