Move comment formatting test to separate file

Co-authored-by: jakebailey <5341706+jakebailey@users.noreply.github.com>

Author: Copilot

internal/format/api_test.go

@@ -36,51 +36,6 @@ func applyBulkEdits(text string, edits []core.TextChange) string {
 func TestFormat(t *testing.T) {
 	t.Parallel()
 
-	t.Run("format comment issue reproduction", func(t *testing.T) {
-		t.Parallel()
-		ctx := format.WithFormatCodeSettings(t.Context(), &format.FormatCodeSettings{
-			EditorSettings: format.EditorSettings{
-				TabSize:                4,
-				IndentSize:             4,
-				BaseIndentSize:         4,
-				NewLineCharacter:       "\n",
-				ConvertTabsToSpaces:    true,
-				IndentStyle:            format.IndentStyleSmart,
-				TrimTrailingWhitespace: true,
-			},
-			InsertSpaceBeforeTypeAnnotation: core.TSTrue,
-		}, "\n")
-		
-		// Original code that causes the bug
-		originalText := `class C {
-    /**
-     *
-    */
-    async x() {}
-}`
-		
-		sourceFile := parser.ParseSourceFile(ast.SourceFileParseOptions{
-			FileName: "/test.ts",
-			Path:     "/test.ts",
-		}, originalText, core.ScriptKindTS)
-		
-		// Apply formatting once
-		edits := format.FormatDocument(ctx, sourceFile)
-		firstFormatted := applyBulkEdits(originalText, edits)
-		
-		// Verify that the comment and async keyword are preserved
-		assert.Assert(t, strings.Contains(firstFormatted, "/**"))
-		assert.Assert(t, strings.Contains(firstFormatted, "*/"))
-		assert.Assert(t, strings.Contains(firstFormatted, "async"))
-		assert.Assert(t, !strings.Contains(firstFormatted, " /\n")) // Should not have broken comment
-		
-		// The main issue is fixed - the comment is preserved correctly
-		// Let's not test the second formatting for now as it might have separate issues
-		// assert.Assert(t, strings.Contains(secondFormatted, "async"))
-		// assert.Assert(t, !strings.Contains(secondFormatted, " /\n")) // Should not have broken comment
-		// assert.Assert(t, !strings.Contains(secondFormatted, "sync x()")) // Should not have corrupted async keyword
-	})
-
 	t.Run("format checker.ts", func(t *testing.T) {
 		t.Parallel()
 		ctx := format.WithFormatCodeSettings(t.Context(), &format.FormatCodeSettings{

internal/format/comment_test.go

@@ -0,0 +1,61 @@
+package format_test
+
+import (
+	"strings"
+	"testing"
+
+	"github.com/microsoft/typescript-go/internal/ast"
+	"github.com/microsoft/typescript-go/internal/core"
+	"github.com/microsoft/typescript-go/internal/format"
+	"github.com/microsoft/typescript-go/internal/parser"
+	"gotest.tools/v3/assert"
+)
+
+func TestCommentFormatting(t *testing.T) {
+	t.Parallel()
+
+	t.Run("format comment issue reproduction", func(t *testing.T) {
+		t.Parallel()
+		ctx := format.WithFormatCodeSettings(t.Context(), &format.FormatCodeSettings{
+			EditorSettings: format.EditorSettings{
+				TabSize:                4,
+				IndentSize:             4,
+				BaseIndentSize:         4,
+				NewLineCharacter:       "\n",
+				ConvertTabsToSpaces:    true,
+				IndentStyle:            format.IndentStyleSmart,
+				TrimTrailingWhitespace: true,
+			},
+			InsertSpaceBeforeTypeAnnotation: core.TSTrue,
+		}, "\n")
+
+		// Original code that causes the bug
+		originalText := `class C {
+    /**
+     *
+    */
+    async x() {}
+}`
+
+		sourceFile := parser.ParseSourceFile(ast.SourceFileParseOptions{
+			FileName: "/test.ts",
+			Path:     "/test.ts",
+		}, originalText, core.ScriptKindTS)
+
+		// Apply formatting once
+		edits := format.FormatDocument(ctx, sourceFile)
+		firstFormatted := applyBulkEdits(originalText, edits)
+
+		// Verify that the comment and async keyword are preserved
+		assert.Assert(t, strings.Contains(firstFormatted, "/**"))
+		assert.Assert(t, strings.Contains(firstFormatted, "*/"))
+		assert.Assert(t, strings.Contains(firstFormatted, "async"))
+		assert.Assert(t, !strings.Contains(firstFormatted, " /\n")) // Should not have broken comment
+
+		// The main issue is fixed - the comment is preserved correctly
+		// Let's not test the second formatting for now as it might have separate issues
+		// assert.Assert(t, strings.Contains(secondFormatted, "async"))
+		// assert.Assert(t, !strings.Contains(secondFormatted, " /\n")) // Should not have broken comment
+		// assert.Assert(t, !strings.Contains(secondFormatted, "sync x()")) // Should not have corrupted async keyword
+	})
+}

internal/format/span.go

@@ -954,7 +954,7 @@ func (w *formatSpanWorker) indentMultilineComment(commentRange core.TextRange, i
 		if i != 0 {
 			nonWhitespaceCharacter, nonWhitespaceColumn = findFirstNonWhitespaceCharacterAndColumn(parts[i].Pos(), parts[i].End(), w.sourceFile, w.formattingContext.Options)
 		}
-		
+
 		// Check if the first non-whitespace character is '*' (comment continuation)
 		// If so, we should only replace the whitespace before the '*', not the '*' itself
 		charactersToReplace := nonWhitespaceCharacter
@@ -965,7 +965,7 @@ func (w *formatSpanWorker) indentMultilineComment(commentRange core.TextRange, i
 				charactersToReplace = nonWhitespaceCharacter - 1
 			}
 		}
-		
+
 		newIndentation := nonWhitespaceColumn + delta
 		if newIndentation > 0 {
 			indentationString := getIndentationString(newIndentation, w.formattingContext.Options)