move rules to more specific override

Author: Lms24

packages/eslint-config-sdk/src/index.js

@@ -167,7 +167,7 @@ module.exports = {
       },
     },
     {
-      // Configuration for test files
+      // Configuration for files in test directories
       env: {
         jest: true,
       },
@@ -195,6 +195,23 @@ module.exports = {
         "jest/no-disabled-tests": "error",
       },
     },
+    {
+      // Configuration only for test files (this won't apply to utils or other files in test directories)
+      env: {
+        jest: true,
+      },
+      files: ['test.ts', '*.test.ts', '*.test.tsx', '*.test.js', '*.test.jsx'],
+      rules: {
+        // Prevent permanent usage of `it.only`, `fit`, `test.only` etc
+        // We want to avoid debugging leftovers making their way into the codebase
+        "jest/no-focused-tests": "error",
+
+        // Prevent permanent usage of `it.skip`, `xit`, `test.skip` etc
+        // We want to avoid debugging leftovers making their way into the codebase
+        // If there's a good reason to skip a test (e.g. bad flakiness), just add an ignore comment
+        "jest/no-disabled-tests": "error",
+      },
+    },
     {
       // Configuration for config files like webpack/rollup
       files: ['*.config.js'],