fix: pass full file path to prettier::resolveConfig, not just the dirname

Author: Xunnamius

prettier.config.js

@@ -10,7 +10,7 @@ module.exports = {
   trailingComma: 'none',
   overrides: [
     {
-      files: '**/*.@(ts|?(@(c|m))js)?(x)',
+      files: '**/*.?(@(c|m))@(ts|js)?(x)',
       options: {
         parser: 'babel-ts',
         printWidth: 90

src/formatters/prettier.ts

@@ -14,18 +14,18 @@ const debug = debugFactory('babel-plugin-tester:formatter');
 type MaybePrettierOptions = PrettierOptions | null;
 const configDirectoryCache: Record<string, MaybePrettierOptions> = Object.create(null);
 
-const getCachedConfig = (directory: string) => {
-  if (!(directory in configDirectoryCache)) {
+const getCachedConfig = (filepath: string) => {
+  if (!(filepath in configDirectoryCache)) {
+    configDirectoryCache[filepath] = resolvePrettierConfig.sync(filepath);
     debug(
-      `caching prettier configuration resolved from ${directory}: %O`,
-      configDirectoryCache[directory]
+      `caching prettier configuration resolved from ${filepath}: %O`,
+      configDirectoryCache[filepath]
     );
-    configDirectoryCache[directory] = resolvePrettierConfig.sync(directory);
   } else {
-    debug(`using cached prettier configuration resolved from ${directory}`);
+    debug(`using cached prettier configuration resolved from ${filepath}`);
   }
 
-  return configDirectoryCache[directory];
+  return configDirectoryCache[filepath];
 };
 
 export type { PrettierOptions };
@@ -58,7 +58,7 @@ export const prettierFormatter: ResultFormatter<{
     filename,
     filepath = filename || path.join(cwd, 'dummy.js'),
     config,
-    prettierOptions = config || getCachedConfig(cwd)
+    prettierOptions = config || getCachedConfig(filepath)
   } = {}
 ) => {
   const finalPrettierOptions = {

test/fixtures/prettier-configured/fixture/.prettierrc.js

@@ -23,4 +23,13 @@ module.exports = {
   endOfLine: "lf",
   embeddedLanguageFormatting: "auto",
   singleAttributePerLine: false,
+  overrides: [
+    {
+      files: "**/*.mts",
+      options: {
+        parser: "babel-ts",
+        singleQuote: true
+      },
+    },
+  ],
 };

test/unit-plugin-tester.test.ts

@@ -3633,11 +3633,11 @@ describe('tests targeting both FixtureOptions and TestObject interfaces', () =>
           tests: {
             'formatted-1': {
               code: `
-            console.log(  "hey"  )
-          `,
+                console.log(  "hey"  )
+              `,
               output: `
-            console.log("hey");
-          `
+                console.log("hey");
+              `
             },
             'formatted-2': {
               codeFixture: codeFileWithPrettierConfig,
@@ -3650,6 +3650,23 @@ describe('tests targeting both FixtureOptions and TestObject interfaces', () =>
         })
       );
 
+      await runPluginTester(
+        getDummyPresetOptions({
+          filepath: getFixturePath('prettier-configured/fixture/code.mts'),
+          formatResult: prettierFormatter,
+          tests: {
+            formatted: {
+              code: `
+                console.log(  "hey"  )
+              `,
+              output: `
+                console.log('hey');
+              `
+            }
+          }
+        })
+      );
+
       expect(logSpy.mock.calls).toMatchObject([['hey'], ['hey']]);
     });
   });