Fix: Curly rule doesn't account for leading comment (fixes #7538) (#7539

) * Fix: Curly rule doesn't account for leading comment (fixes #7538) * PR feedback * simplify check * check .length > 0
eslint · Nov 9, 2016 · 0d60db7 · 0d60db7
1 parent 5003b1c
commit 0d60db7
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/docs/rules/curly.md b/docs/rules/curly.md
@@ -183,6 +183,11 @@ while (true) {
 for (var i = 0; foo; i++) {
     doSomething();
 }
+
+if (foo) {
+    // some comment
+    bar();
+}
 ```
 
 Examples of **correct** code for the `"multi-or-nest"` option:

diff --git a/lib/rules/curly.js b/lib/rules/curly.js
@@ -290,7 +290,9 @@ module.exports = {
                 }
             } else if (multiOrNest) {
                 if (hasBlock && body.body.length === 1 && isOneLiner(body.body[0])) {
-                    expected = false;
+                    const leadingComments = sourceCode.getComments(body.body[0]).leading;
+
+                    expected = leadingComments.length > 0;
                 } else if (!isOneLiner(body)) {
                     expected = true;
                 }

diff --git a/tests/lib/rules/curly.js b/tests/lib/rules/curly.js
@@ -126,6 +126,14 @@ ruleTester.run("curly", rule, {
             code: "if (foo) \n quz = true;",
             options: ["multi-or-nest"]
         },
+        {
+            code: "if (foo) { \n // line of comment \n quz = true; \n }",
+            options: ["multi-or-nest"]
+        },
+        {
+            code: "// line of comment \n if (foo) \n quz = true; \n",
+            options: ["multi-or-nest"]
+        },
         {
             code: "while (true) \n doSomething();",
             options: ["multi-or-nest"]