From 7009a10c1c9cb96faaabd694d7fdd41caf9f23f1 Mon Sep 17 00:00:00 2001
From: Jozef Izso <jizso@slido.com>
Date: Mon, 18 Dec 2023 13:10:06 +0100
Subject: [PATCH] Add tests for results with React component names

---
 .../jest-react-component-test-results.md      | 10 +++++++
 .../__snapshots__/jest-junit.test.ts.snap     | 26 +++++++++++++++++++
 .../jest-react-component-test-results.xml     |  7 +++++
 __tests__/jest-junit.test.ts                  | 23 ++++++++++++++++
 dist/index.js                                 |  5 +++-
 5 files changed, 70 insertions(+), 1 deletion(-)
 create mode 100644 __tests__/__outputs__/jest-react-component-test-results.md
 create mode 100644 __tests__/fixtures/external/jest/jest-react-component-test-results.xml

diff --git a/__tests__/__outputs__/jest-react-component-test-results.md b/__tests__/__outputs__/jest-react-component-test-results.md
new file mode 100644
index 00000000..7891a19b
--- /dev/null
+++ b/__tests__/__outputs__/jest-react-component-test-results.md
@@ -0,0 +1,10 @@
+![Tests passed successfully](https://img.shields.io/badge/tests-1%20passed-success)
+## ✅ <a id="user-content-r0" href="#r0">fixtures/external/jest/jest-react-component-test-results.xml</a>
+**1** tests were completed in **1000ms** with **1** passed, **0** failed and **0** skipped.
+|Test suite|Passed|Failed|Skipped|Time|
+|:---|---:|---:|---:|---:|
+|[\<Component /\>](#r0s0)|1✅|||798ms|
+### ✅ <a id="user-content-r0s0" href="#r0s0">\<Component /\></a>
+```
+✅ <Component /> should render properly
+```
\ No newline at end of file
diff --git a/__tests__/__snapshots__/jest-junit.test.ts.snap b/__tests__/__snapshots__/jest-junit.test.ts.snap
index 3a04722a..b9cfbd32 100644
--- a/__tests__/__snapshots__/jest-junit.test.ts.snap
+++ b/__tests__/__snapshots__/jest-junit.test.ts.snap
@@ -1,5 +1,31 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`jest-junit tests report from #235 testing react components named <ComponentName /> 1`] = `
+TestRunResult {
+  "path": "fixtures/external/jest/jest-react-component-test-results.xml",
+  "suites": Array [
+    TestSuiteResult {
+      "groups": Array [
+        TestGroupResult {
+          "name": "",
+          "tests": Array [
+            TestCaseResult {
+              "error": undefined,
+              "name": "<Component /> should render properly",
+              "result": "success",
+              "time": 704,
+            },
+          ],
+        },
+      ],
+      "name": "\\\\<Component /\\\\>",
+      "totalTime": 798,
+    },
+  ],
+  "totalTime": 1000,
+}
+`;
+
 exports[`jest-junit tests report from ./reports/jest test results matches snapshot 1`] = `
 TestRunResult {
   "path": "fixtures/jest-junit.xml",
diff --git a/__tests__/fixtures/external/jest/jest-react-component-test-results.xml b/__tests__/fixtures/external/jest/jest-react-component-test-results.xml
new file mode 100644
index 00000000..cdaa87f1
--- /dev/null
+++ b/__tests__/fixtures/external/jest/jest-react-component-test-results.xml
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuites name="React components test" tests="1" failures="0" errors="0" time="1.0">
+  <testsuite name="&lt;Component /&gt;" errors="0" failures="0" skipped="0" timestamp="2021-01-24T19:21:45" time="0.798" tests="1">
+    <testcase classname="" name="&lt;Component /&gt; should render properly" time="0.704">
+    </testcase>
+  </testsuite>
+</testsuites>
diff --git a/__tests__/jest-junit.test.ts b/__tests__/jest-junit.test.ts
index b6533e62..d42405cf 100644
--- a/__tests__/jest-junit.test.ts
+++ b/__tests__/jest-junit.test.ts
@@ -82,4 +82,27 @@ describe('jest-junit tests', () => {
     fs.mkdirSync(path.dirname(outputPath), {recursive: true})
     fs.writeFileSync(outputPath, report)
   })
+
+  it('report from #235 testing react components named <ComponentName />', async () => {
+    const fixturePath = path.join(__dirname, 'fixtures', 'external', 'jest', 'jest-react-component-test-results.xml')
+    const trackedFilesPath = path.join(__dirname, 'fixtures', 'external', 'jest', 'files.txt')
+    const outputPath = path.join(__dirname, '__outputs__', 'jest-react-component-test-results.md')
+    const filePath = normalizeFilePath(path.relative(__dirname, fixturePath))
+    const fileContent = fs.readFileSync(fixturePath, {encoding: 'utf8'})
+
+    const trackedFiles = fs.readFileSync(trackedFilesPath, {encoding: 'utf8'}).split(/\n\r?/g)
+    const opts: ParseOptions = {
+      parseErrors: true,
+      trackedFiles
+      //workDir: '/home/dorny/dorny/jest/'
+    }
+
+    const parser = new JestJunitParser(opts)
+    const result = await parser.parse(filePath, fileContent)
+    expect(result).toMatchSnapshot()
+
+    const report = getReport([result])
+    fs.mkdirSync(path.dirname(outputPath), {recursive: true})
+    fs.writeFileSync(outputPath, report)
+  })
 })
diff --git a/dist/index.js b/dist/index.js
index f64acacd..250df1eb 100644
--- a/dist/index.js
+++ b/dist/index.js
@@ -1212,7 +1212,7 @@ class JestJunitParser {
         const suites = junit.testsuites.testsuite === undefined
             ? []
             : junit.testsuites.testsuite.map(ts => {
-                const name = ts.$.name.trim();
+                const name = this.escapeCharacters(ts.$.name.trim());
                 const time = parseFloat(ts.$.time) * 1000;
                 const sr = new test_results_1.TestSuiteResult(name, this.getGroups(ts), time);
                 return sr;
@@ -1281,6 +1281,9 @@ class JestJunitParser {
         var _a, _b;
         return ((_b = (_a = this.options.workDir) !== null && _a !== void 0 ? _a : this.assumedWorkDir) !== null && _b !== void 0 ? _b : (this.assumedWorkDir = (0, path_utils_1.getBasePath)(path, this.options.trackedFiles)));
     }
+    escapeCharacters(s) {
+        return s.replace(/([<>])/g, '\\$1');
+    }
 }
 exports.JestJunitParser = JestJunitParser;