## 1.9.0 - April 2022

* Fixes #16, Unable to use both "ignore" and "inspect" inside "files" property

Author: gmullerb

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 * Closes #15, Allowing mixing different error level.
   * This bug required that the same rule can be defined with different error levels, that is not possible at the moment on eslint. Error level is not available at plugin level (as indicate by ESLint official documentation: "Keep in mind that the error level is not part of context.options, as the error level cannot be known or modified from inside a rule"). So I came up a different approach, which not only **allows to use "the same" rule with different error levels**, but also **allows Mixing custom set of regex rules**, this is useful for creating package/libraries/plugins with your own set of rules and sharing it with others.
+* Fixes #16, Unable to use both "ignore" and "inspect" inside "files" property.
 * Fixes an issue with remaining source calculation and minified files.
 * Improves project configuration.
   * Now uses [`any-eslint-parser`](https://www.npmjs.com/package/any-eslint-parser)

lib/rules/invalid-regex-rule.js

@@ -227,7 +227,7 @@ module.exports = {
                 }
               },
               minProperties: 1,
-              maxProperties: 1
+              maxProperties: 2
             }
           },
           required: ['regex']

lib/rules/required-regex-rule.js

@@ -76,7 +76,7 @@ module.exports = {
                 }
               },
               minProperties: 1,
-              maxProperties: 1
+              maxProperties: 2
             }
           },
           required: ['regex']

tests/lib/rules/invalid-regex-detailed-rule.e2e-test.js

@@ -43,6 +43,20 @@ const shouldNotInspectedFile = {
   ]
 }
 
+const shouldHandleBothIgnoreInspectFile = {
+  code: 'const z = 1\nvar x = "invalid"',
+  filename: 'some.test.js',
+  options: [
+    [{
+      regex: 'invalid',
+      files: {
+        ignore: '.*test\.js',
+        inspect: '.*spec\.js'
+      }
+    }]
+  ]
+}
+
 const shouldFound = {
   code: 'var z = 1\nvar x = "invalid"',
   filename: 'some.js',
@@ -621,7 +635,8 @@ ruleTester.run(
     valid: [
       shouldNotFind,
       shouldIgnoreFile,
-      shouldNotInspectedFile
+      shouldNotInspectedFile,
+      shouldHandleBothIgnoreInspectFile
     ],
     invalid: [
       shouldFound,

tests/lib/rules/required-regex-detailed-rule.e2e-test.js

@@ -34,6 +34,33 @@ const shouldIgnoreFile = {
   ]
 }
 
+const shouldNotInspectFile = {
+  code: 'var z = 1\nvar x = "valid"',
+  filename: 'some.test.js',
+  options: [
+    [{
+      regex: 'required',
+      files: {
+        inspect: '.*spec\.js'
+      }
+    }],
+  ]
+}
+
+const shouldHandleBothIgnoreInspectFile = {
+  code: 'var z = 1\nvar x = "valid"',
+  filename: 'some.test.js',
+  options: [
+    [{
+      regex: 'required',
+      files: {
+        ignore: '.*test\.js',
+        inspect: '.*spec\.js'
+      }
+    }],
+  ]
+}
+
 const shouldFound = {
   code: 'var z = 1\nvar x = "required"',
   filename: 'some.js',
@@ -60,7 +87,9 @@ ruleTester.run(
     valid: [
       shouldIgnoreFile,
       shouldFound,
-      shouldFoundAcrossLines
+      shouldFoundAcrossLines,
+      shouldNotInspectFile,
+      shouldHandleBothIgnoreInspectFile
     ],
     invalid: [
       shouldNotFind,