Build: Add wrapper around ESLint RuleTester

cvisco · Aug 13, 2016 · 9bead61 · 9bead61
1 parent 5d4b5c8
commit 9bead61
Show file tree

Hide file tree

Showing 22 changed files with 50 additions and 84 deletions.
diff --git a/tests/lib/rules/amd-function-arity.js b/tests/lib/rules/amd-function-arity.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/amd-function-arity");
 
@@ -37,9 +37,7 @@ function makeTooFewParamsError(funcName, expected, actual) {
     return { message: message };
 }
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("amd-function-arity", rule, {
+testRule("amd-function-arity", rule, {
 
     valid: [
         // Dependency count and parameter counts equal-- always valid

diff --git a/tests/lib/rules/enforce-define.js b/tests/lib/rules/enforce-define.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/enforce-define");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "Program"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("enforce-define", rule, {
+testRule("enforce-define", rule, {
 
     valid: [
 

diff --git a/tests/lib/rules/no-amd-define.js b/tests/lib/rules/no-amd-define.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-amd-define");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-amd-define", rule, {
+testRule("no-amd-define", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-assign-exports.js b/tests/lib/rules/no-assign-exports.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-assign-exports");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "AssignmentExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-assign-exports", rule, {
+testRule("no-assign-exports", rule, {
 
     valid: [
         fixtures.CJS_WITH_EXPORTS,

diff --git a/tests/lib/rules/no-assign-require.js b/tests/lib/rules/no-assign-require.js
@@ -5,15 +5,13 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-assign-require");
 
 const MESSAGE = "Invalid assignment to `require`.";
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-assign-require", rule, {
+testRule("no-assign-require", rule, {
 
     valid: [
         fixtures.ASSIGN_TO_FOO_REQUIRE

diff --git a/tests/lib/rules/no-commonjs-exports.js b/tests/lib/rules/no-commonjs-exports.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-commonjs-exports");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "AssignmentExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-commonjs-exports", rule, {
+testRule("no-commonjs-exports", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-commonjs-module-exports.js b/tests/lib/rules/no-commonjs-module-exports.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-commonjs-module-exports");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "AssignmentExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-commonjs-module-exports", rule, {
+testRule("no-commonjs-module-exports", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-commonjs-return.js b/tests/lib/rules/no-commonjs-return.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-commonjs-return");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "ReturnStatement"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-commonjs-return", rule, {
+testRule("no-commonjs-return", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-commonjs-wrapper.js b/tests/lib/rules/no-commonjs-wrapper.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-commonjs-wrapper");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-commonjs-wrapper", rule, {
+testRule("no-commonjs-wrapper", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-conditional-require.js b/tests/lib/rules/no-conditional-require.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-conditional-require");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-conditional-require", rule, {
+testRule("no-conditional-require", rule, {
 
     valid: [
         fixtures.AMD_REQUIRE,

diff --git a/tests/lib/rules/no-dynamic-require.js b/tests/lib/rules/no-dynamic-require.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-dynamic-require");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-dynamic-require", rule, {
+testRule("no-dynamic-require", rule, {
 
     valid: [
         fixtures.AMD_REQUIRE,

diff --git a/tests/lib/rules/no-function-define.js b/tests/lib/rules/no-function-define.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-function-define");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-function-define", rule, {
+testRule("no-function-define", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-invalid-define.js b/tests/lib/rules/no-invalid-define.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-invalid-define");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-invalid-define", rule, {
+testRule("no-invalid-define", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-invalid-require.js b/tests/lib/rules/no-invalid-require.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-invalid-require");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-invalid-require", rule, {
+testRule("no-invalid-require", rule, {
 
     valid: [
         fixtures.AMD_REQUIRE,

diff --git a/tests/lib/rules/no-js-extension.js b/tests/lib/rules/no-js-extension.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-js-extension");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "Literal"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-js-extension", rule, {
+testRule("no-js-extension", rule, {
 
     valid: [
         fixtures.BAD_REQUIRE_EMPTY,

diff --git a/tests/lib/rules/no-multiple-define.js b/tests/lib/rules/no-multiple-define.js
@@ -5,13 +5,11 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-multiple-define");
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-multiple-define", rule, {
+testRule("no-multiple-define", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-named-define.js b/tests/lib/rules/no-named-define.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-named-define");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-named-define", rule, {
+testRule("no-named-define", rule, {
 
     valid: [
         fixtures.OBJECT_DEFINE,

diff --git a/tests/lib/rules/no-object-define.js b/tests/lib/rules/no-object-define.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-object-define");
 
@@ -14,9 +14,7 @@ const ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-object-define", rule, {
+testRule("no-object-define", rule, {
 
     valid: [
         fixtures.FUNCTION_DEFINE,

diff --git a/tests/lib/rules/no-require-tourl.js b/tests/lib/rules/no-require-tourl.js
@@ -5,16 +5,14 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const util = require("util");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/no-require-tourl");
 
 const MESSAGE = "Use of `require.%s` is not allowed.";
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("no-require-tourl", rule, {
+testRule("no-require-tourl", rule, {
 
     valid: [
         fixtures.AMD_REQUIRE_RELATIVE,

diff --git a/tests/lib/rules/one-dependency-per-line.js b/tests/lib/rules/one-dependency-per-line.js
@@ -5,7 +5,7 @@
 
 "use strict";
 
-const RuleTester = require("eslint").RuleTester;
+const testRule = require("../../rule-tester");
 const fixtures = require("../../fixtures");
 const rule = require("../../../lib/rules/one-dependency-per-line");
 
@@ -29,9 +29,7 @@ const NEVER_NAMES_ERROR = {
     type: "CallExpression"
 };
 
-const ruleTester = new RuleTester();
-
-ruleTester.run("one-dependency-per-line", rule, {
+testRule("one-dependency-per-line", rule, {
 
     valid: [