Explicitly report non-zero ctest_test return value as an error

scripts/circle/runOnCircle.sh

@@ -32,12 +32,3 @@ case "$1" in
 esac
 
 /opt/cmake/3.6.0/bin/ctest -VV -S ${CTEST_SCRIPT} -Ddashboard_full=OFF -Ddashboard_do_${STEP}=TRUE
-
-if [ $? -eq 0 ]
-then
-  echo "ctest returned success"
-  exit 0
-else
-  echo "ctest returned failure" >&2
-  exit 1
-fi

scripts/dashboard/common.cmake

@@ -394,7 +394,11 @@ if(dashboard_do_test)
   endif()
   message("Calling ctest_test")
   ctest_test(${CTEST_TEST_ARGS} RETURN_VALUE TEST_RESULTS)
-  message("ctest test results return value: ${TEST_RESULTS}")
+  if(${TEST_RESULTS} EQUAL 0)
+    message("ctest test results return value: ${TEST_RESULTS}")
+  else()
+    message(SEND_ERROR "Some tests failed")
+  endif()
   ctest_submit(PARTS Test)
 endif()