Update tests to pass parserOptions globally

jsx-eslint · May 8, 2017 · 1b5743f · 1b5743f
1 parent 0a2c510
commit 1b5743f
Show file tree

Hide file tree

Showing 59 changed files with 1,240 additions and 2,310 deletions.
diff --git a/tests/lib/rules/display-name.js b/tests/lib/rules/display-name.js
@@ -14,7 +14,8 @@ var RuleTester = require('eslint').RuleTester;
 require('babel-eslint');
 
 var parserOptions = {
-  ecmaVersion: 6,
+  ecmaVersion: 8,
+  sourceType: 'module',
   ecmaFeatures: {
     experimentalObjectRestSpread: true,
     jsx: true
@@ -25,7 +26,7 @@ var parserOptions = {
 // Tests
 // ------------------------------------------------------------------------------
 
-var ruleTester = new RuleTester();
+var ruleTester = new RuleTester({parserOptions});
 ruleTester.run('display-name', rule, {
 
   valid: [{
@@ -39,8 +40,7 @@ ruleTester.run('display-name', rule, {
     ].join('\n'),
     options: [{
       ignoreTranspilerName: true
-    }],
-    parserOptions: parserOptions
+    }]
   }, {
     code: [
       'var Hello = React.createClass({',
@@ -57,8 +57,7 @@ ruleTester.run('display-name', rule, {
       react: {
         createClass: 'createClass'
       }
-    },
-    parserOptions: parserOptions
+    }
   }, {
     code: [
       'class Hello extends React.Component {',
@@ -70,17 +69,15 @@ ruleTester.run('display-name', rule, {
     ].join('\n'),
     options: [{
       ignoreTranspilerName: true
-    }],
-    parserOptions: parserOptions
+    }]
   }, {
     code: [
       'class Hello {',
       '  render() {',
       '    return \'Hello World\';',
       '  }',
       '}'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'class Hello extends Greetings {',
@@ -97,8 +94,7 @@ ruleTester.run('display-name', rule, {
       '  method;',
       '}'
     ].join('\n'),
-    parser: 'babel-eslint',
-    parserOptions: parserOptions
+    parser: 'babel-eslint'
   }, {
     code: [
       'class Hello extends React.Component {',
@@ -112,8 +108,7 @@ ruleTester.run('display-name', rule, {
     ].join('\n'),
     options: [{
       ignoreTranspilerName: true
-    }],
-    parserOptions: parserOptions
+    }]
   }, {
     code: [
       'class Hello extends React.Component {',
@@ -126,17 +121,15 @@ ruleTester.run('display-name', rule, {
     options: [{
       ignoreTranspilerName: true
     }],
-    parser: 'babel-eslint',
-    parserOptions: parserOptions
+    parser: 'babel-eslint'
   }, {
     code: [
       'var Hello = createReactClass({',
       '  render: function() {',
       '    return <div>Hello {this.props.name}</div>;',
       '  }',
       '});'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'class Hello extends React.Component {',
@@ -163,8 +156,7 @@ ruleTester.run('display-name', rule, {
       '    return <div>Hello {this.props.name}</div>;',
       '  }',
       '});'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'module.exports = createReactClass({',
@@ -173,8 +165,7 @@ ruleTester.run('display-name', rule, {
       '    return <div>Hello {this.props.name}</div>;',
       '  }',
       '});'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'var Hello = createReactClass({',
@@ -188,8 +179,7 @@ ruleTester.run('display-name', rule, {
     ].join('\n'),
     options: [{
       ignoreTranspilerName: true
-    }],
-    parserOptions: parserOptions
+    }]
   }, {
     code: [
       'export default class {',
@@ -421,7 +411,6 @@ ruleTester.run('display-name', rule, {
     options: [{
       ignoreTranspilerName: true
     }],
-    parserOptions: parserOptions,
     errors: [{
       message: 'Component definition is missing display name'
     }]
@@ -441,7 +430,6 @@ ruleTester.run('display-name', rule, {
         createClass: 'createClass'
       }
     },
-    parserOptions: parserOptions,
     errors: [{
       message: 'Component definition is missing display name'
     }]
@@ -456,7 +444,6 @@ ruleTester.run('display-name', rule, {
     options: [{
       ignoreTranspilerName: true
     }],
-    parserOptions: parserOptions,
     errors: [{
       message: 'Component definition is missing display name'
     }]
@@ -471,7 +458,6 @@ ruleTester.run('display-name', rule, {
     options: [{
       ignoreTranspilerName: true
     }],
-    parserOptions: parserOptions,
     errors: [{
       message: 'Component definition is missing display name'
     }]
@@ -489,7 +475,6 @@ ruleTester.run('display-name', rule, {
     options: [{
       ignoreTranspilerName: true
     }],
-    parserOptions: parserOptions,
     errors: [{
       message: 'Component definition is missing display name'
     }]

diff --git a/tests/lib/rules/forbid-component-props.js b/tests/lib/rules/forbid-component-props.js
@@ -11,7 +11,8 @@ var rule = require('../../../lib/rules/forbid-component-props');
 var RuleTester = require('eslint').RuleTester;
 
 var parserOptions = {
-  ecmaVersion: 6,
+  ecmaVersion: 8,
+  sourceType: 'module',
   ecmaFeatures: {
     experimentalObjectRestSpread: true,
     jsx: true
@@ -27,7 +28,7 @@ require('babel-eslint');
 var CLASSNAME_ERROR_MESSAGE = 'Prop `className` is forbidden on Components';
 var STYLE_ERROR_MESSAGE = 'Prop `style` is forbidden on Components';
 
-var ruleTester = new RuleTester();
+var ruleTester = new RuleTester({parserOptions});
 ruleTester.run('forbid-component-props', rule, {
 
   valid: [{
@@ -37,8 +38,7 @@ ruleTester.run('forbid-component-props', rule, {
       '    return <div className="foo" />;',
       '  }',
       '});'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'var First = createReactClass({',
@@ -47,8 +47,7 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    options: [{forbid: ['style']}],
-    parserOptions: parserOptions
+    options: [{forbid: ['style']}]
   }, {
     code: [
       'var First = createReactClass({',
@@ -57,8 +56,7 @@ ruleTester.run('forbid-component-props', rule, {
       '    return <Foo bar="baz" />;',
       '  }',
       '});'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'var First = createReactClass({',
@@ -68,8 +66,7 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    options: [{forbid: ['style']}],
-    parserOptions: parserOptions
+    options: [{forbid: ['style']}]
   }, {
     code: [
       'var First = createReactClass({',
@@ -79,8 +76,7 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    options: [{forbid: ['style', 'foo']}],
-    parserOptions: parserOptions
+    options: [{forbid: ['style', 'foo']}]
   }, {
     code: [
       'var First = createReactClass({',
@@ -89,8 +85,7 @@ ruleTester.run('forbid-component-props', rule, {
       '    return <this.Foo bar="baz" />;',
       '  }',
       '});'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }, {
     code: [
       'class First extends createReactClass {',
@@ -99,15 +94,13 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '}'
     ].join('\n'),
-    options: [{forbid: ['style']}],
-    parserOptions: parserOptions
+    options: [{forbid: ['style']}]
   }, {
     code: [
       'const First = (props) => (',
       '  <this.Foo {...props} />',
       ');'
-    ].join('\n'),
-    parserOptions: parserOptions
+    ].join('\n')
   }],
 
   invalid: [{
@@ -119,7 +112,6 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    parserOptions: parserOptions,
     errors: [{
       message: CLASSNAME_ERROR_MESSAGE,
       line: 4,
@@ -135,7 +127,6 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    parserOptions: parserOptions,
     errors: [{
       message: STYLE_ERROR_MESSAGE,
       line: 4,
@@ -151,7 +142,6 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    parserOptions: parserOptions,
     options: [{forbid: ['className', 'style']}],
     errors: [{
       message: CLASSNAME_ERROR_MESSAGE,
@@ -168,7 +158,6 @@ ruleTester.run('forbid-component-props', rule, {
       '  }',
       '});'
     ].join('\n'),
-    parserOptions: parserOptions,
     options: [{forbid: ['className', 'style']}],
     errors: [{
       message: STYLE_ERROR_MESSAGE,