finalNewline should ignore empty files

lesshint · Sep 23, 2015 · 306ade2 · 306ade2
1 parent 08eaeec
commit 306ade2
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/lib/linters/final_newline.js b/lib/linters/final_newline.js
@@ -17,7 +17,7 @@ module.exports = function (options) {
 
     maybeLine = node.content[node.content.length - 1];
 
-    if (maybeLine.type !== 'space' && maybeLine.content !== '\n') {
+    if (maybeLine && (maybeLine.type !== 'space' && maybeLine.content !== '\n')) {
         return {
             column: null,
             line: null,

diff --git a/test/specs/linters/final_newline.js b/test/specs/linters/final_newline.js
@@ -52,6 +52,23 @@ describe('lesshint', function () {
             assert.deepEqual(actual, expected);
         });
 
+        it('should ignore empty files', function () {
+            var source = '';
+            var ast;
+            var options = {
+                finalNewline: {
+                    enabled: true
+                }
+            };
+
+            ast = linter.parseAST(source);
+
+            assert.equal(null, finalNewline({
+                config: options,
+                node: ast
+            }));
+        });
+
         it('should return null when disabled', function () {
             var source = '.foo {}';
             var ast;