Handle new line characters as final characters (#40)

cargo-insta/src/cargo.rs

@@ -227,7 +227,7 @@ pub fn find_snapshots<'a>(
         .filter_map(move |e| {
             let fname = e.file_name().to_string_lossy();
             if fname.ends_with(".new")
-                && extensions.contains(&fname.rsplit('.').skip(1).next().unwrap_or(""))
+                && extensions.contains(&fname.rsplit('.').nth(1).unwrap_or(""))
                 && e.path()
                     .strip_prefix(&root)
                     .unwrap()

src/runtime.rs

@@ -419,16 +419,18 @@ fn get_inline_snapshot_value(frozen_value: &str) -> String {
             }
         }
 
-        if buf.ends_with('\n') {
-            buf.truncate(buf.len() - 1);
-        }
-
-        buf
+        buf.trim_end().to_string()
     } else {
-        frozen_value.to_string()
+        frozen_value.trim_end().to_string()
     }
 }
 
+#[test]
+fn test_inline_snapshot_value_newline() {
+    // https://github.com/mitsuhiko/insta/issues/39
+    assert_eq!(get_inline_snapshot_value("\n"), "");
+}
+
 #[allow(clippy::too_many_arguments)]
 pub fn assert_snapshot(
     refval: ReferenceValue<'_>,
@@ -484,8 +486,10 @@ pub fn assert_snapshot(
     };
 
     // if the snapshot matches we're done.
-    if old.as_ref().map_or(false, |x| x.contents() == new_snapshot) {
-        return Ok(());
+    if let Some(ref x) = old {
+        if x.contents().trim_end() == new_snapshot.trim_end() {
+            return Ok(());
+        }
     }
 
     let new = Snapshot::from_components(

tests/test_inline.rs

@@ -27,6 +27,12 @@ fn test_unnamed_single_line() {
     assert_snapshot_matches!("Testing-2");
 }
 
+#[test]
+fn test_newline() {
+    // https://github.com/mitsuhiko/insta/issues/39
+    assert_snapshot_matches!("\n", @"");
+}
+
 #[test]
 fn test_ron_inline() {
     #[derive(Serialize)]