search

architect / tap-arc

Node.js spec-like TAP reporter
https://www.npmjs.com/package/tap-arc
Apache License 2.0
30 stars 3 forks source link

add stable object diffing #21

Closed mixmix closed 2 years ago

mixmix commented 2 years ago

Addresses #19

Relevant tests are passing, but suite is failing (existing failure because of stale snapshots, see #20 )

The problem with native JSON.stringify is it is not deterministic (as in although Object keys are "unordered" actually they are....) I've mutated a test and watched it introduce a failure: image

Then added a deterministic stringify to fix it: image

mixmix commented 2 years ago

sweet, this is going to make tap-arc my regular reporter!

have you got it listed in the tape repo/wiki? i only stumbled across this module because i found it referenced in an issue while trying to upgrade tap-spec

mixmix commented 2 years ago

lmk when you publish a new version 🍍 💥 🌼