Ignore leading newlines on snapshot comparison (#444)

CHANGELOG.md

@@ -8,6 +8,8 @@ All notable changes to insta and cargo-insta are documented here.
 - Added new alternative `match .. { ... }` syntax to redactions for better
   `rustfmt` support.  (#428)
 - The `--package` parameter can be supplied multiple times now.  (#427)
+- Leading newlines in snapshots are now ignored to resolve issues with
+  inline snapshots that were never able to match.  (#444)
 
 ## 1.34.0
 

src/snapshot.rs

@@ -576,7 +576,13 @@ impl From<SnapshotContents> for String {
 
 impl PartialEq for SnapshotContents {
     fn eq(&self, other: &Self) -> bool {
-        self.0.trim_end() == other.0.trim_end()
+        self.0
+            .trim_start_matches(|x| x == '\r' || x == '\n')
+            .trim_end()
+            == other
+                .0
+                .trim_start_matches(|x| x == '\r' || x == '\n')
+                .trim_end()
     }
 }
 

tests/test_inline.rs

@@ -286,3 +286,15 @@ fn test_inline_test_in_loop() {
         assert_snapshot!(i.to_string(), @"0");
     }
 }
+
+#[test]
+fn test_inline_snapshot_whitespace() {
+    assert_snapshot!("\n\nfoo\n\n    bar\n\n", @r###"
+
+
+    foo
+
+        bar
+
+    "###);
+}