Add more tests

eslint · Nov 7, 2019 · cab04fd · cab04fd
1 parent 2ff0cae
commit cab04fd
Show file tree

Hide file tree

Showing 2 changed files with 206 additions and 2 deletions.
diff --git a/lib/rules/multiline-comment-style.js b/lib/rules/multiline-comment-style.js
@@ -286,15 +286,15 @@ module.exports = {
                                 fix(fixer) {
                                     const lineStartIndex = sourceCode.getIndexFromLoc({ line: lineNumber, column: 0 });
                                     const [linePrefix, whitespaceBefore, whitespaceAfter] = lineText.match(/^(\s*)\*?(\s*)/u);
-                                    const commentStartIndex = lineStartIndex + linePrefix.length;
-                                    const leadingWhitespace = whitespaceAfter || ` ${
+                                    const leadingWhitespace = whitespaceAfter.length && whitespaceAfter || ` ${
                                         whitespaceBefore.startsWith(expectedLeadingWhitespace)
                                             ? whitespaceBefore.replace(expectedLeadingWhitespace, "")
                                             : ""
                                     }`;
                                     const replacementText = lineNumber === firstComment.loc.end.line || lineText.length === linePrefix.length
                                         ? expectedLinePrefix
                                         : `${expectedLinePrefix}${leadingWhitespace}`;
+                                    const commentStartIndex = lineStartIndex + linePrefix.length;
 
                                     return fixer.replaceTextRange([lineStartIndex, commentStartIndex], replacementText);
                                 }

diff --git a/tests/lib/rules/multiline-comment-style.js b/tests/lib/rules/multiline-comment-style.js
@@ -1160,6 +1160,210 @@ ruleTester.run("multiline-comment-style", rule, {
             `,
             options: ["separate-lines"],
             errors: [{ messageId: "expectedLines", line: 2 }]
+        },
+        {
+            code: `
+                /*
+                 * foo
+                 *
+                 * bar
+                 */
+            `,
+            output: `
+                /* foo
+${"                   "}
+                   bar */
+            `,
+            options: ["bare-block"],
+            errors: [{ messageId: "expectedBareBlock", line: 2 }]
+        },
+        {
+            code: `
+                /*
+                 * foo
+                 *${" "}
+                 * bar
+                 */
+            `,
+            output: `
+                /* foo
+${"                   "}
+                   bar */
+            `,
+            options: ["bare-block"],
+            errors: [{ messageId: "expectedBareBlock", line: 2 }]
+        },
+        {
+            code: `
+                // foo
+                //
+                // bar
+            `,
+            output: `
+                /*
+                 * foo
+                 *${" "}
+                 * bar
+                 */
+            `,
+            options: ["starred-block"],
+            errors: [{ messageId: "expectedBlock", line: 2 }]
+        },
+        {
+            code: `
+                // foo
+                //${" "}
+                // bar
+            `,
+            output: `
+                /*
+                 * foo
+                 *${" "}
+                 * bar
+                 */
+            `,
+            options: ["starred-block"],
+            errors: [{ messageId: "expectedBlock", line: 2 }]
+        },
+        {
+            code: `
+                // foo
+                //
+                // bar
+            `,
+            output: `
+                /* foo
+${"                   "}
+                   bar */
+            `,
+            options: ["bare-block"],
+            errors: [{ messageId: "expectedBlock", line: 2 }]
+        },
+        {
+            code: `
+                // foo
+                //${" "}
+                // bar
+            `,
+            output: `
+                /* foo
+${"                   "}
+                   bar */
+            `,
+            options: ["bare-block"],
+            errors: [{ messageId: "expectedBlock", line: 2 }]
+        },
+        {
+            code: `
+                /* foo
+
+                   bar */
+            `,
+            output: `
+                // foo
+                //${" "}
+                // bar${" "}
+            `,
+            options: ["separate-lines"],
+            errors: [{ messageid: "expectedlines", line: 2 }]
+        },
+        {
+            code: `
+                /* foo
+${"                   "}
+                   bar */
+            `,
+            output: `
+                // foo
+                //${" "}
+                // bar${" "}
+            `,
+            options: ["separate-lines"],
+            errors: [{ messageid: "expectedlines", line: 2 }]
+        },
+        {
+            code: `
+                /* foo
+
+                   bar */
+            `,
+            output: `
+                /*
+                 * foo
+                 *
+                 * bar
+                 */
+            `,
+            options: ["starred-block"],
+            errors: [
+                { messageId: "startNewline", line: 2 },
+                { messageId: "missingStar", line: 3 },
+                { messageId: "missingStar", line: 4 },
+                { messageId: "endNewline", line: 4 }
+            ]
+        },
+        {
+            code: `
+                /* foo
+${"                   "}
+                   bar */
+            `,
+            output: `
+                /*
+                 * foo
+                 *
+                 * bar
+                 */
+            `,
+            options: ["starred-block"],
+            errors: [
+                { messageId: "startNewline", line: 2 },
+                { messageId: "missingStar", line: 3 },
+                { messageId: "missingStar", line: 4 },
+                { messageId: "endNewline", line: 4 }
+            ]
+        },
+        {
+            code: `
+                /*foo
+
+                  bar */
+            `,
+            output: `
+                /*
+                 *foo
+                 *
+                 *bar${" "}
+                 */
+            `,
+            options: ["starred-block"],
+            errors: [
+                { messageId: "startNewline", line: 2 },
+                { messageId: "missingStar", line: 3 },
+                { messageId: "missingStar", line: 4 },
+                { messageId: "endNewline", line: 4 }
+            ]
+        },
+        {
+            code: `
+                /*foo
+${"                   "}
+                  bar */
+            `,
+            output: `
+                /*
+                 *foo
+                 *
+                 *bar${" "}
+                 */
+            `,
+            options: ["starred-block"],
+            errors: [
+                { messageId: "startNewline", line: 2 },
+                { messageId: "missingStar", line: 3 },
+                { messageId: "missingStar", line: 4 },
+                { messageId: "endNewline", line: 4 }
+            ]
         }
     ]
 });