Fix 2.2.0 newlineAfterBlock regression. Closes #274

lesshint · Oct 28, 2016 · 199e125 · 199e125
1 parent 1af1213
commit 199e125
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 2 deletions.
diff --git a/lib/linters/newline_after_block.js b/lib/linters/newline_after_block.js
@@ -6,7 +6,7 @@ module.exports = {
     message: 'All blocks should be followed by a new line.',
 
     lint: function newlineAfterBlockLinter (config, node) {
-        var regexp = /\n[ \t]*\n?/;
+        var regexp = /\n[ \t]*\n/;
         var parent = node.parent;
         var prev;
 

diff --git a/test/specs/linters/newline_after_block.js b/test/specs/linters/newline_after_block.js
@@ -13,7 +13,25 @@ describe('lesshint', function () {
             });
         });
 
-        it('should not allow adjacent blocks without a new line', function () {
+        it('should not allow adjacent blocks without a blank line', function () {
+            var source = [
+                '.foo { color: red; }',
+                '.bar { color: blue; }'
+            ];
+            var expected = [{
+                message: 'All blocks should be followed by a new line.'
+            }];
+
+            source = source.join('\n');
+
+            return spec.parse(source, function (ast) {
+                var result = spec.linter.lint({}, ast.root.last);
+
+                expect(result).to.deep.equal(expected);
+            });
+        });
+
+        it('should not allow adjacent blocks on the same line', function () {
             var source = [
                 '.foo { color: red; }',
                 '.bar { color: blue; }'