java-snapshot-testing icon indicating copy to clipboard operation
java-snapshot-testing copied to clipboard

When using scenarios, don't throw until test is finished.

Open vectro opened this issue 1 year ago • 1 comments

If a test has multiple scenarios, we currently throw an exception the first time we call toMatchSnapshot() with different data. But then we miss the opportunity to update the rest of the scenarios in the .snap.debug file.

It would be better to let the test finish, and then raise if necessary with all differences together.

vectro avatar Sep 17 '24 20:09 vectro

Yes - agree that this is the preferred behaviour. Would accept a PR implementing this.

jackmatt2 avatar Oct 19 '24 23:10 jackmatt2