Chore: avoid using internal Linter APIs in RuleTester (refs #9161)

This updates `RuleTester` to use the public `Linter#defineRule` API rather than the private `Linter#on` API.
eslint · Aug 28, 2017 · e6e9f1c · e6e9f1c
1 parent d6e436f
commit e6e9f1c
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/lib/testers/rule-tester.js b/lib/testers/rule-tester.js
@@ -159,7 +159,8 @@ class RuleTester {
 
             // we have to clone because merge uses the first argument for recipient
             lodash.cloneDeep(defaultConfig),
-            testerConfig
+            testerConfig,
+            { rules: { "rule-tester/validate-ast": "error" } }
         );
 
         /**
@@ -333,13 +334,14 @@ class RuleTester {
              */
             linter.reset();
 
-            linter.on("Program", node => {
-                beforeAST = cloneDeeplyExcludesParent(node);
-            });
-
-            linter.on("Program:exit", node => {
-                afterAST = node;
-            });
+            linter.defineRule("rule-tester/validate-ast", () => ({
+                Program(node) {
+                    beforeAST = cloneDeeplyExcludesParent(node);
+                },
+                "Program:exit"(node) {
+                    afterAST = node;
+                }
+            }));
 
             // Freezes rule-context properties.
             const originalGet = linter.rules.get;