Add quotation marks around values in tmt-report-result output

teemtee · Sep 23, 2024 · afcca90 · afcca90
1 parent 9871fad
commit afcca90
Show file tree

Hide file tree

Showing 5 changed files with 52 additions and 16 deletions.
diff --git a/tests/execute/restraint/report-result/data/report.sh b/tests/execute/restraint/report-result/data/report.sh
@@ -13,43 +13,43 @@ rlJournalStart
         rlRun "rstrnt-report-result --server http://test-example.com report SKIP" 0 "Generating Restraint report of skipped test."
         rlRun "ls $RESULT_FILE" 0 "Result report successfully generated."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep 'result: skip' $rlRun_LOG
+        rlAssertGrep 'result: "skip"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
         rlRun "rstrnt-report-result --server http://test-example.com --port 55 --disable-plugin avc --message 'Example output message.' -o /tmp/example_output.txt report PASS 66" 0 "Generating Restraint report of passed test."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep "- report_example_output.txt" $rlRun_LOG
-        rlAssertGrep 'name: /report' $rlRun_LOG
-        rlAssertGrep 'result: pass' $rlRun_LOG
+        rlAssertGrep '- "report_example_output.txt"' $rlRun_LOG
+        rlAssertGrep 'name: "/report"' $rlRun_LOG
+        rlAssertGrep 'result: "pass"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
         rlRun "rstrnt-report-result --server http://test-example.com report WARN" 0 "Generating Restraint report of warned test."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep 'result: warn' $rlRun_LOG
+        rlAssertGrep 'result: "warn"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
         rlRun "rstrnt-report-result --server http://test-example.com report FAIL" 0 "Generating Restraint report of failed test."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep 'result: fail' $rlRun_LOG
+        rlAssertGrep 'result: "fail"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
     rlPhaseEnd
 
     rlPhaseStartTest 'Verify mocked RHTS rhts-report-result file generated correctly.'
         rlRun "rhts-report-result rhts-report SKIP /tmp/example_output.txt" 0 "Generating RHTS report of skipped test without optional metric."
         rlRun "ls $RESULT_FILE" 0 "Result report successfully generated."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep 'result: skip' $rlRun_LOG
+        rlAssertGrep 'result: "skip"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
         rlRun "rhts-report-result rhts-report PASS /tmp/example_output.txt 66" 0 "Generating RHTS report of passed test."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep "- rhts-report_example_output.txt" $rlRun_LOG
-        rlAssertGrep 'name: /rhts-report' $rlRun_LOG
-        rlAssertGrep 'result: pass' $rlRun_LOG
+        rlAssertGrep '- "rhts-report_example_output.txt"' $rlRun_LOG
+        rlAssertGrep 'name: "/rhts-report"' $rlRun_LOG
+        rlAssertGrep 'result: "pass"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
         rlRun "rhts-report-result rhts-report WARN /tmp/example_output.txt 66" 0 "Generating RHTS report of warned test."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep 'result: warn' $rlRun_LOG
+        rlAssertGrep 'result: "warn"' $rlRun_LOG
 	rlRun "rm -f $RESULT_FILE" 0 "Result report successfully deleted."
         rlRun "rhts-report-result rhts-report FAIL /tmp/example_output.txt 66" 0 "Generating RHTS report of failed test."
         rlRun -s "cat $RESULT_FILE"
-        rlAssertGrep 'result: fail' $rlRun_LOG
+        rlAssertGrep 'result: "fail"' $rlRun_LOG
     rlPhaseEnd
 
     rlPhaseStartCleanup

diff --git a/tests/execute/result/special.sh b/tests/execute/result/special.sh
@@ -0,0 +1,26 @@
+#!/bin/bash
+. /usr/share/beakerlib/beakerlib.sh || exit 1
+
+rlJournalStart
+    rlPhaseStartSetup
+        rlRun "run=\$(mktemp -d)" 0 "Create run directory"
+        rlRun "pushd special"
+    rlPhaseEnd
+
+    rlPhaseStartTest "Check characters are correctly escaped in tmt-report-result output"
+        rlRun -s "tmt run -v -i $run" 0
+
+        RESULT_FILE="$run/special-chars/execute/data/guest/default-0/0-7-special-characters-in-the-name-1/data/tmt-report-results.yaml"
+        rlRun "yq -ye '.' $RESULT_FILE" 0 "Check the YAML is valid"
+
+        rlAssertGrep "name: \"/0\.\.7 special: characters: \*\$\@|&>< in: the: name\"" "$RESULT_FILE"
+        rlAssertGrep "result: \"pass\"" "$RESULT_FILE"
+        rlAssertGrep "end-time: \".*\"" "$RESULT_FILE"
+        rlAssertGrep "data-path: \"/.*\"" "$RESULT_FILE"
+    rlPhaseEnd
+
+    rlPhaseStartCleanup
+        rlRun "popd"
+        rlRun "rm -r ${run}" 0 "Remove run directory"
+    rlPhaseEnd
+rlJournalEnd
diff --git a/tests/execute/result/special/.fmf/version b/tests/execute/result/special/.fmf/version
@@ -0,0 +1 @@
+1
diff --git a/tests/execute/result/special/special-chars.fmf b/tests/execute/result/special/special-chars.fmf
@@ -0,0 +1,9 @@
+execute:
+    how: tmt
+discover:
+    how: shell
+    tests:
+      - name: '0..7 special: characters: *$@|&>< in: the: name'
+        test: "tmt-report-result '0..7 special: characters: *$@|&>< in: the: name' PASS"
+provision:
+    how: container
diff --git a/tmt/steps/execute/scripts/tmt-report-result b/tmt/steps/execute/scripts/tmt-report-result
@@ -10,14 +10,14 @@ die() { echo "$*" >&2; exit 2; }  # complain to STDERR and exit with error
 needs_arg() { if [ -z "$OPTARG" ]; then die "No arg for --$OPT option"; fi; }
 
 write_report_file () {
-    echo "  - name: ${TESTNAME}" >> "$REPORT_RESULT_OUTPUTFILE"
-    echo "    result: ${TESTRESULT}" >> "$REPORT_RESULT_OUTPUTFILE"
+    echo "  - name: \"${TESTNAME}\"" >> "$REPORT_RESULT_OUTPUTFILE"
+    echo "    result: \"${TESTRESULT}\"" >> "$REPORT_RESULT_OUTPUTFILE"
     if [ -n "${outputFile}" ]; then
         echo "    log:" >> "$REPORT_RESULT_OUTPUTFILE"
-        echo "      - ${outputFile}" >> "$REPORT_RESULT_OUTPUTFILE"
+        echo "      - \"${outputFile}\"" >> "$REPORT_RESULT_OUTPUTFILE"
     fi
     echo "    end-time: \"$(date --utc +%FT%T.%6N%:z)\"" >> "$REPORT_RESULT_OUTPUTFILE"
-    echo "    data-path: ${TMT_TEST_DATA/$TMT_TEST_EXE}" >> "$REPORT_RESULT_OUTPUTFILE"
+    echo "    data-path: \"${TMT_TEST_DATA/$TMT_TEST_EXE}\"" >> "$REPORT_RESULT_OUTPUTFILE"
 }
 
 copy_outputfile_to_data_dir () {