chore(rules): avoid passing unnecessary info into afters (#4049)

dequelabs · Jun 27, 2023 · ab4879c · ab4879c
1 parent 16c5cfa
commit ab4879c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/lib/core/base/rule.js b/lib/core/base/rule.js
@@ -537,9 +537,9 @@ Rule.prototype.after = function after(result, options) {
   var ruleID = this.id;
   afterChecks.forEach(check => {
     var beforeResults = findCheckResults(result.nodes, check.id);
-    var option = getCheckOption(check, ruleID, options);
+    var checkOption = getCheckOption(check, ruleID, options);
 
-    var afterResults = check.after(beforeResults, option);
+    var afterResults = check.after(beforeResults, checkOption.options);
 
     if (this.reviewOnFail) {
       afterResults.forEach(checkResult => {

diff --git a/test/core/base/rule.js b/test/core/base/rule.js
@@ -1483,11 +1483,7 @@ describe('Rule', function () {
                 id: 'cats',
                 enabled: true,
                 after: function (results, options) {
-                  assert.deepEqual(options, {
-                    enabled: true,
-                    options: { dogs: true },
-                    absolutePaths: undefined
-                  });
+                  assert.deepEqual(options, { dogs: true });
                   success = true;
                   return results;
                 }