fix: fix errors in no-multiple-empty-lines (#240)

yeonjuan · Dec 1, 2024 · edcbf13 · edcbf13
1 parent 80a2bec
commit edcbf13
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 2 deletions.
diff --git a/packages/eslint-plugin/lib/rules/no-multiple-empty-lines.js b/packages/eslint-plugin/lib/rules/no-multiple-empty-lines.js
@@ -136,7 +136,7 @@ module.exports = {
           check(
             lines,
             // @ts-ignore
-            node.quasi.loc.start.line,
+            node.loc.start.line,
             getTemplateTokens(tokens)
           );
         }

diff --git a/packages/eslint-plugin/lib/rules/utils/node.js b/packages/eslint-plugin/lib/rules/utils/node.js
@@ -53,7 +53,7 @@ function isNodeTokensOnSameLine(node) {
  * @returns {boolean}
  */
 function isRangesOverlap(rangeA, rangeB) {
-  return rangeA[0] < rangeB[1] && rangeB[0] < rangeB[1];
+  return rangeA[0] < rangeB[1] && rangeB[0] < rangeA[1];
 }
 
 /**

diff --git a/packages/eslint-plugin/tests/rules/no-multiple-empty-lines.test.js b/packages/eslint-plugin/tests/rules/no-multiple-empty-lines.test.js
@@ -153,6 +153,31 @@ html\`<html>
 </div>
 </body>
 </html>\`
+`,
+      errors: [
+        {
+          message: "More than 2 blank lines not allowed.",
+        },
+      ],
+    },
+    {
+      code: `
+const html = /* html */\`
+<div style=\${"aa"}>
+    <div></div>
+
+
+
+
+</div>\`
+`,
+      output: `
+const html = /* html */\`
+<div style=\${"aa"}>
+    <div></div>
+
+
+</div>\`
 `,
       errors: [
         {