Fix test to verify standalone \r is escaped, not left verbatim

Copilot · hishamco · web-flow · commit 9c494b08fb47 · 2026-04-26T03:55:34.000Z
Agent-Logs-Url: https://github.com/OrchardCoreContrib/OrchardCoreContrib.PoExtractor/sessions/2dd9ced8-94a7-48d5-9acc-840734cc3e30 Co-authored-by: hishamco <3237266+hishamco@users.noreply.github.com>
diff --git a/test/OrchardCoreContrib.PoExtractor.Tests/PoWriterTests.cs b/test/OrchardCoreContrib.PoExtractor.Tests/PoWriterTests.cs
@@ -49,7 +49,7 @@ public void WriteRecord_Escapes()
     }
 
     [Fact]
-    public void WriteRecord_ShouldNotEscape_CarriageReturn()
+    public void WriteRecord_ShouldEscape_CarriageReturn()
     {
         // Arrange
         var localizableString = new LocalizableString
@@ -65,7 +65,7 @@ public void WriteRecord_ShouldNotEscape_CarriageReturn()
 
         // Assert
         var result = ReadPoStream();
-        Assert.Equal("msgid \"Orchard\rCore\"", result[0]);
+        Assert.Equal("msgid \"Orchard\\rCore\"", result[0]);
         Assert.Equal("msgstr \"\"", result[1]);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public void WriteRecord_Escapes()`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`[Fact]`
`52`		`- public void WriteRecord_ShouldNotEscape_CarriageReturn()`
	`52`	`+ public void WriteRecord_ShouldEscape_CarriageReturn()`
`53`	`53`	`{`
`54`	`54`	`// Arrange`
`55`	`55`	`var localizableString = new LocalizableString`
`@@ -65,7 +65,7 @@ public void WriteRecord_ShouldNotEscape_CarriageReturn()`
`65`	`65`
`66`	`66`	`// Assert`
`67`	`67`	`var result = ReadPoStream();`
`68`		`- Assert.Equal("msgid \"Orchard\rCore\"", result[0]);`
	`68`	`+ Assert.Equal("msgid \"Orchard\\rCore\"", result[0]);`
`69`	`69`	`Assert.Equal("msgstr \"\"", result[1]);`
`70`	`70`	`}`
`71`	`71`