.

Author: SimonCropp

src/StrictJsonTests/SerializationTests.ImmutableArray_Empty.verified.json

@@ -1,3 +1,3 @@
 {
   "Array": null
-}
+}

src/StrictJsonTests/SerializationTests.ImmutableArray_Uninitialized.verified.json

@@ -1,3 +1,3 @@
 {
   "Array": null
-}
+}

src/Verify.ExceptionParsing.Tests/ExceptionMessageFormatSamples.cs

@@ -14,7 +14,7 @@ public Task AllCategories()
         };
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", Dir("MyTests.Test2.received.txt"), Dir("MyTests.Test2.verified.txt")), null, new("received text"), new("verified text"))
+            new(new("txt", Dir("MyTests.Test2.received.txt"), Dir("MyTests.Test2.verified.txt")), null, new("received text"), "verified text")
         };
         var delete = new List<string>
         {
@@ -33,7 +33,7 @@ public Task NotEqualWithMessage()
     {
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", Dir("MyTests.Test1.received.txt"), Dir("MyTests.Test1.verified.txt")), "The comparer reported a difference", new("received text"), new("verified text"))
+            new(new("txt", Dir("MyTests.Test1.received.txt"), Dir("MyTests.Test1.verified.txt")), "The comparer reported a difference", new("received text"), "verified text")
         };
 
         return BuildVerify([], notEquals, [], []);

src/Verify.ExceptionParsing.Tests/ExceptionParsingTests.cs

@@ -31,7 +31,7 @@ public Task WithMessage()
     {
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", receivedTxt, verifiedTxt), "TheMessage", new("receivedText"), new("verifiedText")),
+            new(new("txt", receivedTxt, verifiedTxt), "TheMessage", new("receivedText"), "verifiedText"),
             new(new("bin", receivedBin, verifiedBin), "TheMessage", null, null)
         };
 
@@ -132,7 +132,7 @@ public Task SingleNotEqual()
     {
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", receivedTxt, verifiedTxt), null, new("receivedText"), new("verifiedText"))
+            new(new("txt", receivedTxt, verifiedTxt), null, new("receivedText"), "verifiedText")
         };
 
         return ParseVerify([], notEquals, [], []);
@@ -153,7 +153,7 @@ public Task MultipleItem()
         };
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", receivedTxt, verifiedTxt), null, new("receivedText"), new("verifiedText")),
+            new(new("txt", receivedTxt, verifiedTxt), null, new("receivedText"), "verifiedText"),
             new(new("bin", receivedBin, verifiedBin), null, null, null)
         };
         var delete = new List<string>
@@ -178,7 +178,7 @@ public Task SingleItem()
         };
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", receivedTxt, verifiedTxt), null, new("receivedText"), new("verifiedText"))
+            new(new("txt", receivedTxt, verifiedTxt), null, new("receivedText"), "verifiedText")
         };
         var delete = new List<string>
         {
@@ -242,4 +242,4 @@ static Task ParseVerify(
                 result
             });
     }
-}
+}

src/Verify.ExceptionParsing.Tests/VerifyExceptionMessageBuilderTests.cs

@@ -42,7 +42,7 @@ public Task SingleNotEqual_Text() =>
             @new: [],
             notEquals:
             [
-                new(new("txt", receivedTxt, verifiedTxt), null, new("received content"), new("verified content"))
+                new(new("txt", receivedTxt, verifiedTxt), null, new("received content"), "verified content")
             ],
             delete: [],
             equal: []);
@@ -53,7 +53,7 @@ public Task SingleNotEqual_WithMessage() =>
             @new: [],
             notEquals:
             [
-                new(new("txt", receivedTxt, verifiedTxt), "Comparer reported difference", new("received content"), new("verified content"))
+                new(new("txt", receivedTxt, verifiedTxt), "Comparer reported difference", new("received content"), "verified content")
             ],
             delete: [],
             equal: []);
