insta icon indicating copy to clipboard operation
insta copied to clipboard

Shorten snapshot file names?

Open max-sixty opened this issue 1 year ago • 1 comments

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 avatar Jun 04 '23 04:06 max-sixty