requireNewlineBeforeSingleStatementsInIf: don't blow on single if branch

Fixes #2174
jscs-dev · Mar 11, 2016 · fbe9bf4 · fbe9bf4
1 parent bbde83c
commit fbe9bf4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/lib/rules/require-newline-before-single-statements-in-if.js b/lib/rules/require-newline-before-single-statements-in-if.js
@@ -82,6 +82,10 @@ module.exports.prototype = {
             var consequentNode = node.consequent;
             var alternateNode = node.alternate;
 
+            if (!alternateNode) {
+                return;
+            }
+
             if (isExpressionStatement(consequentNode)) {
                 assertDifferentLine(consequentNode, getToken(consequentNode, 'Keyword', 'previousSibling'));
             }

diff --git a/test/specs/rules/require-newline-before-single-statements-in-if.js b/test/specs/rules/require-newline-before-single-statements-in-if.js
@@ -14,6 +14,10 @@ describe('rules/require-newline-before-single-statements-in-if', function() {
             checker.configure({ requireNewlineBeforeSingleStatementsInIf: true });
         });
 
+        it('should not blow up for single branch `if`', function() {
+            expect(checker.checkString('if (x) true')).to.have.no.errors();
+        });
+
         it('should ignore if in a block statment', function() {
             expect(checker.checkString('if (x) {\n doX();\n} else {\n doY();\n}')).to.have.no.errors();
         });