Ported test suite: infer-plugins-ext-dir (#34)

- Relative plugins in nested configurations are resolved from the CWD at the moment, instead than from the folder the config file was found in, it should be fixed.

Author: 43081j

.gitignore

@@ -12,3 +12,4 @@ Thumbs.db
 
 /dist
 /node_modules
+/test/__fixtures__/.temp-*

package-lock.json

@@ -40,7 +40,7 @@
     "find-up-json": "^2.0.5",
     "function-once": "^3.0.1",
     "import-meta-resolve": "^4.1.0",
-    "is-binary-path": "^3.0.0",
+    "is-binary-path": "^2.0.0",
     "js-yaml": "^4.1.0",
     "json-sorted-stringify": "^1.0.1",
     "json5": "^2.2.3",
@@ -54,7 +54,7 @@
     "tiny-readdir": "^2.7.4",
     "tiny-readdir-glob": "^1.23.2",
     "tiny-spinner": "^2.0.5",
-    "worktank": "^2.7.3",
+    "worktank": "^3.0.2",
     "zeptomatch": "^2.0.1",
     "zeptomatch-escape": "^1.0.1",
     "zeptomatch-is-static": "^1.0.1"

package.json

@@ -1,14 +1,20 @@
 import os from "node:os";
+import process from "node:process";
 import WorkTank from "worktank";
 import { resolve } from "./utils.js";
 import type { Options, Prettier } from "./types.js";
 
 function makeParallel(options: Options): Prettier {
   const pool = new WorkTank<Prettier>({
-    name: "prettier",
-    size: options.parallelWorkers || Math.max(1, os.cpus().length - 1),
-    methods: new URL("./prettier_serial.js", import.meta.url),
-    warmup: true,
+    pool: {
+      name: "prettier",
+      size: options.parallelWorkers || Math.max(1, os.cpus().length - 1),
+    },
+    worker: {
+      autoInstantiate: true,
+      env: process.env,
+      methods: new URL("./prettier_serial.js", import.meta.url),
+    },
   });
 
   return {

src/prettier_parallel.ts

@@ -277,8 +277,15 @@ async function getTargetsPaths(
   }
 
   const filesExplicitPaths = targetFiles;
-  const filesFoundPaths = globResult.filesFound;
-  const foldersFoundPaths = [rootPath, ...globResult.directoriesFound];
+
+  const globFilesFoundPaths = globResult.filesFound;
+  const directoryFilesFoundPaths = directoriesResultsFiles.flat();
+  const filesFoundPaths = [...globFilesFoundPaths, ...directoryFilesFoundPaths];
+
+  const globFoldersFoundPaths = globResult.directoriesFound;
+  const directoryFoldersFoundPaths = directoriesResults.flatMap((result) => result.directories);
+  const foldersFoundPaths = [rootPath, ...globFoldersFoundPaths, ...directoryFoldersFoundPaths];
+
   return [filesPaths, filesNames, filesNamesToPaths, filesExplicitPaths, filesFoundPaths, foldersFoundPaths];
 }
 

src/utils.ts

@@ -0,0 +1,10 @@
+export default {
+  overrides: [
+    {
+      files: ["dir/*.foo"],
+      options: {
+        plugins: ["../plugin-extensions/plugin.cjs"]
+      }
+    },
+  ]
+};

test/__fixtures__/infer-plugins-ext-dir-with-complex-overrides/.prettierrc.mjs

@@ -0,0 +1 @@
+1.foo

test/__fixtures__/infer-plugins-ext-dir-with-complex-overrides/1.foo

@@ -0,0 +1 @@
+2.foo

test/__fixtures__/infer-plugins-ext-dir-with-complex-overrides/dir/2.foo

@@ -0,0 +1,3 @@
+export default {
+  plugins: ["../plugin-extensions/plugin.cjs"],
+};

test/__fixtures__/infer-plugins-ext-dir-with-config/foo.mjs

@@ -0,0 +1 @@
+contents