From 408e1b1b3c5790adb6782f539380abd531736319 Mon Sep 17 00:00:00 2001
From: fatfisz <fatfisz@gmail.com>
Date: Sun, 7 May 2017 15:18:55 +0200
Subject: [PATCH] Add the "children" property of the config

---
 lib/rules/jsx-curly-spacing.js       |   9 +-
 tests/lib/rules/jsx-curly-spacing.js | 719 ++++++++++++++++++++++++---
 2 files changed, 654 insertions(+), 74 deletions(-)

diff --git a/lib/rules/jsx-curly-spacing.js b/lib/rules/jsx-curly-spacing.js
index ce8c4b1acf..589d7f3e7d 100644
--- a/lib/rules/jsx-curly-spacing.js
+++ b/lib/rules/jsx-curly-spacing.js
@@ -68,6 +68,9 @@ module.exports = {
         properties: {
           attributes: {
             $ref: '#/definitions/basicConfigOrBoolean'
+          },
+          children: {
+            $ref: '#/definitions/basicConfigOrBoolean'
           }
         }
       }]
@@ -97,6 +100,7 @@ module.exports = {
     var DEFAULT_WHEN = SPACING.never;
     var DEFAULT_ALLOW_MULTILINE = true;
     var DEFAULT_ATTRIBUTES = true;
+    var DEFAULT_CHILDREN = false;
 
     var sourceCode = context.getSourceCode();
     var originalConfig = context.options[0] || {};
@@ -106,6 +110,8 @@ module.exports = {
     });
     var attributes = has(originalConfig, 'attributes') ? originalConfig.attributes : DEFAULT_ATTRIBUTES;
     var attributesConfig = attributes ? normalizeConfig(attributes, defaultConfig, true) : null;
+    var children = has(originalConfig, 'children') ? originalConfig.children : DEFAULT_CHILDREN;
+    var childrenConfig = children ? normalizeConfig(children, defaultConfig, true) : null;
 
     // --------------------------------------------------------------------------
     // Helpers
