Refactor a bunch of tests to use template literal syntax instead of [].join(

tests/lib/rules/forbid-foreign-prop-types.js

@@ -53,88 +53,88 @@ ruleTester.run('forbid-foreign-prop-types', rule, {
   }],
 
   invalid: [{
-    code: [
-      'var Foo = createReactClass({',
-      '  propTypes: Bar.propTypes,',
-      '  render: function() {',
-      '    return <Foo className="bar" />;',
-      '  }',
-      '});'
-    ].join('\n'),
+    code: `
+      var Foo = createReactClass({
+        propTypes: Bar.propTypes,
+        render: function() {
+          return <Foo className="bar" />;
+        }
+      });
+    `,
     errors: [{
       message: ERROR_MESSAGE,
       type: 'Identifier'
     }]
   },
   {
-    code: [
-      'var Foo = createReactClass({',
-      '  propTypes: Bar["propTypes"],',
-      '  render: function() {',
-      '    return <Foo className="bar" />;',
-      '  }',
-      '});'
-    ].join('\n'),
+    code: `
+      var Foo = createReactClass({
+        propTypes: Bar["propTypes"],
+        render: function() {
+          return <Foo className="bar" />;
+        }
+      });
+    `,
     errors: [{
       message: ERROR_MESSAGE,
       type: 'Literal'
     }]
   },
   {
-    code: [
-      'var { propTypes } = SomeComponent',
-      'var Foo = createReactClass({',
-      '  propTypes,',
-      '  render: function() {',
-      '    return <Foo className="bar" />;',
-      '  }',
-      '});'
-    ].join('\n'),
+    code: `
+      var { propTypes } = SomeComponent
+      var Foo = createReactClass({
+        propTypes,
+        render: function() {
+          return <Foo className="bar" />;
+        }
+      });
+    `,
     errors: [{
       message: ERROR_MESSAGE,
       type: 'Property'
     }]
   },
   {
-    code: [
-      'var { propTypes: things, ...foo } = SomeComponent',
-      'var Foo = createReactClass({',
-      '  propTypes,',
-      '  render: function() {',
-      '    return <Foo className="bar" />;',
-      '  }',
-      '});'
-    ].join('\n'),
+    code: `
+      var { propTypes: things, ...foo } = SomeComponent
+      var Foo = createReactClass({
+        propTypes,
+        render: function() {
+          return <Foo className="bar" />;
+        }
+      });
+    `,
     parser: 'babel-eslint',
     errors: [{
       message: ERROR_MESSAGE,
       type: 'Property'
     }]
   },
   {
-    code: [
-      'class MyComponent extends React.Component {',
-      '  static fooBar = {',
-      '    baz: Qux.propTypes.baz',
-      '  };',
-      '}'
-    ].join('\n'),
+    code: `
+      class MyComponent extends React.Component {
+        static fooBar = {
+          baz: Qux.propTypes.baz
+        };
+      }
+    `,
     parser: 'babel-eslint',
     errors: [{
       message: ERROR_MESSAGE,
       type: 'Identifier'
     }]
   },
   {
-    code: [
-      'var { propTypes: typesOfProps } = SomeComponent',
-      'var Foo = createReactClass({',
-      '  propTypes: typesOfProps,',
-      '  render: function() {',
-      '    return <Foo className="bar" />;',
-      '  }',
-      '});'
-    ].join('\n'),
+    code: `
+      var { propTypes: typesOfProps } = SomeComponent
+      var Foo = createReactClass({
+        propTypes: typesOfProps,
+        render: function() {
+          return <Foo className="bar" />;
+        }
+      });
+    `,
     errors: [{
       message: ERROR_MESSAGE,
       type: 'Property'

tests/lib/rules/jsx-closing-tag-location.js

@@ -27,39 +27,39 @@ const MESSAGE_OWN_LINE = [{message: 'Closing tag of a multiline JSX expression m
 const ruleTester = new RuleTester({parserOptions});
 ruleTester.run('jsx-closing-tag-location', rule, {
   valid: [{
-    code: [
-      '<App>',
-      '  foo',
-      '</App>'
-    ].join('\n')
+    code: `
+      <App>
+        foo
+      </App>
+    `
   }, {
-    code: [
-      '<App>foo</App>'
-    ].join('\n')
+    code: `
+      <App>foo</App>
+    `
   }],
 
   invalid: [{
-    code: [
-      '<App>',
-      '  foo',
-      '  </App>'
-    ].join('\n'),
-    output: [
-      '<App>',
-      '  foo',
-      '</App>'
-    ].join('\n'),
+    code: `
+      <App>
+        foo
+        </App>
+    `,
+    output: `
+      <App>
+        foo
+      </App>
+    `,
     errors: MESSAGE_MATCH_INDENTATION
   }, {
-    code: [
-      '<App>',
-      '  foo</App>'
-    ].join('\n'),
-    output: [
-      '<App>',
-      '  foo',
-      '</App>'
-    ].join('\n'),
+    code: `
+      <App>
+        foo</App>
+    `,
+    output: `
+      <App>
+        foo
+      </App>
+    `,
     errors: MESSAGE_OWN_LINE
   }]
 });

tests/lib/rules/jsx-handler-names.js

@@ -27,105 +27,63 @@ const parserOptions = {
 const ruleTester = new RuleTester({parserOptions});
 ruleTester.run('jsx-handler-names', rule, {
   valid: [{
-    code: [
-      '<TestComponent onChange={this.handleChange} />'
-    ].join('\n')
+    code: '<TestComponent onChange={this.handleChange} />'
   }, {
-    code: [
-      '<TestComponent onChange={this.props.onChange} />'
-    ].join('\n')
+    code: '<TestComponent onChange={this.props.onChange} />'
   }, {
-    code: [
-      '<TestComponent onChange={this.props.onFoo} />'
-    ].join('\n')
+    code: '<TestComponent onChange={this.props.onFoo} />'
   }, {
-    code: [
-      '<TestComponent isSelected={this.props.isSelected} />'
-    ].join('\n')
+    code: '<TestComponent isSelected={this.props.isSelected} />'
   }, {
-    code: [
-      '<TestComponent shouldDisplay={this.state.shouldDisplay} />'
-    ].join('\n')
+    code: '<TestComponent shouldDisplay={this.state.shouldDisplay} />'
   }, {
-    code: [
-      '<TestComponent shouldDisplay={arr[0].prop} />'
-    ].join('\n')
+    code: '<TestComponent shouldDisplay={arr[0].prop} />'
   }, {
-    code: [
-      '<TestComponent onChange={props.onChange} />'
-    ].join('\n')
+    code: '<TestComponent onChange={props.onChange} />'
   }, {
-    code: [
-      '<TestComponent ref={this.handleRef} />'
-    ].join('\n')
+    code: '<TestComponent ref={this.handleRef} />'
   }, {
-    code: [
-      '<TestComponent ref={this.somethingRef} />'
-    ].join('\n')
+    code: '<TestComponent ref={this.somethingRef} />'
   }, {
-    code: [
-      '<TestComponent test={this.props.content} />'
-    ].join('\n'),
+    code: '<TestComponent test={this.props.content} />',
     options: [{
       eventHandlerPrefix: 'on',
       eventHandlerPropPrefix: 'on'
     }]
   }, {
-    code: [
-      '<TestComponent onChange={props::handleChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={props::handleChange} />',
     parser: 'babel-eslint'
   }, {
-    code: [
-      '<TestComponent onChange={::props.onChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={::props.onChange} />',
     parser: 'babel-eslint'
   }, {
-    code: [
-      '<TestComponent onChange={props.foo::handleChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={props.foo::handleChange} />',
     parser: 'babel-eslint'
   }, {
-    code: [
-      '<TestComponent only={this.only} />'
-    ].join('\n')
+    code: '<TestComponent only={this.only} />'
   }],
 
   invalid: [{
-    code: [
-      '<TestComponent onChange={this.doSomethingOnChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={this.doSomethingOnChange} />',
     errors: [{message: 'Handler function for onChange prop key must begin with \'handle\''}]
   }, {
-    code: [
-      '<TestComponent onChange={this.handlerChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={this.handlerChange} />',
     errors: [{message: 'Handler function for onChange prop key must begin with \'handle\''}]
   }, {
-    code: [
-      '<TestComponent only={this.handleChange} />'
-    ].join('\n'),
+    code: '<TestComponent only={this.handleChange} />',
     errors: [{message: 'Prop key for handleChange must begin with \'on\''}]
   }, {
-    code: [
-      '<TestComponent handleChange={this.handleChange} />'
-    ].join('\n'),
+    code: '<TestComponent handleChange={this.handleChange} />',
     errors: [{message: 'Prop key for handleChange must begin with \'on\''}]
   }, {
-    code: [
-      '<TestComponent onChange={this.onChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={this.onChange} />',
     errors: [{message: 'Handler function for onChange prop key must begin with \'handle\''}]
   }, {
-    code: [
-      '<TestComponent onChange={props::onChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={props::onChange} />',
     parser: 'babel-eslint',
     errors: [{message: 'Handler function for onChange prop key must begin with \'handle\''}]
   }, {
-    code: [
-      '<TestComponent onChange={props.foo::onChange} />'
-    ].join('\n'),
+    code: '<TestComponent onChange={props.foo::onChange} />',
     parser: 'babel-eslint',
     errors: [{message: 'Handler function for onChange prop key must begin with \'handle\''}]
   }]