@@ -108,7 +108,7 @@ public Task AllCategories()
         };
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", receivedTxt, verifiedTxt), null, new("received"), new("verified"))
+            new(new("txt", receivedTxt, verifiedTxt), null, new("received"), "verified")
         };
         var delete = new List<string>
         {
@@ -139,8 +139,8 @@ public Task MultipleNotEqual_MixedMessageAndNoMessage()
     {
         var notEquals = new List<NotEqualResult>
         {
-            new(new("txt", receivedTxt, verifiedTxt), null, new("received text"), new("verified text")),
-            new(new("txt", receivedTxt, verifiedTxt), "Custom comparison message", new("received2"), new("verified2"))
+            new(new("txt", receivedTxt, verifiedTxt), null, new("received text"), "verified text"),
+            new(new("txt", receivedTxt, verifiedTxt), "Custom comparison message", new("received2"), "verified2")
         };
 
         return BuildVerify([], notEquals, [], []);

src/Verify.MSTest.SourceGenerator.Tests/NoMatchTests.NoAttributes.verified.txt

@@ -1 +1 @@
-{}
+{}

src/Verify.Tests/NewLineTests.cs

@@ -78,14 +78,14 @@ public async Task TrailingNewlinesRaw()
         var exception = await Assert.ThrowsAnyAsync<Exception>(() => Verify("a\n", settings));
         Assert.Contains("carriage return", exception.ToString());
 
-        // Trailing \n tolerance still applies for \n-only verified files
+        // Trailing newlines are now compared exactly, with no tolerance
         await File.WriteAllTextAsync(file, "a\n");
         await Verify("a\n", settings);
-        await Verify("a", settings);
+        await Assert.ThrowsAsync<VerifyException>(() => Verify("a", settings));
 
         await File.WriteAllTextAsync(file, "a\n\n");
         await Verify("a\n\n", settings);
-        await Verify("a\n", settings);
+        await Assert.ThrowsAsync<VerifyException>(() => Verify("a\n", settings));
         File.Delete(file);
     }
 

src/Verify/Compare/Comparer.cs

@@ -9,13 +9,12 @@ public static async Task<EqualityResult> Text(FilePair filePair, StringBuilder r
             return new(Equality.New, null, received, null);
         }
 
-        var verifiedText = await File.ReadAllTextAsync(filePair.VerifiedPath);
-        if (verifiedText.Contains('\r'))
+        var verified = await File.ReadAllTextAsync(filePair.VerifiedPath);
+        if (verified.Contains('\r'))
         {
             throw new($@"Verified file must use \n line endings, but it contains a \r (carriage return). Path: {filePair.VerifiedPath}. See https://github.com/verifytests/verify#text-file-settings");
         }
 
-        var verified = new StringBuilder(verifiedText);
         var result = await CompareStrings(filePair.Extension, received, verified, settings, bypassComparer);
         if (result.IsEqual)
         {
@@ -26,35 +25,16 @@ public static async Task<EqualityResult> Text(FilePair filePair, StringBuilder r
         return new(Equality.NotEqual, result.Message, received, verified);
     }
 
-    static Task<CompareResult> CompareStrings(string extension, StringBuilder received, StringBuilder verified, VerifySettings settings, bool bypassComparer)
+    static Task<CompareResult> CompareStrings(string extension, StringBuilder received, string verified, VerifySettings settings, bool bypassComparer)
     {
-        if (verified.Length > 0 &&
-            verified.Length - 1 == received.Length &&
-            verified.LastChar() == '\n')
-        {
-            verified.Length -= 1;
-        }
-
-        // StringBuilder is broken on older .net https://github.com/dotnet/runtime/issues/27684
-#if NET6_0_OR_GREATER
-        var isEqual = verified.Equals(received);
-        if (!isEqual &&
-            !bypassComparer &&
-            settings.TryFindStringComparer(extension, out var compare))
-        {
-            return compare(received.ToString(), verified.ToString(), settings.Context);
-        }
-#else
-        var receivedString = received.ToString();
-        var verifiedString = verified.ToString();
-        var isEqual = receivedString.Equals(verifiedString);
+        // StringBuilder.Equals(ReadOnlySpan<char>) is native on net6+ and provided by Polyfill on net framework
+        var isEqual = received.Equals(verified.AsSpan());
         if (!isEqual &&
             !bypassComparer &&
             settings.TryFindStringComparer(extension, out var compare))
         {
-            return compare(receivedString, verifiedString, settings.Context);
+            return compare(received.ToString(), verified, settings.Context);
         }
-#endif
 
         return Task.FromResult(new CompareResult(isEqual));
     }

src/Verify/Verifier/EqualityResult.cs

@@ -1,7 +1,7 @@
-readonly struct EqualityResult(Equality equality, string? message, StringBuilder? receivedText, StringBuilder? verifiedText)
+readonly struct EqualityResult(Equality equality, string? message, StringBuilder? receivedText, string? verifiedText)
 {
     public Equality Equality { get; } = equality;
     public string? Message { get; } = message;
     public StringBuilder? ReceivedText { get; } = receivedText;
-    public StringBuilder? VerifiedText { get; } = verifiedText;
-}
+    public string? VerifiedText { get; } = verifiedText;
+}

src/Verify/Verifier/NotEqualResult.cs

@@ -2,10 +2,10 @@
     FilePair file,
     string? message,
     StringBuilder? receivedText,
-    StringBuilder? verifiedText)
+    string? verifiedText)
 {
     public FilePair File { get; } = file;
     public string? Message { get; } = message;
     public StringBuilder? ReceivedText { get; } = receivedText;
-    public StringBuilder? VerifiedText { get; } = verifiedText;
-}
+    public string? VerifiedText { get; } = verifiedText;
+}