diff options
author | Johannes Schindelin <johannes.schindelin@gmx.de> | 2022-07-14 12:00:34 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-07-14 20:02:06 +0300 |
commit | 7253f7ca9fdc5a19f2a9ee5867a20182e10551d6 (patch) | |
tree | 91635b77458abaab61509962ff09fb94f4152d50 /t/test-lib-junit.sh | |
parent | cadcafc3311de7f6fae5f3add10cde4f93268ff8 (diff) |
tests: fix incorrect --write-junit-xml code
In 78d5e4cfb4b (tests: refactor --write-junit-xml code, 2022-05-21),
this developer refactored the `--write-junit-xml` code a bit, including
the part where the current test case's title was used in a `set`
invocation, but failed to account for the fact that some test cases'
titles start with a long option, which the `set` misinterprets as being
intended for parsing.
Let's fix this by using the `set -- <...>` form.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/test-lib-junit.sh')
-rw-r--r-- | t/test-lib-junit.sh | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/t/test-lib-junit.sh b/t/test-lib-junit.sh index c959183c7e..79c31c788b 100644 --- a/t/test-lib-junit.sh +++ b/t/test-lib-junit.sh @@ -46,7 +46,7 @@ finalize_test_case_output () { shift case "$test_case_result" in ok) - set "$*" + set -- "$*" ;; failure) junit_insert="<failure message=\"not ok $test_count -" @@ -65,17 +65,17 @@ finalize_test_case_output () { junit_insert="$junit_insert<system-err>$(xml_attr_encode \ "$(cat "$GIT_TEST_TEE_OUTPUT_FILE")")</system-err>" fi - set "$1" " $junit_insert" + set -- "$1" " $junit_insert" ;; fixed) - set "$* (breakage fixed)" + set -- "$* (breakage fixed)" ;; broken) - set "$* (known breakage)" + set -- "$* (known breakage)" ;; skip) message="$(xml_attr_encode --no-lf "$skipped_reason")" - set "$1" " <skipped message=\"$message\" />" + set -- "$1" " <skipped message=\"$message\" />" ;; esac |