Add tests for affected rules

Author: arensman

tests/lib/rules/no-arrow-tests.js

@@ -190,6 +190,26 @@ ruleTester.run("no-arrow-tests", rule, {
                 },
             ],
         },
+        {
+            code: "QUnit.module('module', { before: () => {} });",
+            output: "QUnit.module('module', { before: function() {} });",
+            errors: [
+                {
+                    messageId: "noArrowFunction",
+                    type: "ArrowFunctionExpression",
+                },
+            ],
+        },
+        {
+            code: "QUnit.module('module', { after: () => {} });",
+            output: "QUnit.module('module', { after: function() {} });",
+            errors: [
+                {
+                    messageId: "noArrowFunction",
+                    type: "ArrowFunctionExpression",
+                },
+            ],
+        },
         {
             code: "module('module', { setup: () => {} });",
             output: "module('module', { setup: function() {} });",

tests/lib/rules/no-qunit-start-in-tests.js

@@ -53,6 +53,14 @@ ruleTester.run("no-qunit-start-in-tests", rule, {
             code: 'QUnit.module("module", { afterEach: function() { QUnit.start(); } });',
             errors: [createError("afterEach hook")],
         },
+        {
+            code: 'QUnit.module("module", { before: function() { QUnit.start(); } });',
+            errors: [createError("before hook")],
+        },
+        {
+            code: 'QUnit.module("module", { after: function() { QUnit.start(); } });',
+            errors: [createError("after hook")],
+        },
         {
             code: 'QUnit.module("module", { setup: function() { QUnit.start(); } });',
             errors: [createError("setup hook")],

tests/lib/rules/no-setup-teardown.js

@@ -29,6 +29,12 @@ ruleTester.run("no-setup-teardown", rule, {
         // afterEach
         "QUnit.module('Module', { afterEach: function () {} });",
 
+        // before
+        "QUnit.module('Module', { before: function () {} });",
+
+        // after
+        "QUnit.module('Module', { after: function () {} });",
+
         // both
         "QUnit.module('Module', { beforeEach: function () {}, afterEach: function () {} });",
 

tests/lib/rules/resolve-async.js

@@ -185,6 +185,10 @@ ruleTester.run("resolve-async", rule, {
             code: "QUnit.module('name', { teardown: function () { QUnit.stop(); } });",
             errors: [createNeedStartCallsMessage("Property")],
         },
+        {
+            code: "QUnit.module('name', { before: function () { QUnit.stop(); } });",
+            errors: [createNeedStartCallsMessage("Property")],
+        },
         {
             code: "QUnit.module('name', { beforeEach: function () { QUnit.stop(); } });",
             errors: [createNeedStartCallsMessage("Property")],
@@ -193,6 +197,10 @@ ruleTester.run("resolve-async", rule, {
             code: "QUnit.module('name', { afterEach: function () { QUnit.stop(); } });",
             errors: [createNeedStartCallsMessage("Property")],
         },
+        {
+            code: "QUnit.module('name', { after: function () { QUnit.stop(); } });",
+            errors: [createNeedStartCallsMessage("Property")],
+        },
 
         // Multiple start() calls needed
         {