rspec-snapshot
rspec-snapshot copied to clipboard
New Configuration Options, Use OJ for serialization / deserialization
Pr is very messy right now, started it in an internal hackathon - mostly leaving here for comment / so I stop ignoring it. Before something like this merges, it should
- be split into multiple prs
- be cleaned of dumb editor changes (
"to'is my vim config, but is not actually a good change) - be reviewed for design / features
Features added:
- configure more about the way the snapshot is saved (configurable path, configurable extension)
- read an env var to decide whether to save new snapshots or fail on a mismatch
- load a snapshot from a name, (nominally, for use in other tests as a mock object, but I guess you could use it for other things too!)
Hello @rrcobb , can you check the serializer branch? I've tried add a snapshot_serializer config and introduce awesome_print as a default serializer.
Nice, I'll take a look at that branch and see whether I can just close this one. It could be there are small pieces of this that are worth pulling out, like making oj an optional serializer, taking a snapshot to a particular file, etc
I like this PR, please continue your work.
Some of this has been integrated into version 2.0.0. Up to you if you want to rebase and work out the conflicts for the remainder
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
No data about Duplication