Fix: no-multiple-empty-lines errors when no line breaks (fixes eslint…

…#6449)
strawbrary · Jun 17, 2016 · 09a8809 · 09a8809
1 parent 414206c
commit 09a8809
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/rules/no-multiple-empty-lines.js b/lib/rules/no-multiple-empty-lines.js
@@ -98,7 +98,7 @@ module.exports = {
 
                 linesRangeStart.push(0);
                 lines.forEach(function(str, i) {
-                    var length = i < fullLines.length ? fullLines[i].length : 0,
+                    var length = Array.isArray(fullLines) && i < fullLines.length ? fullLines[i].length : 0,
                         trimmed = str.trim();
 
                     if ((firstNonBlankLine === -1) && (trimmed !== "")) {

diff --git a/tests/lib/rules/no-multiple-empty-lines.js b/tests/lib/rules/no-multiple-empty-lines.js
@@ -149,6 +149,10 @@ ruleTester.run("no-multiple-empty-lines", rule, {
         {
             code: "// valid 17\nvar a = 5;\n\n",
             options: [ { max: 0, maxEOF: 1 } ]
+        },
+        {
+            code: "var a = 5;",
+            options: [ { max: 1 } ]
         }
     ],