add tests and fix bugs

Author: ota-meshi

lib/internal/utils.js

@@ -181,8 +181,9 @@ function divideDirectiveComment(value) {
  * @returns {object} The source code location.
  */
 function getLoc(context, nodeOrToken) {
-    const sourceCode = context.sourceCode || context.getSourceCode?.()
-    if (typeof sourceCode?.getLoc === "function") {
+    const sourceCode =
+        context.sourceCode || (context.getSourceCode && context.getSourceCode())
+    if (sourceCode && typeof sourceCode.getLoc === "function") {
         return sourceCode.getLoc(nodeOrToken)
     }
     return nodeOrToken.loc

lib/rules/disable-enable-pair.js

@@ -44,7 +44,8 @@ module.exports = {
         /** @type {import('@eslint/core').TextSourceCode} */
         const sourceCode = context.sourceCode || context.getSourceCode()
 
-        const firstToken = sourceCode.ast?.tokens?.[0]
+        const firstToken =
+            sourceCode.ast && sourceCode.ast.tokens && sourceCode.ast.tokens[0]
 
         if (allowWholeFile && !firstToken) {
             return {}

lib/rules/no-aggregating-enable.js

@@ -34,7 +34,7 @@ module.exports = {
 
             if (count >= 2) {
                 context.report({
-                    loc: utils.toForceLocation(comment.loc),
+                    loc: utils.toForceLocation(utils.getLoc(context, comment)),
                     messageId: "aggregatingEnable",
                     data: { count },
                 })

tests/lib/rules/no-aggregating-enable.js

@@ -32,6 +32,22 @@ tester.run("no-aggregating-enable", rule, {
             /*eslint-enable no-redeclare*/
             /*eslint-enable no-shadow*/
         `,
+        // Language plugin
+        ...(semver.satisfies(Linter.version, ">=9.6.0")
+            ? [
+                  {
+                      code: `
+            /*eslint-disable no-redeclare, no-shadow*/
+            /*eslint-enable no-redeclare*/
+            /*eslint-enable no-shadow*/
+            a {}`,
+                      plugins: {
+                          css: require("@eslint/css").default,
+                      },
+                      language: "css/css",
+                  },
+              ]
+            : []),
     ],
     invalid: [
         {
@@ -80,5 +96,24 @@ tester.run("no-aggregating-enable", rule, {
                   },
               ]
             : []),
+        // Language plugin
+        ...(semver.satisfies(Linter.version, ">=9.6.0")
+            ? [
+                  {
+                      code: `
+                /*eslint-disable no-redeclare*/
+                /*eslint-disable no-shadow*/
+                /*eslint-enable*/
+            a {}`,
+                      plugins: {
+                          css: require("@eslint/css").default,
+                      },
+                      language: "css/css",
+                      errors: [
+                          "This `eslint-enable` comment affects 2 `eslint-disable` comments. An `eslint-enable` comment should be for an `eslint-disable` comment.",
+                      ],
+                  },
+              ]
+            : []),
     ],
 })