disallowPaddingNewlinesBeforeKeywords: should not ignore comments

Fixes #2237

Author: markelog

lib/rules/disallow-padding-newlines-before-keywords.js

@@ -88,7 +88,7 @@ module.exports.prototype = {
     check: function(file, errors) {
         file.iterateTokensByTypeAndValue('Keyword', this._keywords, function(token) {
             errors.assert.linesBetween({
-                token: file.getPrevToken(token),
+                token: file.getPrevToken(token, { includeComments: true }),
                 nextToken: token,
                 atMost: 1,
                 message: 'Keyword `' + token.value + '` should not have an empty line above it'

test/specs/rules/disallow-padding-newlines-before-keywords.js

@@ -73,5 +73,11 @@ describe('rules/disallow-padding-newlines-before-keywords', function() {
                     'function x() { var a;\n\nreturn; }'
                 )).to.have.one.validation.error.from('disallowPaddingNewlinesBeforeKeywords');
         });
+
+        it('should not ignore comments', function() {
+            expect(checker.checkString(
+                    'if (true) {} \n //comment \n else {}'
+                )).to.not.have.errors();
+        });
     });
 });