Issue #265 - Harmonize success message of tests (#266)

yoavnir · Oct 12, 2024 · b3ab532 · b3ab532
1 parent 845aa54
commit b3ab532
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/specs/src/test/ProcessingTest.cc b/specs/src/test/ProcessingTest.cc
@@ -749,7 +749,7 @@ int main(int argc, char** argv)
 		}
 		std::cout << "\n";
 	} else {
-		std::cout << "\nAll tests passed.\n";
+		std::cout << "\n*** All tests passed.\n";
 	}
 
 	return (errorCount==0) ? 0 : 4;

diff --git a/specs/src/test/TokenTest.cc b/specs/src/test/TokenTest.cc
@@ -96,7 +96,7 @@ int main(int argc, char** argv)
 	if (failedTests) {
 		std::cout << "\n" << failedTests << " failed tests.\n";
 	} else {
-		std::cout << "\nAll tests passed.\n";
+		std::cout << "\n*** All tests passed.\n";
 	}
 
 	return (failedTests==0) ? 0 : 4;

diff --git a/specs/tests/valgrind_specs.py b/specs/tests/valgrind_specs.py
@@ -15,7 +15,10 @@ def run_case(spec, input, description, expected_rc=memcheck.RetCode_SUCCESS, con
         sys.stdout.write("Failed. RC={}; info={}; expected: {}\n".format(memcheck.RetCode_strings[rc],info,memcheck.RetCode_strings[expected_rc]))
         exit(4)
     else:
-        sys.stdout.write("No leaks\n")
+        if memcheck.no_valgrind:
+        	sys.stdout.write("valgrind skipped\n")
+        else:
+	        sys.stdout.write("No leaks\n")
 
 # Parse the one command line options
 parser = argparse.ArgumentParser()
@@ -735,3 +738,5 @@ def run_case(spec, input, description, expected_rc=memcheck.RetCode_SUCCESS, con
 """
 run_case(s,i,"inline variable")
 
+
+sys.stdout.write("\n*** All tests passed.\n")