fix: remove invalid xml characters in junit reporter output (fix #1144)

[ciddan]

This line in data.ts contains two control characters, visible in vs code:

That line is scrubbed by the new removeInvalidXMLCharacters function, ensuring that the XML output doesn't contain the control characters.

Reproduction

Reproduction StackBlitz.

Let the tests run and open test_result/junit.xml. The file contains disallowed characters, specifically in the tag.

You can use any XML validator to check, such as the one from w3schools.

[netlify]

✅ Deploy Preview for vitest-dev ready!

Name	Link
🔨 Latest commit	e4d9dec
🔍 Latest deploy log	https://app.netlify.com/sites/vitest-dev/deploys/6255e5d6ad3c4c0008c499ab
😎 Deploy Preview	https://deploy-preview-1145--vitest-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[dheardal]

I'm still seeing issues with this from the output of a react test:

Vitest version 0.17.0

[kaliatech]

I'm seeing similar with vitest 0.18.0. It looks as if the encoding and escaping stop at some point when processing a large failure message.

Here is a copy paste with large sections removed. It shows correct encoding/escaping at first, but then seems to stop until getting to the next test case.

<testcase classname="src/components/team-view/TeamView.test.tsx" name="Test the TeamView component &gt; Should render successfully" time="0.0180000000">
            <failure message="Found multiple elements by: [data-testid=&quot;row-a1&quot;]

Here are the matching elements:

Ignored nodes: comments, &lt;script /&gt;, &lt;style /&gt;
[36m&lt;div[39m
  [33mdata-e2etestid[39m=[32m&quot;row-a1&quot;[39m

......LOTS OF VALID ENCODING......

                [36m/&gt;[39m
                [36m&lt;path[39m
                  [33mclip-rule[39m=[32m&quot;evenodd&quot;[39m
                  [33md[39m=[32m&quot;M206.012 384.08C208.492 383.134 210.972..." type="TestingLibraryElementError">
TestingLibraryElementError: Found multiple elements by: [data-testid="row-a1"]

Here are the matching elements:

Ignored nodes: comments, <script />, <style />
[36m<div[39m
  [33mdata-e2etestid[39m=[32m"row-a1"[39m
  [33mdata-testid[39m=[32m"row-a1"[39m

.....NOTE HOW IT SUDDENY STOPPED ENCODING IN PREVIOUS SECTION (AND ALSO HIDDEN INVALID CHARS)....

.....LOTS OF MISSING ENCODING AND INVALID CHARACTERS...

                  �[33mclip-rule�[39m=�[32m"evenodd"�[39m
                  �[33md�[39m=�[32m"M206.012 384.08C208.492 383.134 ...
 ❯ Object.getElementError ../node_modules/@testing-library/dom/dist/config.js:40:19
 ❯ getElementError ../node_modules/@testing-library/dom/dist/query-helpers.js:25:35
 ❯ getMultipleElementsFoundError ../node_modules/@testing-library/dom/dist/query-helpers.js:29:10
 ❯ ../node_modules/@testing-library/dom/dist/query-helpers.js:66:13
 ❯ ../node_modules/@testing-library/dom/dist/query-helpers.js:111:19
 ❯ src/components/team-view/TeamView.test.tsx:34:34
            </failure>
        </testcase>

.....BUT THEN WHEN IT ENDS AND GOES TO NEXT TEST CASE, EVERYTHING IS GOOD AGAIN...