Address review comments

Author: ahoppen

Sources/SwiftSyntaxMacrosTestSupport/Assertions.swift

@@ -70,7 +70,9 @@ public func assertMacroExpansion(
     fixedSource: expectedFixedSource,
     testModuleName: testModuleName,
     testFileName: testFileName,
-    indentationWidth: indentationWidth
+    indentationWidth: indentationWidth,
+    file: file,
+    line: line
   )
 }
 

Sources/SwiftSyntaxMacrosTestSupportFrameworkAgnostic/Assertions.swift

@@ -73,7 +73,8 @@ public struct TestFailureLocation {
   }
 }
 
-/// Defines the details of a test failure, consisting of a message and the location at which the l
+/// Defines the details of a test failure, consisting of a message and the location at which the test failure should be
+/// shown.
 public struct TestFailureSpec {
   public let message: String
   public let location: TestFailureLocation
@@ -502,8 +503,8 @@ public func assertMacroExpansion(
   }
 
   assertStringsEqualWithDiff(
-    expandedSourceFile.description.droppingLast(while: \.isNewline),
-    expectedExpandedSource.droppingLast(while: \.isNewline),
+    expandedSourceFile.description.drop(while: \.isNewline).droppingLast(while: \.isNewline),
+    expectedExpandedSource.drop(while: \.isNewline).droppingLast(while: \.isNewline),
     "Macro expansion did not produce the expected expanded source",
     additionalInfo: """
       Actual expanded source:

Sources/_SwiftSyntaxTestSupportFrameworkAgnostic/AssertEqualWithDiff.swift

@@ -31,7 +31,8 @@ public struct TestFailureLocation {
   }
 }
 
-/// Defines the details of a test failure, consisting of a message and the location at which the l
+/// Defines the details of a test failure, consisting of a message and the location at which the test failure should be
+/// shown.
 public struct TestFailureSpec {
   public let message: String
   public let location: TestFailureLocation

Sources/_SwiftSyntaxTestSupportFrameworkAgnostic/String+TrimmingTrailingWhitespace.swift

@@ -11,7 +11,6 @@
 //===----------------------------------------------------------------------===//
 
 extension String {
-  // This implementation is really slow; to use it outside a test it should be optimized.
   public func trimmingTrailingWhitespace() -> String {
     return
       self

Tests/SwiftParserTest/translated/RecoveryTests.swift

@@ -2454,7 +2454,7 @@ final class RecoveryTests: ParserTestCase {
         DiagnosticSpec(message: "unexpected code '!=baz' in parameter clause"),
       ],
       fixedSource: """
-         func foo1(bar: <#type#>!=baz) {}
+        func foo1(bar: <#type#>!=baz) {}
         """
     )
   }