@@ -247,7 +253,8 @@ module.exports = {
           break;
 
         case 'JSXElement':
-          return;
+          config = childrenConfig;
+          break;
 
         default:
           return;
diff --git a/tests/lib/rules/jsx-curly-spacing.js b/tests/lib/rules/jsx-curly-spacing.js
index daf01b8a4c..c0ac7b85e7 100644
--- a/tests/lib/rules/jsx-curly-spacing.js
+++ b/tests/lib/rules/jsx-curly-spacing.js
@@ -27,21 +27,33 @@ var parserOptions = {
 var ruleTester = new RuleTester({parserOptions});
 ruleTester.run('jsx-curly-spacing', rule, {
   valid: [{
-    code: '<App foo={bar} />;'
+    code: '<App foo={bar}>{bar}</App>;'
+  }, {
+    code: '<App foo={bar}>{ bar }</App>;'
   }, {
     code: [
       '<App foo={',
-      '{ bar: true, baz: true }',
-      '} />;'
+      'bar',
+      '}>',
+      '{bar}',
+      '</App>;'
     ].join('\n')
+  }, {
+    code: '<App foo={{ bar: true, baz: true }}>{{ bar: true, baz: true }}</App>;'
+  }, {
+    code: '<App foo={{ bar: true, baz: true }}>{ { bar: true, baz: true } }</App>;'
   }, {
     code: [
       '<App foo={',
-      'bar',
-      '} />;'
+      '{ bar: true, baz: true }',
+      '}>',
+      '{{ bar: true, baz: true }}',
+      '</App>;'
     ].join('\n')
   }, {
-    code: '<App foo={{ bar: true, baz: true }} />;'
+    code: '<App>{ foo /* comment */ }</App>'
+  }, {
+    code: '<App>{ /* comment */ foo }</App>'
   }, {
     code: '<App foo={bar} />;',
     options: [{attributes: true}]
@@ -55,6 +67,13 @@ ruleTester.run('jsx-curly-spacing', rule, {
   }, {
     code: '<App foo={{ bar: true, baz: true }} />;',
     options: [{attributes: true}]
+  }, {
+    code: [
+      '<App foo={',
+      '{ bar: true, baz: true }',
+      '} />;'
+    ].join('\n'),
+    options: [{attributes: true}]
   }, {
     code: '<App foo={bar} />;',
     options: [{attributes: false}]
@@ -68,6 +87,13 @@ ruleTester.run('jsx-curly-spacing', rule, {
   }, {
     code: '<App foo={{ bar: true, baz: true }} />;',
     options: [{attributes: false}]
+  }, {
+    code: [
+      '<App foo={',
+      '{ bar: true, baz: true }',
+      '} />;'
+    ].join('\n'),
+    options: [{attributes: false}]
   }, {
     code: '<App foo={ bar } />;',
     options: [{attributes: false}]
@@ -75,15 +101,54 @@ ruleTester.run('jsx-curly-spacing', rule, {
     code: '<App foo={ { bar: true, baz: true } } />;',
     options: [{attributes: false}]
   }, {
-    code: '<App foo={bar} />;',
-    options: [{when: 'never'}]
+    code: '<App>{bar}</App>;',
+    options: [{children: true}]
   }, {
     code: [
-      '<App foo={',
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: true}]
+  }, {
+    code: '<App>{{ bar: true, baz: true }}</App>;',
+    options: [{children: true}]
+  }, {
+    code: [
+      '<App>{',
       '{ bar: true, baz: true }',
-      '} />;'
+      '}</App>;'
     ].join('\n'),
-    options: [{when: 'never', spacing: {objectLiterals: 'never'}}]
+    options: [{children: true}]
+  }, {
+    code: '<App>{bar}</App>;',
+    options: [{children: false}]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: false}]
+  }, {
+    code: '<App>{{ bar: true, baz: true }}</App>;',
+    options: [{children: false}]
+  }, {
+    code: [
+      '<App>{',
+      '{ bar: true, baz: true }',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: false}]
+  }, {
+    code: '<App>{ bar }</App>;',
+    options: [{children: false}]
+  }, {
+    code: '<App>{ { bar: true, baz: true } }</App>;',
+    options: [{children: false}]
+  }, {
+    code: '<App foo={bar} />;',
+    options: [{when: 'never'}]
   }, {
     code: '<App foo={bar} />;',
     options: [{when: 'never', allowMultiline: false}]
@@ -100,6 +165,13 @@ ruleTester.run('jsx-curly-spacing', rule, {
   }, {
     code: '<App foo={{ bar: true, baz: true }} />;',
     options: [{when: 'never', spacing: {objectLiterals: 'never'}}]
+  }, {
+    code: [
+      '<App foo={',
+      '{ bar: true, baz: true }',
+      '} />;'
+    ].join('\n'),
+    options: [{when: 'never', spacing: {objectLiterals: 'never'}}]
   }, {
     code: '<App foo={ bar } />;',
     options: [{when: 'always'}]
@@ -119,6 +191,13 @@ ruleTester.run('jsx-curly-spacing', rule, {
   }, {
     code: '<App foo={{ bar: true, baz: true }} />;',
     options: [{when: 'always', spacing: {objectLiterals: 'never'}}]
+  }, {
+    code: [
+      '<App foo={',
+      '{ bar: true, baz: true }',
+      '} />;'
+    ].join('\n'),
+    options: [{when: 'always', spacing: {objectLiterals: 'never'}}]
   }, {
     code: '<App foo={bar} />;',
     options: [{attributes: {when: 'never'}}]
@@ -137,7 +216,7 @@ ruleTester.run('jsx-curly-spacing', rule, {
       '{ bar: true, baz: true }',
       '} />;'
     ].join('\n'),
-    options: [{when: 'never'}]
+    options: [{attributes: {when: 'never'}}]
   }, {
     code: '<App foo={ {bar:baz} } />;',
     options: [{attributes: {when: 'always'}}]
@@ -176,11 +255,6 @@ ruleTester.run('jsx-curly-spacing', rule, {
       '} />;'
     ].join('\n'),
     options: [{attributes: {when: 'never', allowMultiline: true}}]
-  }, {
-    code: [
-      '<div>{/* comment */}</div>;'
-    ].join('\n'),
-    options: [{attributes: {when: 'never'}}]
   }, {
     code: '<App foo={bar/* comment */} />;',
     options: [{attributes: {when: 'never'}}]
@@ -193,31 +267,102 @@ ruleTester.run('jsx-curly-spacing', rule, {
       'bar',
       '} />;'
     ].join('\n'),
-    options: [{when: 'always', spacing: {}}]
+    options: [{attributes: {when: 'always', spacing: {}}}]
+  }, {
+    code: '<App foo={{ bar: true, baz: true }} />;',
+    options: [{attributes: {when: 'always', spacing: {objectLiterals: 'never'}}}]
   }, {
     code: [
       '<App foo={',
-      'bar',
+      '{ bar: true, baz: true }',
       '} />;'
     ].join('\n'),
-    options: [{attributes: {when: 'always', spacing: {}}}]
-  }, {
-    code: '<App foo={{ bar: true, baz: true }} />;',
     options: [{attributes: {when: 'always', spacing: {objectLiterals: 'never'}}}]
+  }, {
+    code: '<App>{bar}</App>;',
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always'}}]
+  }, {
+    code: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', allowMultiline: false}}]
+  }, {
+    code: '<App>{{ bar:baz }}</App>;',
+    options: [{children: {when: 'never'}}]
   }, {
     code: [
-      '<App foo={',
+      '<App>{',
+      '{ bar: true, baz: true }',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{ {bar:baz} }</App>;',
+    options: [{children: {when: 'always'}}]
+  }, {
+    code: [
+      '<App>{',
+      '{ bar: true, baz: true }',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'always'}}]
+  }, {
+    code: [
+      '<App>{',
       'bar',
-      '} />;'
+      '}</App>;'
     ].join('\n'),
-    options: [{when: 'always', allowMultiline: true}]
+    options: [{children: {when: 'always'}}]
   }, {
     code: [
-      '<App foo={',
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'always', allowMultiline: true}}]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'never', allowMultiline: true}}]
+  }, {
+    code: [
+      '<App>{/* comment */}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{bar/* comment */}</App>;',
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', spacing: {}}}]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'always', spacing: {}}}]
+  }, {
+    code: '<App>{{ bar: true, baz: true }}</App>;',
+    options: [{children: {when: 'always', spacing: {objectLiterals: 'never'}}}]
+  }, {
+    code: [
+      '<App>{',
       '{ bar: true, baz: true }',
-      '} />;'
+      '}</App>;'
     ].join('\n'),
-    options: [{when: 'always', spacing: {objectLiterals: 'never'}}]
+    options: [{children: {when: 'always', spacing: {objectLiterals: 'never'}}}]
   }, {
     code: '<App {...bar} />;'
   }, {
@@ -283,7 +428,7 @@ ruleTester.run('jsx-curly-spacing', rule, {
       'bar',
       '} {',
       '...bar',
-      '}/>;'
+      '} />;'
     ].join('\n'),
     options: [{attributes: {when: 'always'}}]
   }, {
@@ -293,21 +438,72 @@ ruleTester.run('jsx-curly-spacing', rule, {
     code: '<App foo={3} bar={ {a: 2} } />',
     options: [{attributes: {when: 'never', spacing: {objectLiterals: 'always'}}}]
   }, {
-    code: '<div foo={ bar }>{bar}</div>',
+    code: '<App>{bar/* comment */}</App>;',
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{bar} {baz}</App>;'
+  }, {
+    code: '<App>{bar} {baz}</App>;',
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always'}}]
+  }, {
+    code: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always', allowMultiline: false}}]
+  }, {
+    code: '<App>{{ bar:baz }} {baz}</App>;',
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{ {bar:baz} } { baz }</App>;',
+    options: [{children: {when: 'always'}}]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '} {',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    options: [{children: {when: 'always'}}]
+  }, {
+    code: '<App>{bar/* comment */} {baz/* comment */}</App>;',
+    options: [{children: {when: 'never'}}]
+  }, {
+    code: '<App>{3} { {a: 2} }</App>',
+    options: [{children: {when: 'never', spacing: {objectLiterals: 'always'}}}]
+  }, {
+    code: '<App foo={ bar }>{bar}</App>',
     options: [{attributes: {when: 'always'}}]
   }],
 
   invalid: [{
-    code: '<App foo={ bar } />;',
-    output: '<App foo={bar} />;',
+    code: '<App foo={ bar }>{bar}</App>;',
+    output: '<App foo={bar}>{bar}</App>;',
     errors: [{
       message: 'There should be no space after \'{\''
     }, {
       message: 'There should be no space before \'}\''
     }]
   }, {
-    code: '<App foo={ { bar: true, baz: true } } />;',
-    output: '<App foo={{ bar: true, baz: true }} />;',
+    code: '<App foo={ bar }>{ bar }</App>;',
+    output: '<App foo={bar}>{ bar }</App>;',
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App foo={ { bar: true, baz: true } }>{{ bar: true, baz: true }}</App>;',
+    output: '<App foo={{ bar: true, baz: true }}>{{ bar: true, baz: true }}</App>;',
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App foo={ { bar: true, baz: true } }>{ { bar: true, baz: true } }</App>;',
+    output: '<App foo={{ bar: true, baz: true }}>{ { bar: true, baz: true } }</App>;',
     errors: [{
       message: 'There should be no space after \'{\''
     }, {
@@ -331,6 +527,24 @@ ruleTester.run('jsx-curly-spacing', rule, {
     }, {
       message: 'There should be no space before \'}\''
     }]
+  }, {
+    code: '<App>{ bar }</App>;',
+    output: '<App>{bar}</App>;',
+    options: [{children: true}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{ { bar: true, baz: true } }</App>;',
+    output: '<App>{{ bar: true, baz: true }}</App>;',
+    options: [{children: true}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
   }, {
     code: '<App foo={ bar } />;',
     output: '<App foo={bar} />;',
@@ -376,30 +590,13 @@ ruleTester.run('jsx-curly-spacing', rule, {
       message: 'There should be no newline before \'}\''
     }]
   }, {
-    code: [
-      '<App foo={',
-      '{ bar: true, baz: true }',
-      '} />;'
-    ].join('\n'),
+    code: '<App foo={{ bar: true, baz: true }} />;',
     output: '<App foo={ { bar: true, baz: true } } />;',
-    options: [{when: 'never', allowMultiline: false, spacing: {objectLiterals: 'always'}}],
-    errors: [{
-      message: 'There should be no newline after \'{\''
-    }, {
-      message: 'There should be no newline before \'}\''
-    }]
-  }, {
-    code: [
-      '<App foo={',
-      '{ bar: true, baz: true }',
-      '} />;'
-    ].join('\n'),
-    output: '<App foo={{ bar: true, baz: true }} />;',
-    options: [{when: 'always', allowMultiline: false, spacing: {objectLiterals: 'never'}}],
+    options: [{when: 'never', spacing: {objectLiterals: 'always'}}],
     errors: [{
-      message: 'There should be no newline after \'{\''
+      message: 'A space is required after \'{\''
     }, {
-      message: 'There should be no newline before \'}\''
+      message: 'A space is required before \'}\''
     }]
   }, {
     code: [
@@ -408,21 +605,12 @@ ruleTester.run('jsx-curly-spacing', rule, {
       '} />;'
     ].join('\n'),
     output: '<App foo={ { bar: true, baz: true } } />;',
-    options: [{when: 'always', allowMultiline: false, spacing: {objectLiterals: 'always'}}],
+    options: [{when: 'never', allowMultiline: false, spacing: {objectLiterals: 'always'}}],
     errors: [{
       message: 'There should be no newline after \'{\''
     }, {
       message: 'There should be no newline before \'}\''
     }]
-  }, {
-    code: '<App foo={{ bar: true, baz: true }} />;',
-    output: '<App foo={ { bar: true, baz: true } } />;',
-    options: [{when: 'never', spacing: {objectLiterals: 'always'}}],
-    errors: [{
-      message: 'A space is required after \'{\''
-    }, {
-      message: 'A space is required before \'}\''
-    }]
   }, {
     code: '<App foo={bar} />;',
     output: '<App foo={ bar } />;',
@@ -454,6 +642,19 @@ ruleTester.run('jsx-curly-spacing', rule, {
     }, {
       message: 'There should be no space before \'}\''
     }]
+  }, {
+    code: [
+      '<App foo={',
+      '{ bar: true, baz: true }',
+      '} />;'
+    ].join('\n'),
+    output: '<App foo={{ bar: true, baz: true }} />;',
+    options: [{when: 'always', allowMultiline: false, spacing: {objectLiterals: 'never'}}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
   }, {
     code: '<App foo={{ bar: true, baz: true }} />;',
     output: '<App foo={ { bar: true, baz: true } } />;',
@@ -463,6 +664,19 @@ ruleTester.run('jsx-curly-spacing', rule, {
     }, {
       message: 'A space is required before \'}\''
     }]
+  }, {
+    code: [
+      '<App foo={',
+      '{ bar: true, baz: true }',
+      '} />;'
+    ].join('\n'),
+    output: '<App foo={ { bar: true, baz: true } } />;',
+    options: [{when: 'always', allowMultiline: false, spacing: {objectLiterals: 'always'}}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
   }, {
     code: '<App foo={ bar } />;',
     output: '<App foo={bar} />;',
@@ -665,6 +879,208 @@ ruleTester.run('jsx-curly-spacing', rule, {
     }, {
       message: 'There should be no space before \'}\''
     }]
+  }, {
+    code: '<App>{ bar }</App>;',
+    output: '<App>{bar}</App>;',
+    options: [{children: true, when: 'never'}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    output: '<App>{bar}</App>;',
+    options: [{children: true, when: 'never', allowMultiline: false}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
+  }, {
+    code: '<App>{ { bar: true, baz: true } }</App>;',
+    output: '<App>{{ bar: true, baz: true }}</App>;',
+    options: [{children: true, when: 'never', spacing: {objectLiterals: 'never'}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{{ bar: true, baz: true }}</App>;',
+    output: '<App>{ { bar: true, baz: true } }</App>;',
+    options: [{children: true, when: 'never', spacing: {objectLiterals: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{bar}</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: true, when: 'always'}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    output: '<App>{ bar }</App>;',
+    options: [{children: true, when: 'always', allowMultiline: false}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
+  }, {
+    code: '<App>{ { bar: true, baz: true } }</App>;',
+    output: '<App>{{ bar: true, baz: true }}</App>;',
+    options: [{children: true, when: 'always', spacing: {objectLiterals: 'never'}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{{ bar: true, baz: true }}</App>;',
+    output: '<App>{ { bar: true, baz: true } }</App>;',
+    options: [{children: true, when: 'always', spacing: {objectLiterals: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{ bar }</App>;',
+    output: '<App>{bar}</App>;',
+    options: [{children: {when: 'never'}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{ bar }</App>;',
+    output: '<App>{bar}</App>;',
+    options: [{children: {when: 'never', allowMultiline: false}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{bar}</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{bar}</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', allowMultiline: false}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{ bar}</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{bar }</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }]
+  }, {
+    code: '<App>{ bar}</App>;',
+    output: '<App>{bar}</App>;',
+    options: [{children: {when: 'never'}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }]
+  }, {
+    code: '<App>{bar }</App>;',
+    output: '<App>{bar}</App>;',
+    options: [{children: {when: 'never'}}],
+    errors: [{
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    output: '<App>{bar}</App>;',
+    options: [{children: {when: 'never', allowMultiline: false}}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '}</App>;'
+    ].join('\n'),
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', allowMultiline: false}}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
+  }, {
+    code: '<App>{bar}</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', spacing: {}}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{ bar}</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', spacing: {}}}],
+    errors: [{
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{bar }</App>;',
+    output: '<App>{ bar }</App>;',
+    options: [{children: {when: 'always', spacing: {}}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }]
+  }, {
+    code: '<App>{ {bar: true, baz: true} }</App>;',
+    output: '<App>{{bar: true, baz: true}}</App>;',
+    options: [{children: {when: 'always', spacing: {objectLiterals: 'never'}}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
   }, {
     code: '<App { ...bar } />;',
     output: '<App {...bar} />;',
@@ -895,16 +1311,16 @@ ruleTester.run('jsx-curly-spacing', rule, {
       message: 'A space is required before \'}\''
     }]
   }, {
-    code: '<div foo={ foo /* comment */ } />',
-    output: '<div foo={foo /* comment */} />',
+    code: '<App foo={ foo /* comment */ } />',
+    output: '<App foo={foo /* comment */} />',
     errors: [{
       message: 'There should be no space after \'{\''
     }, {
       message: 'There should be no space before \'}\''
     }]
   }, {
-    code: '<div foo={foo /* comment */} />',
-    output: '<div foo={ foo /* comment */ } />',
+    code: '<App foo={foo /* comment */} />',
+    output: '<App foo={ foo /* comment */ } />',
     options: [{attributes: {when: 'always'}}],
     errors: [{
       message: 'A space is required after \'{\''
@@ -912,21 +1328,178 @@ ruleTester.run('jsx-curly-spacing', rule, {
       message: 'A space is required before \'}\''
     }]
   }, {
-    code: '<div foo={ /* comment */ foo } />',
-    output: '<div foo={/* comment */ foo} />',
+    code: '<App foo={ /* comment */ foo } />',
+    output: '<App foo={/* comment */ foo} />',
     errors: [{
       message: 'There should be no space after \'{\''
     }, {
       message: 'There should be no space before \'}\''
     }]
   }, {
-    code: '<div foo={/* comment */ foo} />',
-    output: '<div foo={ /* comment */ foo } />',
+    code: '<App foo={/* comment */ foo} />',
+    output: '<App foo={ /* comment */ foo } />',
     options: [{attributes: {when: 'always'}}],
     errors: [{
       message: 'A space is required after \'{\''
     }, {
       message: 'A space is required before \'}\''
     }]
+  }, {
+    code: '<App>{ bar } { baz }</App>;',
+    output: '<App>{bar} {baz}</App>;',
+    options: [{children: {when: 'never'}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }, {
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{ bar } { baz }</App>;',
+    output: '<App>{bar} {baz}</App>;',
+    options: [{children: {when: 'never', allowMultiline: false}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }, {
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: '<App>{bar} {baz}</App>;',
+    output: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }, {
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{bar} {baz}</App>;',
+    output: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always', allowMultiline: false}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }, {
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{ bar} { baz}</App>;',
+    output: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required before \'}\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{bar } {baz }</App>;',
+    output: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required after \'{\''
+    }]
+  }, {
+    code: '<App>{ bar} { baz}</App>;',
+    output: '<App>{bar} {baz}</App>;',
+    options: [{children: {when: 'never'}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space after \'{\''
+    }]
+  }, {
+    code: '<App>{bar } {baz }</App>;',
+    output: '<App>{bar} {baz}</App>;',
+    options: [{children: {when: 'never'}}],
+    errors: [{
+      message: 'There should be no space before \'}\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '} {',
+      'baz',
+      '}</App>;'
+    ].join('\n'),
+    output: '<App>{bar} {baz}</App>;',
+    options: [{children: {when: 'never', allowMultiline: false}}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }, {
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
+  }, {
+    code: [
+      '<App>{',
+      'bar',
+      '} {',
+      'baz',
+      '}</App>;'
+    ].join('\n'),
+    output: '<App>{ bar } { baz }</App>;',
+    options: [{children: {when: 'always', allowMultiline: false}}],
+    errors: [{
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }, {
+      message: 'There should be no newline after \'{\''
+    }, {
+      message: 'There should be no newline before \'}\''
+    }]
+  }, {
+    code: '<App>{ 3 } bar={{a: 2}}</App>',
+    output: '<App>{3} bar={ {a: 2} }</App>',
+    options: [{children: {when: 'never', spacing: {objectLiterals: 'always'}}}],
+    errors: [{
+      message: 'There should be no space after \'{\''
+    }, {
+      message: 'There should be no space before \'}\''
+    }, {
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{foo /* comment */}</App>',
+    output: '<App>{ foo /* comment */ }</App>',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
+  }, {
+    code: '<App>{/* comment */ foo}</App>',
+    output: '<App>{ /* comment */ foo }</App>',
+    options: [{children: {when: 'always'}}],
+    errors: [{
+      message: 'A space is required after \'{\''
+    }, {
+      message: 'A space is required before \'}\''
+    }]
   }]
 });