Allow ExpressionStatements in IfExpressions

commit 900dcd2
Author: William C. Johnson <wcjohnson@oigroup.net>
Date:   Fri Apr 7 17:18:26 2017 -0400

    Fixtures for `ExpressionStatements` in `IfExpression`s

commit 71060c3
Author: William C. Johnson <wcjohnson@oigroup.net>
Date:   Fri Apr 7 17:18:03 2017 -0400

    Allow `ExpressionStatements` in `IfExpressions`

src/index.js

@@ -552,6 +552,11 @@ export default function (babel) {
     if (t.isBlockStatement(path.node[key])) {
       path.get(key).canSwapBetweenExpressionAndStatement = () => true;
       path.get(key).replaceExpressionWithStatements(path.node[key].body);
+      return path.node[key];
+    } else if (t.isExpressionStatement(path.node[key])) {
+      return path.node[key].expression;
+    } else {
+      return path.node[key];
     }
   }
 
@@ -838,11 +843,11 @@ export default function (babel) {
         validate: assertNodeType("Expression")
       },
       consequent: {
-        validate: assertNodeType("Expression", "BlockStatement")
+        validate: assertNodeType("Expression", "BlockStatement", "ExpressionStatement")
       },
       alternate: {
         optional: true,
-        validate: assertNodeType("Expression", "BlockStatement")
+        validate: assertNodeType("Expression", "BlockStatement", "ExpressionStatement")
       }
     }
   });
@@ -1128,15 +1133,16 @@ export default function (babel) {
       },
 
       IfExpression(path) {
-        blockToExpression(path, "consequent");
+        const consequent = blockToExpression(path, "consequent");
 
+        let alternate;
         if (path.node.alternate) {
-          blockToExpression(path, "alternate");
+          alternate = blockToExpression(path, "alternate");
         } else {
-          path.get("alternate").replaceWith(t.nullLiteral());
+          alternate = t.nullLiteral();
         }
 
-        path.replaceWith(t.conditionalExpression(path.node.test, path.node.consequent, path.node.alternate));
+        path.replaceWith(t.conditionalExpression(path.node.test, consequent, alternate));
       },
 
       AssignmentExpression(path) {

test/fixtures/if-expressions/js-style/actual.js

@@ -0,0 +1,3 @@
+x = if true {1} else 2
+
+y = if(false) 2 else 3

test/fixtures/if-expressions/js-style/exec.js

@@ -0,0 +1,8 @@
+x = if true {2} else 3
+assert(x == 2, "2")
+
+y = if (false) 4 else {5}
+assert(y == 5, "5")
+
+z = if (true) 6 else 7
+assert(z == 6, "6")

test/fixtures/if-expressions/js-style/expected.js

@@ -0,0 +1,3 @@
+const x = true ? 1 : 2;
+
+const y = false ? 2 : 3;