search

bahmutov / snap-shot

Jest-like snapshot feature for the rest of us, works magically by finding the right caller function
169 stars 3 forks source link

Fix update option usage in snapshot(data, update) #81

Closed JaakkoL closed 5 years ago

JaakkoL commented 6 years ago

Currently the update option is never used to override the default options, which makes it impossible to update a single test update. This fixes it.

bahmutov commented 6 years ago

Hmm, the tests seem to change values and try to save new ones. BTW I switched to using https://github.com/bahmutov/snap-shot-it because it is runtime and does not use static code analysis, so it works much better.

JaakkoL commented 6 years ago

Ah, so it seems. All the tests work on my local. Ran it with node 6 and 8 and no errors whatsoever. Cannot seem to figure out why Travis fails.

I'll give a look at snap-shot-it instead.