Add warning for invalid nunit xml files (#286)

* Allow ResultsParser to fail on non-nunit xml files * yarn build * Add file check for NUnit XML format before parsing * yarn build * Update src/model/results-check.ts Co-authored-by: Koji Hasegawa <[email protected]> --------- Co-authored-by: Koji Hasegawa <[email protected]>
game-ci · Nov 8, 2024 · 0483262 · 0483262
1 parent 05a00ef
commit 0483262
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 7 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/model/results-check.ts b/src/model/results-check.ts
@@ -22,9 +22,18 @@ const ResultsCheck = {
       files.map(async filepath => {
         if (!filepath.endsWith('.xml')) return;
         core.info(`Processing file ${filepath}...`);
-        const fileData = await ResultsParser.parseResults(path.join(artifactsPath, filepath));
-        core.info(fileData.summary);
-        runs.push(fileData);
+        try {
+          const content = fs.readFileSync(path.join(artifactsPath, filepath), 'utf8');
+          if (!content.includes('<test-run')) {
+            // noinspection ExceptionCaughtLocallyJS
+            throw new Error('File does not appear to be a NUnit XML file');
+          }
+          const fileData = await ResultsParser.parseResults(path.join(artifactsPath, filepath));
+          core.info(fileData.summary);
+          runs.push(fileData);
+        } catch (error: any) {
+          core.warning(`Failed to parse ${filepath}: ${error.message}`);
+        }
       }),
     );