From 9c558bfe717f441bdb50e78b0c35b04ab869b82e Mon Sep 17 00:00:00 2001 From: Brian Ng Date: Wed, 22 Feb 2017 12:28:52 -0600 Subject: [PATCH] Remove extra parens around ternary arguments of a new call (#776) --- src/fast-path.js | 2 +- tests/conditional/__snapshots__/jsfmt.spec.js.snap | 11 +++++++++++ tests/conditional/new-expression.js | 3 +++ 3 files changed, 15 insertions(+), 1 deletion(-) create mode 100644 tests/conditional/new-expression.js diff --git a/src/fast-path.js b/src/fast-path.js index f9827742..ada741ee 100644 --- a/src/fast-path.js +++ b/src/fast-path.js @@ -419,11 +419,11 @@ FPp.needsParens = function(assumeExpressionContext) { case "BinaryExpression": case "LogicalExpression": case "LogicalExpression": - case "NewExpression": case "ExportDefaultDeclaration": case "AwaitExpression": return true; + case "NewExpression": case "CallExpression": return name === "callee" && parent.callee === node; diff --git a/tests/conditional/__snapshots__/jsfmt.spec.js.snap b/tests/conditional/__snapshots__/jsfmt.spec.js.snap index 88cf5040..c8095d6e 100644 --- a/tests/conditional/__snapshots__/jsfmt.spec.js.snap +++ b/tests/conditional/__snapshots__/jsfmt.spec.js.snap @@ -78,3 +78,14 @@ const { configureStore } = process.env.NODE_ENV === \\"production\\" : require(\\"./configureDevStore\\"); // b " `; + +exports[`new-expression.js 1`] = ` +"const testConsole = new TestConsole( + config.useStderr ? process.stderr : process.stdout +); +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +const testConsole = new TestConsole( + config.useStderr ? process.stderr : process.stdout +); +" +`; diff --git a/tests/conditional/new-expression.js b/tests/conditional/new-expression.js new file mode 100644 index 00000000..45526395 --- /dev/null +++ b/tests/conditional/new-expression.js @@ -0,0 +1,3 @@ +const testConsole = new TestConsole( + config.useStderr ? process.stderr : process.stdout +);