web icon indicating copy to clipboard operation
web copied to clipboard

Make snapshot directory configurable

Open LarsDenBakker opened this issue 3 years ago • 4 comments

We currently store snapshots next to the test file itself. This doesn't work out when you are using a compiler and output to a separate directory. We should make the snapshot directory configurable.

LarsDenBakker avatar Jun 14 '21 07:06 LarsDenBakker

This would be very helpful, currently I have to set up exceptions in .gitignore to check only the snapshots folder, which is a bit cumbersome.

alfonsogarciacaro avatar Sep 25 '21 01:09 alfonsogarciacaro

Are there any updates on this issue? We are moving from karma to web-test-runner and this is the still an inconvenience.

JakobVogelsang avatar Dec 05 '21 21:12 JakobVogelsang

I opened a PR about this about a week ago: https://github.com/modernweb-dev/web/pull/1866

I'm not sure if there's some protocol I need to follow in order to get someone to give it a look-see, or if everyone's just busy with other things for now.

lovenm avatar Feb 07 '22 13:02 lovenm

@LarsDenBakker any thoughts on the PR? The dom snapshots currently feel a bit awkward on Typescript due to this. I was not entirely sure if the way I implemented it in the PR was an optimal way, but I can also work more on this if there are some better ideas.

lovenm avatar Apr 10 '22 14:04 lovenm