lib/tests/phoronix-test-suite.sh: handle TimesToRun not set in test-d…

…efinition.xml The test-definition.xml of some benchmarks don't have explicit TimesToRun parameter, so set them to run 3 times in this situation. /var/lib/phoronix-test-suite/test-profiles/pts/build-imagemagick-1.7.2$ grep TimesToRun test-definition.xml <-- nothing == fix issue == failed to force times to run of build-imagemagick-1.7.2 Signed-off-by: Yujie Liu <[email protected]> Signed-off-by: Philip Li <[email protected]>
intel · May 10, 2024 · 03a371f · 03a371f
1 parent 651ee6f
commit 03a371f
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/tests/phoronix-test-suite.sh b/lib/tests/phoronix-test-suite.sh
@@ -133,7 +133,11 @@ force_times_to_run()
 		local target=${environment_directory}/../test-profiles/pts/${test}/test-definition.xml
 		if [[ -f $target ]]; then
 			local times_to_run=$(grep -oP '(?<=<TimesToRun>).*(?=</TimesToRun>)' $target)
-			[[ $times_to_run ]] && log_cmd export FORCE_TIMES_TO_RUN=$times_to_run
+			if [[ $times_to_run ]]; then
+				log_cmd export FORCE_TIMES_TO_RUN=$times_to_run
+			else
+				log_cmd export FORCE_TIMES_TO_RUN=3
+			fi
 		else
 			log_cmd export FORCE_TIMES_TO_RUN=3
 		fi