Add more tests for "assignment like" node (#19447)

src/language-js/comments/attach/handle-if-statement-comments.js

@@ -100,7 +100,7 @@ function handleCommentsBetween({
   // - treat as a dangling comment otherwise
   if (
     !isConsequentBlockStatement &&
-    isSingleLineComment &&
+    isSingleLineComment(comment, text) &&
     // Comment and `precedingNode` are on same line
     !hasNewlineInRange(text, locEnd(precedingNode), locStart(comment))
   ) {

src/language-js/comments/attach/union-type.js

@@ -1,16 +1,14 @@
 import { addLeadingComment } from "../../../main/comments/utilities.js";
 import { locEnd, locStart } from "../../location/index.js";
-import { isBlockComment } from "../../utilities/comment-types.js";
 import { isPrettierIgnoreComment } from "../../utilities/is-prettier-ignore-comment.js";
 import { isUnionType } from "../../utilities/node-types.js";
 import { stripComments } from "../../utilities/strip-comments.js";
-import { isSingleLineComment } from "../attach/utilities.js";
+import { isSingleLineBlockComment } from "../attach/utilities.js";
 
 function shouldAttachToUnionTypeFirstElement(node, { comment, text, options }) {
   if (
     isUnionType(node) &&
-    isBlockComment(comment) &&
-    isSingleLineComment(comment, text) &&
+    isSingleLineBlockComment(comment, text) &&
     !isPrettierIgnoreComment(comment)
   ) {
     const text = stripComments(options);

src/language-js/comments/attach/utilities.js

@@ -4,7 +4,10 @@ import {
 } from "../../../main/comments/utilities.js";
 import hasNewlineInRange from "../../../utilities/has-newline-in-range.js";
 import { locEnd, locStart } from "../../location/index.js";
-import { isLineComment } from "../../utilities/comment-types.js";
+import {
+  isBlockComment,
+  isLineComment,
+} from "../../utilities/comment-types.js";
 
 /**
 @import {Node, Comment, NodeMap} from "../../types/estree.js";
@@ -15,10 +18,18 @@ import { isLineComment } from "../../utilities/comment-types.js";
 @param {string} text
 @returns {boolean}
 */
-const isSingleLineComment = (comment, text) =>
-  isLineComment(comment) ||
+const isSingleLineBlockComment = (comment, text) =>
+  isBlockComment(comment) &&
   !hasNewlineInRange(text, locStart(comment), locEnd(comment));
 
+/**
+@param {Comment} comment
+@param {string} text
+@returns {boolean}
+*/
+const isSingleLineComment = (comment, text) =>
+  isLineComment(comment) || isSingleLineBlockComment(comment, text);
+
 /**
  * @param {Node} node
  * @returns {void}
@@ -50,5 +61,6 @@ function addBlockStatementFirstComment(node, comment) {
 export {
   addBlockOrNotComment,
   addBlockStatementFirstComment,
+  isSingleLineBlockComment,
   isSingleLineComment,
 };

tests/format/js/comments/__snapshots__/format.test.js.snap

@@ -7216,7 +7216,7 @@ try {
 ================================================================================
 `;
 
-exports[`variable_declarator.js - {"semi":false} format 1`] = `
+exports[`variable-declarator.js - {"semi":false} format 1`] = `
 ====================================options=====================================
 parsers: ["babel", "flow", "typescript"]
 semi: false
@@ -7366,7 +7366,7 @@ const foo3 = 123
 ================================================================================
 `;
 
-exports[`variable_declarator.js format 1`] = `
+exports[`variable-declarator.js format 1`] = `
 ====================================options=====================================
 parsers: ["babel", "flow", "typescript"]
                                                       printWidth: 80 (default) |

tests/format/js/comments/assignment/__snapshots__/format.test.js.snap

@@ -0,0 +1,65 @@
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
+
+exports[`variable-declarator.js format 1`] = `
+====================================options=====================================
+parsers: ["babel", "flow", "typescript"]
+                                                      printWidth: 80 (default) |
+=====================================input======================================
+let a1 // commment
+ = 1;
+let a12
+// commment
+ = 1;
+let a2 /* commment */
+ = 1;
+let a22
+/* commment */
+ = 1;
+let a3 /*
+commment */
+ = 1;
+let a32
+/*
+commment */
+ = 1;
+let a4 = // comment
+1;
+let a42 =
+// comment
+1;
+let a5 = /* commment */
+1;
+let a52 =
+/* commment */
+1;
+
+=====================================output=====================================
+let a1 = 1; // commment
+let a12 =
+  // commment
+  1;
+let a2 = /* commment */ 1;
+let a22 =
+  /* commment */
+  1;
+let a3 =
+  /*
+commment */
+  1;
+let a32 =
+  /*
+commment */
+  1;
+let a4 =
+  // comment
+  1;
+let a42 =
+  // comment
+  1;
+let a5 = /* commment */ 1;
+let a52 =
+  /* commment */
+  1;
+
+================================================================================
+`;

tests/format/js/comments/assignment/format.test.js

@@ -0,0 +1 @@
+runFormatTest(import.meta, ["babel", "flow", "typescript"]);

tests/format/js/comments/assignment/variable-declarator.js

@@ -0,0 +1,27 @@
+let a1 // commment
+ = 1;
+let a12
+// commment
+ = 1;
+let a2 /* commment */
+ = 1;
+let a22
+/* commment */
+ = 1;
+let a3 /*
+commment */
+ = 1;
+let a32
+/*
+commment */
+ = 1;
+let a4 = // comment
+1;
+let a42 =
+// comment
+1;
+let a5 = /* commment */
+1;
+let a52 =
+/* commment */
+1;