difflicious icon indicating copy to clipboard operation
difflicious copied to clipboard

Feedback after migrating from diffx

Open ghostbuster91 opened this issue 2 years ago • 10 comments

The purpose of this issue is to gather feedback from migrating projects to difflicious.

https://github.com/ghostbuster91/pgn-parser/pull/2 - .assertNoDiff has to be called on the differ instance. For more complex types it is cumbersome. It would be better to write it in a way where the type can be inferred from the passed arguments.

ghostbuster91 avatar Jul 10 '23 20:07 ghostbuster91

Yes me too. I just gave a try to the library but gave up when I realized what I have to do something like Diff[Seq[Player]].assert... . I think something like Diff.assertNoDiff(a1,a2) that infers the type would be a lot simpler.

kostaskougios avatar Apr 11 '24 16:04 kostaskougios

I also recently migrated from diffx and I am using this assertion method variation from the munit support:

https://github.com/2m/rallyeye/blob/391227a93eb4b608416b6745595280d2c9ac04aa/modules/backend/src/test/scala/difflicious.scala#L24-L26

2m avatar Jun 30 '24 11:06 2m