test: add the progress so far

Author: pralkarz

src/config_editorconfig.ts

@@ -10,7 +10,7 @@ const getEditorConfig = memoize((folderPath: string, filesNames: string[]): Prom
   for (let i = 0, l = filesNames.length; i < l; i++) {
     const fileName = filesNames[i];
     const filePath = fastJoinedPath(folderPath, fileName);
-    if (!Known.hasFilePath(filePath)) continue;
+    // if (!Known.hasFilePath(filePath)) continue;
     return fs.readFile(filePath, "utf8").then(EditorConfig.parse).catch(noop);
   }
 });

src/index.ts

@@ -32,7 +32,11 @@ async function runStdin(options: Options, pluginsDefaultOptions: PluginsOptions,
   const fileContent = (await getStdin()) || "";
 
   try {
-    const formatted = await prettier.format(fileName, fileContent, options.formatOptions, options.contextOptions, pluginsDefaultOptions, pluginsCustomOptions);
+    const editorConfigNames = options.editorConfig ? [".editorconfig"] : [];
+    const editorConfig = options.editorConfig ? getEditorConfigFormatOptions(await getEditorConfigResolved(fileName, editorConfigNames)) : {};
+    const formatOptions = { ...editorConfig, ...options.formatOptions };
+
+    const formatted = await prettier.format(fileName, fileContent, formatOptions, options.contextOptions, pluginsDefaultOptions, pluginsCustomOptions);
     if (options.check || options.list) {
       if (formatted !== fileContent) {
         stdout.warn("(stdin)");

src/known.ts

@@ -2,7 +2,7 @@
 
 class Known {
   private filesPaths: Set<string> = new Set();
-  private filesNames: Set<string> = new Set();
+  filesNames: Set<string> = new Set();
 
   addFilesPaths = (filesPaths: Array<string> | Set<string>): void => {
     if (!this.filesPaths.size) {

test/__tests__/__snapshots__/stdin-filepath.js.snap

@@ -40,15 +40,15 @@ exports[`apply editorconfig for stdin-filepath with nonexistent file (stdout) 1`
 
 exports[`apply editorconfig for stdin-filepath with nonexistent file (write) 1`] = `[]`;
 
-exports[`don’t apply editorconfig outside project for stdin-filepath with nonexistent directory (stderr) 1`] = `""`;
+exports[`don't apply editorconfig outside project for stdin-filepath with nonexistent directory (stderr) 1`] = `""`;
 
-exports[`don’t apply editorconfig outside project for stdin-filepath with nonexistent directory (stdout) 1`] = `
+exports[`don't apply editorconfig outside project for stdin-filepath with nonexistent directory (stdout) 1`] = `
 "function f() {
-  console.log("should be indented with 2 spaces")
+  console.log("should be indented with 2 spaces");
 }"
 `;
 
-exports[`don’t apply editorconfig outside project for stdin-filepath with nonexistent directory (write) 1`] = `[]`;
+exports[`don't apply editorconfig outside project for stdin-filepath with nonexistent directory (write) 1`] = `[]`;
 
 exports[`format correctly if stdin content compatible with stdin-filepath (stderr) 1`] = `""`;
 

test/__tests__/stdin-filepath.js

@@ -32,7 +32,7 @@ describe("gracefully handle stdin-filepath with nonexistent directory", () => {
 });
 
 describe("apply editorconfig for stdin-filepath with nonexistent file", () => {
-  runCli("", ["--stdin-filepath", "editorconfig/nonexistent.js"], {
+  runCli("editorconfig", ["--stdin-filepath", "nonexistent.js"], {
     input: dedent`
       function f() {
         console.log("should be indented with a tab");
@@ -45,8 +45,8 @@ describe("apply editorconfig for stdin-filepath with nonexistent file", () => {
 
 describe("apply editorconfig for stdin-filepath with nonexistent directory", () => {
   runCli(
-    "",
-    ["--stdin-filepath", "editorconfig/nonexistent/one/two/three.js"],
+    "editorconfig",
+    ["--stdin-filepath", "nonexistent/one/two/three.js"],
     {
       input: dedent`
         function f() {
@@ -61,8 +61,8 @@ describe("apply editorconfig for stdin-filepath with nonexistent directory", ()
 
 describe("apply editorconfig for stdin-filepath with a deep path", () => {
   runCli(
-    "",
-    ["--stdin-filepath", "editorconfig/" + "a/".repeat(30) + "three.js"],
+    "editorconfig",
+    ["--stdin-filepath", "a/".repeat(30) + "three.js"],
     {
       input: dedent`
         function f() {
@@ -75,6 +75,8 @@ describe("apply editorconfig for stdin-filepath with a deep path", () => {
   });
 });
 
+// TODO: This is currently a false positive as no config actually gets resolved, but Prettier
+// somehow formats the input correctly anyway.
 describe("apply editorconfig for stdin-filepath in root", () => {
   const code = dedent`
     function f() {
@@ -93,8 +95,8 @@ describe("apply editorconfig for stdin-filepath in root", () => {
 
 describe("apply editorconfig for stdin-filepath with a deep path", () => {
   runCli(
-    "",
-    ["--stdin-filepath", "editorconfig/" + "a/".repeat(30) + "three.js"],
+    "editorconfig",
+    ["--stdin-filepath", "a/".repeat(30) + "three.js"],
     {
       input: dedent`
         function f() {
@@ -107,7 +109,9 @@ describe("apply editorconfig for stdin-filepath with a deep path", () => {
   });
 });
 
-describe("don’t apply editorconfig outside project for stdin-filepath with nonexistent directory", () => {
+// TODO: This is currently a false positive. Gotta investigate how it's handled in Prettier v3 to
+// gauge the expected behavior.
+describe("don't apply editorconfig outside project for stdin-filepath with nonexistent directory", () => {
   runCli(
     "",
     [