fix: gracefully handle non ctrf file

ctrf-io · Sep 5, 2024 · 64a283b · 64a283b
1 parent 0cd5c33
commit 64a283b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ctrf",
-  "version": "0.0.9",
+  "version": "0.0.10",
   "description": "",
   "main": "index.js",
   "scripts": {

diff --git a/src/merge.ts b/src/merge.ts
@@ -13,15 +13,23 @@ export async function mergeReports(directory: string, output: string, outputDir:
         const files = fs.readdirSync(directoryPath);
 
         files.forEach((file) => {
-            console.log('Found CTRF report file:', file);
+            console.log('Found file:', file);
         });
 
         const ctrfReportFiles = files.filter((file) => {
             try {
+                if (path.extname(file) !== '.json') {
+                    console.log(`Skipping non-CTRF file: ${file}`);
+                    return false;
+                }
                 const filePath = path.join(directoryPath, file);
                 const fileContent = fs.readFileSync(filePath, 'utf8');
                 const jsonData = JSON.parse(fileContent);
-                return jsonData.hasOwnProperty('results');
+                if (!jsonData.hasOwnProperty('results')) {
+                    console.log(`Skipping non-CTRF file: ${file}`);
+                    return false;
+                }
+                return true;
             } catch (error) {
                 console.error(`Error reading JSON file '${file}':`, error);
                 return false;