jasmine-focused-test-detector 1.0.2 → 1.0.4

package/.idea/workspace.xml

@@ -46,7 +46,7 @@
       <option name="number" value="Default" />
       <option name="presentableId" value="Default" />
       <updated>1767089273794</updated>
-      <workItem from="1767089274873" duration="3921000" />
+      <workItem from="1767089274873" duration="7676000" />
     </task>
     <servers />
   </component>

package/index.js

@@ -1,9 +1,9 @@
 #!/usr/bin/env node
-import fs from 'fs';
-import {globby} from 'globby';
-import {hideBin} from 'yargs/helpers';
+
+import { hideBin } from 'yargs/helpers';
 import yargs from 'yargs/yargs';
-import { parse } from '@typescript-eslint/parser';
+import { ESLint } from 'eslint';
+import { globby } from 'globby';
 
 const argv = yargs(hideBin(process.argv))
     .option('files', {
@@ -13,7 +13,7 @@ const argv = yargs(hideBin(process.argv))
         demandOption: true
     })
     .help()
-    .argv;
+    .parse();
 
 const filePatterns = argv.files.split(',').map(f => f.trim());
 
@@ -25,44 +25,34 @@ const filePatterns = argv.files.split(',').map(f => f.trim());
         process.exit(0);
     }
 
-    let found = false;
-    const focusedFiles = [];
-
-    function visit(node, filePath) {
-        if (
-            node.type === 'CallExpression' &&
-            node.callee.type === 'Identifier' &&
-            (node.callee.name === 'fit' || node.callee.name === 'fdescribe')
-        ) {
-            found = true;
-            console.log(filePath)
-            if (!focusedFiles.includes(filePath)) focusedFiles.push(filePath);
-        }
-        for (const key in node) {
-            if (node[key] && typeof node[key] === 'object') {
-                visit(node[key], filePath);
+    const eslint = new ESLint({
+        baseConfig: {
+            plugins: ['jasmine'],
+            env: {
+                jasmine: true,
+                node: true
+            },
+            rules: {
+                'jasmine/no-focused-tests': 'error'
             }
-        }
-    }
+        },
+        useEslintrc: false
+    });
 
-    for (const filePath of files) {
-        const code = fs.readFileSync(filePath, 'utf8');
-        let ast;
-        try {
-            ast = parse(code, { sourceType: 'module', ecmaVersion: 2020 });
-        } catch (err) {
-            console.warn(`Warning: Failed to parse ${filePath}: ${err.message}`);
-            continue;
-        }
-        visit(ast, filePath);
-    }
+    const results = await eslint.lintFiles(files);
+
+    const focusedFiles = results
+        .filter(r => r.messages.some(m => m.ruleId === 'jasmine/no-focused-tests'))
+        .map(r => r.filePath);
 
-    if (found) {
+    if (focusedFiles.length > 0) {
         console.error('\n==========================================');
         console.error('BUILD FAILED: Found focused tests (fit/fdescribe)');
-        console.error('==========================================');
+        console.error('Please remove all fit/fdescribe calls before committing.');
+        console.error('==========================================\n');
         focusedFiles.forEach(f => console.error(`ERROR: Focused test found in ${f}`));
-        console.error('Please remove all fit/fdescribe calls before committing.\n');
+        console.error('==========================================\n');
+
         process.exit(1);
     }
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "jasmine-focused-test-detector",
-  "version": "1.0.2",
-  "description": "Detect focused tests (fit/fdescribe) in Jasmine test files",
+  "version": "1.0.4",
+  "description": "CLI tool to detect focused tests (fit/fdescribe/it.only/describe.only) in Jasmine test files using ESLint",
   "main": "index.js",
   "type": "module",
   "bin": {
@@ -14,9 +14,10 @@
   "author": "Pouya Sadat Alhosseini",
   "license": "MIT",
   "dependencies": {
-    "@typescript-eslint/parser": "^6.2.0",
     "globby": "^14.1.0",
-    "yargs": "^17.7.2"
+    "yargs": "^17.7.2",
+    "eslint": "^8.50.0",
+    "eslint-plugin-jasmine": "^4.0.0"
   },
   "repository": {
     "type": "git",
@@ -27,6 +28,7 @@
     "frontend",
     "jasmine",
     "test",
-    "cli"
+    "cli",
+    "eslint"
   ]
 }