fix: Stricter RuleTester validations

Author: iChenLei

lib/rules/no-unexpected-string-bool.js

@@ -24,6 +24,10 @@ module.exports = {
           if (node.value === "true" || node.value === "false") {
             context.report({
               node,
+              data: {
+                true: "true",
+                false: "false",
+              },
               messageId: node.value === "true" ? "trueString" : "falseString",
             });
           }

lib/rules/omit-bool-attributes.js

@@ -24,7 +24,11 @@ module.exports = {
             context.report({
               node,
               messageId: "omitWarn",
-              data: { attrKey: node.key },
+              data: {
+                attrKey: node.key,
+                true: "true",
+                false: "false",
+              },
             });
           }
         }

lib/rules/report-interpolation-error.js

@@ -24,7 +24,7 @@ module.exports = {
         }
         try {
           espreeParser = require("espree");
-        // eslint-disable-next-line no-unused-vars
+          // eslint-disable-next-line no-unused-vars
         } catch (_) {
           // ...
         }
@@ -38,13 +38,13 @@ module.exports = {
               espreeParser.parse(`({${node.value}})`, {
                 ecmaVersion: espreeParser.latestEcmaVersion,
               });
-            // eslint-disable-next-line no-unused-vars
+              // eslint-disable-next-line no-unused-vars
             } catch (_) {
               try {
                 espreeParser.parse(`(${node.value})`, {
                   ecmaVersion: espreeParser.latestEcmaVersion,
                 });
-              // eslint-disable-next-line no-unused-vars
+                // eslint-disable-next-line no-unused-vars
               } catch (_) {
                 context.report({
                   node,

tests/rule-tester-compat.js

@@ -1,12 +1,12 @@
 const eslint = require("eslint");
 const semver = require("semver");
 
-const ruleTester = semver.gte(eslint.Linter.version, '9.0.0-0') ?
-    new eslint.RuleTester({
-        languageOptions: {
-            parser: require("@wxml/parser"),
-        },
-    }) :
-    new eslint.RuleTester({ parser: require.resolve("@wxml/parser") });
+const ruleTester = semver.gte(eslint.Linter.version, "9.0.0-0")
+  ? new eslint.RuleTester({
+      languageOptions: {
+        parser: require("@wxml/parser"),
+      },
+    })
+  : new eslint.RuleTester({ parser: require.resolve("@wxml/parser") });
 
-module.exports.RuleTester = ruleTester;
+module.exports.RuleTester = ruleTester;

tests/rules/colon-style-event-binding.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/colon-style-event-binding");
 
-
-
 tester.run("colon-style-event-binding", rule, {
   valid: [
     {

tests/rules/empty-tag-self-closing.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/empty-tag-self-closing");
 
-
-
 tester.run("empty-tag-self-closing", rule, {
   valid: [
     {

tests/rules/event-binding-style.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/event-binding-style");
 
-
-
 tester.run("event-binding-style", rule, {
   valid: [
     {

tests/rules/forbid-attributes.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/forbid-attributes");
 
-
-
 tester.run("forbid-attributes", rule, {
   valid: [
     {

tests/rules/forbid-tags.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/forbid-tags");
 
-
-
 tester.run("forbid-tags", rule, {
   valid: [
     {

tests/rules/max-depth.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/max-depth");
 
-
-
 tester.run("max-depth", rule, {
   valid: [
     {

tests/rules/max-len.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/max-len");
 
-
-
 tester.run("max-len", rule, {
   valid: [
     {

tests/rules/max-lines.js

@@ -3,8 +3,6 @@ const path = require("path");
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/max-lines");
 
-
-
 function readFixureFiles(filename) {
   if (filename) {
     return fs.readFileSync(

tests/rules/no-const-and-let-in-wxs.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-const-and-let-in-wxs");
 
-
-
 tester.run("no-const-and-let-in-wxs", rule, {
   valid: [
     {

tests/rules/no-dot-this-in-wx-key.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-dot-this-in-wx-key");
 
-
-
 tester.run("no-dot-this-in-wx-key", rule, {
   valid: [
     {

tests/rules/no-duplicate-attributes.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-duplicate-attributes");
 
-
-
 tester.run("no-duplicate-attributes", rule, {
   valid: [
     {

tests/rules/no-dynamic-wx-key.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-dynamic-wx-key");
 
-
-
 tester.run("no-dynamic-wx-key", rule, {
   valid: [
     {

tests/rules/no-inconsistent-tagname.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-inconsistent-tagname");
 
-
-
 tester.run("no-inconsistent-tagname", rule, {
   valid: [
     {

tests/rules/no-index-in-wx-key.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-index-in-wx-key");
 
-
-
 tester.run("no-index-in-wx-key", rule, {
   valid: [
     {

tests/rules/no-inline-wxs.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-inline-wxs");
 
-
-
 tester.run("no-inline-wxs", rule, {
   valid: [
     {

tests/rules/no-unexpected-string-bool.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-unexpected-string-bool");
 
-
-
 tester.run("no-unexpected-string-bool", rule, {
   valid: [
     {

tests/rules/no-unnecessary-block.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-unnecessary-block");
 
-
-
 tester.run("no-unnecessary-block", rule, {
   valid: [
     {

tests/rules/no-vue-directive.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-vue-directive");
 
-
-
 tester.run("no-vue-directive", rule, {
   valid: [
     {

tests/rules/no-wx-for-with-wx-else.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-wx-for-with-wx-else");
 
-
-
 tester.run("no-wx-for-with-wx-else", rule, {
   valid: [
     {

tests/rules/no-wx-for-with-wx-if.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-wx-for-with-wx-if");
 
-
-
 tester.run("no-wx-for-with-wx-if", rule, {
   valid: [
     {

tests/rules/no-wx-if-string.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/no-wx-if-string");
 
-
-
 tester.run("no-wx-if-string", rule, {
   valid: [
     {

tests/rules/omit-bool-attributes.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/omit-bool-attributes");
 
-
-
 tester.run("omit-bool-attributes", rule, {
   valid: [
     {

tests/rules/quotes.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/quotes");
 
-
-
 tester.run("quotes", rule, {
   valid: [
     {

tests/rules/report-interpolation-error.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/report-interpolation-error");
 
-
-
 tester.run("report-interpolation-error", rule, {
   valid: [
     {

tests/rules/report-wxml-syntax-error.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/report-wxml-syntax-error");
 
-
-
 tester.run("report-wxml-syntax-error", rule, {
   valid: [
     {

tests/rules/report-wxs-syntax-error.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/report-wxs-syntax-error");
 
-
-
 tester.run("report-wxs-syntax-error", rule, {
   valid: [
     {

tests/rules/required-attributes.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/required-attributes");
 
-
-
 tester.run("required-attributes", rule, {
   valid: [
     {

tests/rules/required-root-tag.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/required-root-tag");
 
-
-
 tester.run("required-root-tag", rule, {
   valid: [
     {

tests/rules/required-tags.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/required-tags");
 
-
-
 tester.run("required-tags", rule, {
   valid: [
     {

tests/rules/wx-key.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/wx-key");
 
-
-
 tester.run("wx-key", rule, {
   valid: [
     {

tests/rules/wxs-module-prop.js

@@ -1,8 +1,6 @@
 const tester = require("../rule-tester-compat").RuleTester;
 const rule = require("../../lib/rules/wxs-module-prop");
 
-
-
 tester.run("wxs-module-prop", rule, {
   valid: [
     {