Breaking: Switch class option for lines-around-comment (fixes eslin…

…t#8564)
Mardak · May 8, 2017 · 4ba7f79 · 4ba7f79
1 parent c70b0ed
commit 4ba7f79
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/rules/lines-around-comment.js b/lib/rules/lines-around-comment.js
@@ -212,7 +212,7 @@ module.exports = {
          * @returns {boolean} True if the comment is at block start.
          */
         function isCommentAtBlockStart(token) {
-            return isCommentAtParentStart(token, "ClassBody") || isCommentAtParentStart(token, "BlockStatement") || isCommentAtParentStart(token, "SwitchCase");
+            return isCommentAtParentStart(token, "BlockStatement") || isCommentAtParentStart(token, "SwitchCase");
         }
 
         /**
@@ -221,7 +221,7 @@ module.exports = {
          * @returns {boolean} True if the comment is at block end.
          */
         function isCommentAtBlockEnd(token) {
-            return isCommentAtParentEnd(token, "ClassBody") || isCommentAtParentEnd(token, "BlockStatement") || isCommentAtParentEnd(token, "SwitchCase") || isCommentAtParentEnd(token, "SwitchStatement");
+            return isCommentAtParentEnd(token, "BlockStatement") || isCommentAtParentEnd(token, "SwitchCase") || isCommentAtParentEnd(token, "SwitchStatement");
         }
 
         /**
@@ -230,7 +230,7 @@ module.exports = {
          * @returns {boolean} True if the comment is at object start.
          */
         function isCommentAtObjectStart(token) {
-            return isCommentAtParentStart(token, "ObjectExpression") || isCommentAtParentStart(token, "ObjectPattern");
+            return isCommentAtParentStart(token, "ObjectExpression") || isCommentAtParentStart(token, "ObjectPattern") || isCommentAtParentStart(token, "ClassBody");
         }
 
         /**
@@ -239,7 +239,7 @@ module.exports = {
          * @returns {boolean} True if the comment is at object end.
          */
         function isCommentAtObjectEnd(token) {
-            return isCommentAtParentEnd(token, "ObjectExpression") || isCommentAtParentEnd(token, "ObjectPattern");
+            return isCommentAtParentEnd(token, "ObjectExpression") || isCommentAtParentEnd(token, "ObjectPattern") || isCommentAtParentEnd(token, "ClassBody");
         }
 
         /**

diff --git a/tests/lib/rules/lines-around-comment.js b/tests/lib/rules/lines-around-comment.js
@@ -221,7 +221,7 @@ ruleTester.run("lines-around-comment", rule, {
         },
         {
             code: "class A {\n/**\n* hi\n */\nconstructor() {}\n}",
-            options: [{ allowBlockStart: true }],
+            options: [{ allowObjectStart: true }],
             parserOptions: { ecmaVersion: 6 }
         },
         {
@@ -414,7 +414,7 @@ ruleTester.run("lines-around-comment", rule, {
             code: "class B {\nconstructor() {}\n\n/**\n* hi\n */\n}",
             options: [{
                 afterBlockComment: true,
-                allowBlockEnd: true
+                allowObjectEnd: true
             }],
             parserOptions: { ecmaVersion: 6 }
         },