Finish tests

eslint · Feb 9, 2021 · 183f9e9 · 183f9e9
1 parent ee3157e
commit 183f9e9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 11 deletions.
diff --git a/lib/config-array/ignore-pattern.js b/lib/config-array/ignore-pattern.js
@@ -167,7 +167,7 @@ class IgnorePattern {
                 const relPath = relPathRaw && (relPathRaw + dirSuffix(filePath));
                 const adoptedIg = dot ? dotIg : ig;
                 const result = relPath !== "" && adoptedIg.ignores(relPath);
-
+                
                 debug("Check", { filePath, dot, relativePath: relPath, result });
                 return result;
             },

diff --git a/lib/flat-compat.js b/lib/flat-compat.js
@@ -153,7 +153,7 @@ function translateESLintRC(eslintrcConfig, {
             flatConfig.plugins[pluginName] = plugin;
 
             // create a config for any processors
-            if (plugin.processors) { console.dir(pluginProcessors);
+            if (plugin.processors) {
                 for (const processorName of Object.keys(plugin.processors)) {
                     if (processorName.startsWith(".")) {
                         debug(`Assigning processor: ${pluginName}/${processorName}`);
@@ -262,7 +262,6 @@ class FlatCompat {
                     // This filters config array elements by `files`/`excludedFiles` then merges the elements.
                     const finalConfig = eslintrcArray.extractConfig(filePath);
 
-
                     // Test the `ignorePattern` properties of the final config.
                     return Boolean(finalConfig.ignores) && finalConfig.ignores(filePath);
                 }]

diff --git a/tests/lib/flat-compat.js b/tests/lib/flat-compat.js
@@ -74,8 +74,8 @@ describe("FlatCompat", () => {
 
                     assert.strictEqual(result.length, 1);
                     assert.typeOf(result[0].ignores[0], "function");
-                    assert.isTrue(result[0].ignores[0]("/usr/eslint/foo.jsx"));
-                    assert.isFalse(result[0].ignores[0]("/usr/eslint/foo.js"));
+                    assert.isTrue(result[0].ignores[0](path.join(baseDirectory, "foo.jsx")));
+                    assert.isFalse(result[0].ignores[0](path.join(baseDirectory, "foo.js")));
                 });
 
                 it("should translate ignorePatterns array into ignores array", () => {
@@ -85,8 +85,8 @@ describe("FlatCompat", () => {
 
                     assert.strictEqual(result.length, 1);
                     assert.typeOf(result[0].ignores[0], "function");
-                    assert.isTrue(result[0].ignores[0]("/usr/eslint/foo.jsx"));
-                    assert.isFalse(result[0].ignores[0]("/usr/eslint/foo.js"));
+                    assert.isTrue(result[0].ignores[0](path.join(baseDirectory, "foo.jsx")));
+                    assert.isFalse(result[0].ignores[0](path.join(baseDirectory, "foo.js")));
                 });
 
                 it("should ignore second argument of ignore function from ignorePatterns", () => {
@@ -96,8 +96,8 @@ describe("FlatCompat", () => {
 
                     assert.strictEqual(result.length, 1);
                     assert.typeOf(result[0].ignores[0], "function");
-                    assert.isTrue(result[0].ignores[0]("/usr/eslint/foo.jsx", {}));
-                    assert.isFalse(result[0].ignores[0]("/usr/eslint/foo.js", ""));
+                    assert.isTrue(result[0].ignores[0](path.join(baseDirectory, "foo.jsx"), {}));
+                    assert.isFalse(result[0].ignores[0](path.join(baseDirectory, "foo.js"), ""));
                 });
 
                 it("should combine ignorePatterns from extended configs", () => {
@@ -108,8 +108,8 @@ describe("FlatCompat", () => {
 
                     assert.strictEqual(result.length, 1);
                     assert.typeOf(result[0].ignores[0], "function");
-                    assert.isTrue(result[0].ignores[0]("/usr/eslint/foo/baz.js"));
-                    assert.isFalse(result[0].ignores[0]("/usr/eslint/foo/bar/baz.js"));
+                    assert.isTrue(result[0].ignores[0](path.join(baseDirectory, "foo/baz.js")));
+                    assert.isFalse(result[0].ignores[0](path.join(baseDirectory, "foo/bar/baz.js")));
                 });