skyweb07 / Snap.swift

Snapshot testing in a snap 🎨
MIT License
422 stars 19 forks source link

Integrate kaleidoscope #27

Closed PhilippeRiegert closed 6 years ago

PhilippeRiegert commented 6 years ago

Hi, thanks so much for this library!

I recently migrate from FBSnapshotTestCase and I was wondering if it would be possible to integrate Kaleidoscope? It was very useful in FBSnapshotTestCase when it fail to compare easily with Kaleidoscope with the ksdiff command line directly in the console.

Thanks !

skyweb07 commented 6 years ago

Hey @PhilippeRiegert, thanks for the suggestions, it's a good idea to integrate with that tool but you could also take advantage of the Snap built-in tool using the attachments API from Xcode that let you see the same as Kaleidoscope

method_1

It will generate a report with the failed image/ the diffed one and the original in case of failure on the test, it's pretty handy for this situations. I'll also take a look at Kaleidos to see if it's easy to integrate.

Thanks and happy new year 🎉

PhilippeRiegert commented 6 years ago

Thanks ! 🎉