Shorten snapshot file names?

[max-sixty]

This might be too much of a breaking change, or I might be misunderstanding the structure, but: we current have snapshots at a path like:

web/book/tests/examples/snapshots/examples__book__introduction__0.snap

This is from an assert_snapshot! in web/book/tests/examples/book.rs with a snapshot name of introduction/0.

Given that the snapshots are already in the path examples, is there a reason to prefix the filename of the snapshot with examples__? Could the name of the snapshot instead just be:

web/book/tests/examples/snapshots/book__introduction__0.snap

[max-sixty]

One option is to write new snapshots with the shorter name while supporting older snapshots with longer names.

It would be technically possible to have conflicts, but extremely unlikely (in the example above, we'd need a module web/book/tests/examples/examples/book.rs containing a test with exactly the same name, and the new module would have to write with the new naming scheme, while the old module remains on the existing naming scheme).

Happy to give this a go if it would get merged...

[iTrooz]

There is already a configurable option prepend_module_to_snapshot that would allow for ~that. That said, this option is currently only configurable per-test and not project-wide

I'd love to submit a PR to add this setting into the insta config file

[max-sixty]

yes that would be welcome!

some more ambitious ideas in #647, but just adding a config for prepend_module_to_snapshot would be good progress