Maximilian Roos
Maximilian Roos
For discussion, re #492. There are tests which confirm that a passing test removes any associated pending snapshot. And a test that checks `--unreferenced=delete` picks up pending snapshots; so running...
This is an example of what I've been suggesting at https://github.com/mitsuhiko/insta/pull/506 (+ some linked issues): - ~Stacked on https://github.com/mitsuhiko/insta/pull/528 so that would need to merge first~ - Enforces the correct...
Not a big deal, but I think we can just defer
### What happened? This panics on not finding files: ```rust #[test] fn test_glob_path() { insta::glob!("../**/*.rs", |path| { let input = std::fs::read_to_string(path).unwrap(); insta::assert_snapshot!(input); }) } ``` ``` failures: ---- test_glob_path stdout...
This is a bit refine-y, but: In various parts of the code & docs, we use a bunch of different terms for "snapshot": - Snapshot - Value - Contents For...
Would that make sense? We'd benefit from shared infra, possibly easier for users to discover `insta_cmd`?
One feature that could be useful: Currently `r` rejects the current snapshot. `R` could reject _all_ pending snapshots. Same for Accept & Skip.
Currently when a snapshots needs updating, `cargo insta test` doesn't show a diff, only: ``` info: 1 snapshot to review use `cargo insta review` to review snapshots [Command exited with...
this isn't a bug report — instead it's a pattern I've been working with `claude` for a few weeks, to the extent I thought it would be worth sharing. I...