diff options
author | Johannes Schindelin <johannes.schindelin@gmx.de> | 2020-03-23 15:44:35 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-03-23 20:22:51 +0300 |
commit | d3507cc71286c669674d8e184a3796de6af0cea7 (patch) | |
tree | 77df868c25de5753917103f1e185bc81e44a756b /t/test-lib.sh | |
parent | 274b9cc25322d9ee79aa8e6d4e86f0ffe5ced925 (diff) |
tests(junit-xml): avoid invalid XML
When a test case is run in a subshell, we finalize the JUnit-style XML
when said subshell exits. But then we continue to write into that XML as
if nothing had happened.
This leads to Azure Pipelines' Publish Test Results task complaining:
Failed to read /home/vsts/work/1/s/t/out/TEST-t0000-basic.xml.
Error : Unexpected end tag. Line 110, position 5.
And indeed, the resulting XML is incorrect.
Let's "re-open" the XML in such a case, i.e. remove the previously added
closing tags.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/test-lib.sh')
-rw-r--r-- | t/test-lib.sh | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/t/test-lib.sh b/t/test-lib.sh index 0ea1e5a05e..7f0c02c016 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -1085,6 +1085,7 @@ finalize_junit_xml () { junit_time=$(test-tool date getnanos $junit_suite_start) sed -e "s/\(<testsuite.*\) time=\"[^\"]*\"/\1/" \ -e "s/<testsuite [^>]*/& time=\"$junit_time\"/" \ + -e '/^ *<\/testsuite/d' \ <"$junit_xml_path" >"$junit_xml_path.new" mv "$junit_xml_path.new" "$junit_xml_path" |