Inline do expressions inside of JSX (#3607)

Fixes #3602
prettier · Dec 29, 2017 · 7eca801 · 7eca801
1 parent cfb8987
commit 7eca801
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 0 deletions.
diff --git a/src/language-js/printer-estree.js b/src/language-js/printer-estree.js
@@ -1691,6 +1691,7 @@ function printPathNoParens(path, options, print, args) {
           n.expression.type === "JSXEmptyExpression" ||
           n.expression.type === "TemplateLiteral" ||
           n.expression.type === "TaggedTemplateExpression" ||
+          n.expression.type === "DoExpression" ||
           (isJSXNode(parent) &&
             (n.expression.type === "ConditionalExpression" ||
               isBinaryish(n.expression))));

diff --git a/tests/jsx-do/__snapshots__/jsfmt.spec.js.snap b/tests/jsx-do/__snapshots__/jsfmt.spec.js.snap
@@ -0,0 +1,16 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`do.js 1`] = `
+<div>
+  {do {
+	  1
+  }}
+</div>
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+<div>
+  {do {
+    1;
+  }}
+</div>;
+
+`;
diff --git a/tests/jsx-do/do.js b/tests/jsx-do/do.js
@@ -0,0 +1,5 @@
+<div>
+  {do {
+	  1
+  }}
+</div>
diff --git a/tests/jsx-do/jsfmt.spec.js b/tests/jsx-do/jsfmt.spec.js
@@ -0,0 +1 @@
+run_spec(__dirname, ["babylon"]);