More updates based on PR feedback

SQUASH before merging

Made updates to test_xdd_debug_init.sh to just call XDD directly without
putting it in a string first.

Not sure what the idea was behind the redirection of the shell scripts
to the log file was. As fas as I can tell, it never resulted in anything
being written out to test-dir/logs. For now, I am just getting rid of
the redirection till someone can show me what it was suppose to do.

Signed-off-by: Brian Atkinson <batkinson@lanl.gov>

tests/debug/CMakeLists.txt

@@ -5,7 +5,7 @@ set(DEBUG_TESTS
 foreach(TEST ${DEBUG_TESTS})
   configure_file("${TEST}" "${TEST}" COPYONLY)
   add_test(NAME "${TEST}"
-    COMMAND sh -c "${CMAKE_CURRENT_BINARY_DIR}/${TEST} 2> ${TESTDIR}/logs/${TEST}.log"
+    COMMAND ${CMAKE_CURRENT_BINARY_DIR}/${TEST}
     WORKING_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}")
   set_tests_properties("${TEST}" PROPERTIES LABELS "debug")
   set_tests_properties("${TEST}" PROPERTIES SKIP_RETURN_CODE 255)

tests/debug/test_xdd_debug_init.sh

@@ -16,16 +16,16 @@ initialize_test
 
 echo "Foobar and I am here motherfucker"
 
-xdd_cmd="${XDDTEST_XDD_EXE} -op write -reqsize 128 -numreqs 1 -targets 1 /dev/null -verbose -debug INIT"
-${xdd_cmd} 2>&1 | grep "bound to NUMA node"
+${XDDTEST_XDD_EXE} -op write -reqsize 128 -numreqs 1 -targets 1 /dev/null -verbose -debug INIT \
+  2>&1 | grep "bound to NUMA node"
 
 if [[ $? -ne 0 ]]; then
   # test failed
   finalize_test 1
 fi
 
-xdd_cmd="${XDDTEST_XDD_EXE} -op write -reqsize 128 -numreqs 1 -targets 1 /dev/null -verbose"
-${xdd_cmd} 2>&1 | grep "bound to NUMA node"
+${XDDTEST_XDD_EXE} -op write -reqsize 128 -numreqs 1 -targets 1 /dev/null -verbose \
+  2>&1 | grep "bound to NUMA node"
 
 if [[ $? -ne 1 ]]; then
   # test failed

tests/functional/CMakeLists.txt

@@ -21,7 +21,7 @@ set(FUNCTIONAL
 foreach(TEST ${FUNCTIONAL})
   configure_file("${TEST}" "${TEST}" COPYONLY)
   add_test(NAME "${TEST}"
-    COMMAND sh -c "${CMAKE_CURRENT_BINARY_DIR}/${TEST} 2> ${TESTDIR}/logs/${TEST}.log"
+    COMMAND ${CMAKE_CURRENT_BINARY_DIR}/${TEST}
     WORKING_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}")
   set_tests_properties("${TEST}" PROPERTIES LABELS "functional")
   set_tests_properties("${TEST}" PROPERTIES SKIP_RETURN_CODE 255)