Inline snapshot overwriting for snapbox

[not-my-profile]

Snapbox currently only appears to support SNAPSHOTS=overwrite for assert_eq_path and assert_matches_path but not for inline snapshot assertions like assert_eq or assert_matches.

The popular cargo-insta tool does support the updating of inline snapshots. So I think the current lack of support should certainly be documented.

If supporting inline snapshots is out of the scope of snapbox, I think a reference to Insta in the documentation would be nice.

If it is within scope of snapbox, reliably parsing Rust code would require a dependency on syn, which however could be feature-gated and disabled by default. The inline::FilePatcher implementation of cargo-insta looks quite brief, which would suggest that supporting this feature wouldn't take much additional code here.

[epage]

expect-test is also pretty minimal.

The biggest problem is getting the path to the file. At the moment, only heuristics work or people have to opt-in to a hacky solution. rust-lang/cargo#3946 would unblock that which is what I've been waiting on.

[not-my-profile]

I see, thanks for elaborating :)

(link to the issue you mentioned: rust-lang/cargo#3946)

[epage]

Thanks; its too easy to overlook crossing repo boundaries when linking

[epage]

As mentioned in #225, the new API I expect for this would be something like assert_eq(data!("..."), ...) which would also allow assert_eq(Data::from_path("..."), ...), allowing the code to be unified between the snapshot being in-source vs external.

In fact, that API refactor can happen before the requisite cargo features are available.

[epage]

Oops, we had #135 for this. Closing in favor of that. #251 will close it.