Fix formatting of ternary (#15679)

changelog_unreleased/javascript/15677.md

@@ -0,0 +1,19 @@
+#### Fix formatting of ternary in function call (#15677 by @fisker)
+
+<!-- prettier-ignore -->
+```jsx
+// Input
+stopDirectory = await (useCache
+  ? memoizedFindProjectRoot
+  : findProjectRootWithoutCache)(path.dirname(path.resolve(filePath)));
+
+// Prettier stable
+stopDirectory = await (useCache
+  ? memoizedFindProjectRoot
+  : findProjectRootWithoutCache)(path.dirname(path.resolve(filePath)));
+
+// Prettier main
+stopDirectory = await (
+  useCache ? memoizedFindProjectRoot : findProjectRootWithoutCache
+)(path.dirname(path.resolve(filePath)));
+```

src/language-js/print/ternary-old.js

@@ -137,6 +137,7 @@ const ancestorNameMap = new Map([
   ["ThrowStatement", "argument"],
   ["UnaryExpression", "argument"],
   ["YieldExpression", "argument"],
+  ["AwaitExpression", "argument"],
 ]);
 function shouldExtraIndentForConditionalExpression(path) {
   const { node } = path;

src/language-js/print/ternary.js

@@ -76,6 +76,7 @@ const ancestorNameMap = new Map([
   ["ThrowStatement", "argument"],
   ["UnaryExpression", "argument"],
   ["YieldExpression", "argument"],
+  ["AwaitExpression", "argument"],
 ]);
 /**
  * Do we want to wrap the entire ternary in its own indent?

tests/format/js/ternaries/parenthesis/__snapshots__/jsfmt.spec.js.snap

@@ -0,0 +1,39 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`await-expression.js - {"experimentalTernaries":true} format 1`] = `
+====================================options=====================================
+experimentalTernaries: true
+parsers: ["babel"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+stopDirectory = await (useCache
+  ? memoizedFindProjectRoot
+  : findProjectRootWithoutCache)(path.dirname(path.resolve(filePath)));
+
+=====================================output=====================================
+stopDirectory = await (
+  useCache ?
+    memoizedFindProjectRoot
+  : findProjectRootWithoutCache)(path.dirname(path.resolve(filePath)));
+
+================================================================================
+`;
+
+exports[`await-expression.js format 1`] = `
+====================================options=====================================
+parsers: ["babel"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+stopDirectory = await (useCache
+  ? memoizedFindProjectRoot
+  : findProjectRootWithoutCache)(path.dirname(path.resolve(filePath)));
+
+=====================================output=====================================
+stopDirectory = await (
+  useCache ? memoizedFindProjectRoot : findProjectRootWithoutCache
+)(path.dirname(path.resolve(filePath)));
+
+================================================================================
+`;

tests/format/js/ternaries/parenthesis/await-expression.js

@@ -0,0 +1,3 @@
+stopDirectory = await (useCache
+  ? memoizedFindProjectRoot
+  : findProjectRootWithoutCache)(path.dirname(path.resolve(filePath)));

tests/format/js/ternaries/parenthesis/jsfmt.spec.js

@@ -0,0 +1,7 @@
+const parsers = [
+  // Didn't test flow and typescript since they can't parse `await (a ? b : c)` correctly
+  "babel", // "flow", "typescript"
+];
+
+run_spec(import.meta, parsers);
+run_spec(import.meta, parsers, { experimentalTernaries: true });