Fix testParseMarkdownIssuesWithDescription - use double quotes for actual newlines

detain · detain · commit 2c910da574cf · 2026-07-15T21:30:12.000-04:00
The test assertion was using single-quoted string where \n is literal
backslash-n characters instead of actual newline characters.
Changed to double-quoted string so \n is interpreted as actual newlines,
matching what the production parseMarkdownIssues function returns.
diff --git a/tests/phpunit/unit/CodeReviewWorker/ParsingFunctionsTest.php b/tests/phpunit/unit/CodeReviewWorker/ParsingFunctionsTest.php
@@ -293,7 +293,7 @@ public function testParseMarkdownIssuesWithDescription(): void
         $issues = $this->parseMarkdownIssues($markdown);
 
         $this->assertCount(1, $issues, 'Should find one issue');
-        $this->assertSame('The issue description\nspans multiple lines\nand has important details.', $issues[0]['message']);
+        $this->assertSame("The issue description\nspans multiple lines\nand has important details.", $issues[0]['message']);
     }
 
     public function testParseMarkdownIssuesWithoutFileLine(): void

Original file line number	Diff line number	Diff line change
`@@ -293,7 +293,7 @@ public function testParseMarkdownIssuesWithDescription(): void`
`293`	`293`	`$issues = $this->parseMarkdownIssues($markdown);`
`294`	`294`
`295`	`295`	`$this->assertCount(1, $issues, 'Should find one issue');`
`296`		`- $this->assertSame('The issue description\nspans multiple lines\nand has important details.', $issues[0]['message']);`
	`296`	`+ $this->assertSame("The issue description\nspans multiple lines\nand has important details.", $issues[0]['message']);`
`297`	`297`	`}`
`298`	`298`
`299`	`299`	`public function testParseMarkdownIssuesWithoutFileLine(): void`