typelevel / weaver-test

A test framework that runs everything in parallel.
https://typelevel.org/weaver-test/
Other
46 stars 8 forks source link

Inline Expecty and improve macros on long/wide outputs #19

Open keynmol opened 8 months ago

keynmol commented 8 months ago

Expecty macro libary has been historically maintained by @Baccata, me, and Eugene Yokota, the current maintainer of the fork.

The code from it has been inlined and improved in the Verify testing framework maintained by Eugene. Inlining the dependency makes release cycles obviously easier.

I believe it's time we design the assertion macros from scratch, as part of this repo, not separate dependency, and address multiple issues related to how Expecty renders assertion failures. See screenshot for an example of an unusable output.

Once we inline the dependency, this issue can be broken down into actionable, smaller issues.

image

zainab-ali commented 1 month ago

As part of #64 , expecty was replaced with a clue macro that resembles MUnit's clue. It uses a Show typeclass instance to render values.

It doesn't remove rendering issues -clue(intitialized) would still render a long output - but it moves the problem towards providing a better Show instance.