JSXSpreadChildren, throw in transform-react-jsx (#4988)

packages/babel-generator/src/generators/jsx.js

@@ -35,6 +35,13 @@ export function JSXExpressionContainer(node: Object) {
   this.token("}");
 }
 
+export function JSXSpreadChild(node: Object) {
+  this.token("{");
+  this.token("...");
+  this.print(node.expression, node);
+  this.token("}");
+}
+
 export function JSXText(node: Object) {
   this.token(node.value);
 }

packages/babel-generator/test/fixtures/types/XJSSpreadChildren/actual.js

@@ -0,0 +1 @@
+<div>{...this.props.children}</div>;

packages/babel-generator/test/fixtures/types/XJSSpreadChildren/expected.js

@@ -0,0 +1 @@
+<div>{...this.props.children}</div>;

packages/babel-helper-builder-react-jsx/src/index.js

@@ -17,6 +17,10 @@ export default function (opts) {
     throw path.buildCodeFrameError("Namespace tags are not supported. ReactJSX is not XML.");
   };
 
+  visitor.JSXSpreadChild = function(path) {
+    throw path.buildCodeFrameError("Spread children are not supported.");
+  };
+
   visitor.JSXElement = {
     exit(path, file) {
       let callExpr = buildElementCall(path.get("openingElement"), file);

packages/babel-plugin-transform-react-jsx/test/fixtures/react/should-disallow-spread-children/actual.js

@@ -0,0 +1 @@
+<div>{...children}</div>;

packages/babel-plugin-transform-react-jsx/test/fixtures/react/should-disallow-spread-children/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "Spread children are not supported."
+}

packages/babel-types/src/definitions/jsx.js

@@ -39,7 +39,7 @@ defineType("JSXElement", {
     children: {
       validate: chain(
         assertValueType("array"),
-        assertEach(assertNodeType("JSXText", "JSXExpressionContainer", "JSXElement"))
+        assertEach(assertNodeType("JSXText", "JSXExpressionContainer", "JSXSpreadChild", "JSXElement"))
       )
     }
   }
@@ -59,6 +59,16 @@ defineType("JSXExpressionContainer", {
   }
 });
 
+defineType("JSXSpreadChild", {
+  visitor: ["expression"],
+  aliases: ["JSX", "Immutable"],
+  fields: {
+    expression: {
+      validate: assertNodeType("Expression")
+    }
+  }
+});
+
 defineType("JSXIdentifier", {
   builder: ["name"],
   aliases: ["JSX", "Expression"],