Fixed input_file on windows (#386)

Author: mitsuhiko

.github/workflows/tests.yml

@@ -23,7 +23,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: dtolnay/rust-toolchain@master
         with:
-          toolchain: 1.56.1
+          toolchain: 1.60.0
       - name: Test
         run: make cargotest
 

CHANGELOG.md

@@ -4,6 +4,8 @@ All notable changes to insta and cargo-insta are documented here.
 
 ## 1.30.0
 
+- Resolved a bug on Windows that caused `input_file` not to be written into the
+  snapshots.  (#386)
 - Snapshots are accepted when running with `--accept` even if a test outside
   insta fails.  (#358)
 - Mark settings drop guard as `#[must_use]`.

src/runtime.rs

@@ -310,16 +310,9 @@ impl<'a> SnapshotAssertionContext<'a> {
 
     /// Given a path returns the local path within the workspace.
     pub fn localize_path(&self, p: &Path) -> Option<PathBuf> {
-        self.cargo_workspace
-            .join(p)
-            .canonicalize()
-            .ok()
-            .and_then(|s| {
-                self.cargo_workspace
-                    .canonicalize()
-                    .ok()
-                    .and_then(|cw| s.strip_prefix(cw).ok().map(|x| x.to_path_buf()))
-            })
+        let workspace = self.cargo_workspace.canonicalize().ok()?;
+        let p = self.cargo_workspace.join(p).canonicalize().ok()?;
+        p.strip_prefix(&workspace).ok().map(|x| x.to_path_buf())
     }
 
     /// Creates the new snapshot from input values.