Require '::a.b' to have a preceding ; in no-semi style (#1442)

src/printer.js

@@ -3556,6 +3556,7 @@ function exprNeedsASIProtection(node) {
     node.type === "TemplateLiteral" ||
     node.type === "TemplateElement" ||
     node.type === "JSXElement" ||
+    node.type === "BindExpression" ||
     node.type === "RegExpLiteral" ||
     (node.type === "Literal" && node.pattern) ||
     (node.type === "Literal" && node.regex);

tests/no-semi-babylon-extensions/__snapshots__/jsfmt.spec.js.snap

@@ -0,0 +1,19 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`no-semi.js 1`] = `
+a
+;::b.c
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+a;
+::b.c;
+
+`;
+
+exports[`no-semi.js 2`] = `
+a
+;::b.c
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+a
+;::b.c
+
+`;

tests/no-semi-babylon-extensions/jsfmt.spec.js

@@ -0,0 +1,2 @@
+run_spec(__dirname, { parser: "babylon" });
+run_spec(__dirname, { semi: false, parser: "babylon" });

tests/no-semi-babylon-extensions/no-semi.js

@@ -0,0 +1,2 @@
+a
+;::b.c

tests/no-semi/__snapshots__/jsfmt.spec.js.snap

@@ -103,8 +103,6 @@ x; (x, y) => x
 
 class X {} [1, 2, 3].forEach(fn)
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary
@@ -269,8 +267,6 @@ x;
 class X {}
 [1, 2, 3].forEach(fn);
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary
@@ -424,8 +420,6 @@ x; (x, y) => x
 
 class X {} [1, 2, 3].forEach(fn)
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary
@@ -590,8 +584,6 @@ x
 class X {}
 ;[1, 2, 3].forEach(fn)
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary
@@ -745,8 +737,6 @@ x; (x, y) => x
 
 class X {} [1, 2, 3].forEach(fn)
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary
@@ -911,8 +901,6 @@ x
 class X {}
 ;[1, 2, 3].forEach(fn)
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary

tests/no-semi/no-semi.js

@@ -19,8 +19,6 @@ x; (x, y) => x
 
 class X {} [1, 2, 3].forEach(fn)
 
-// TODO: enable lang features
-// x; ::b.c
 // TODO: upgrade parser
 // class A {
 //   async; // The semicolon is *not* necessary