Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Too big zio-test output when assertion failed on case class instances #8579

Closed
evis opened this issue Dec 20, 2023 · 10 comments
Closed

Too big zio-test output when assertion failed on case class instances #8579

evis opened this issue Dec 20, 2023 · 10 comments

Comments

@evis
Copy link
Contributor

evis commented Dec 20, 2023

When test failed, it's hard to understand why. Example: https://gist.github.com/evis/cd51be073b26c6a9f9ff45723d988e26

In this example, it's quite difficult to grasp the reason. Ideally, I'd like to see something like this:

receiptContent.payments(0).amount: expected 100, actual 10
receiptContent.rows(0).price: expected 100, actual 10

Or integrate something like https://diffx-scala.readthedocs.io/en/latest/index.html

@987Nabil
Copy link
Contributor

987Nabil commented Jan 1, 2024

If you use intellij, the intellij zio plugin should now (or at least soon) show a diff link that will open the intellij diff view.

@jdegoes
Copy link
Member

jdegoes commented Jan 8, 2024

/bounty $500

For an implementation that properly "diffs" structured data, and reports only the actual differences, in as flattened a way as possible, rather than printing out the pretty-printed case class values.

Copy link

algora-pbc bot commented Jan 8, 2024

💎 $500 bounty created by ZIO
🙋 If you start working on this, comment /attempt #8579 to notify everyone
👉 To claim this bounty, submit a pull request that includes the text /claim #8579 somewhere in its body
📝 Before proceeding, please make sure you can receive payouts in your country
💵 Payment arrives in your account 2-5 days after the bounty is rewarded
💯 You keep 100% of the bounty award
🙏 Thank you for contributing to zio/zio!

Attempt Started (GMT+0) Solution
🟢 @urbit-pilled Jan 8, 2024, 7:04:23 PM #8614

@atulkrs
Copy link

atulkrs commented Jan 8, 2024

can u assign this to me?

@urbit-pilled
Copy link
Contributor

urbit-pilled commented Jan 8, 2024

/attempt #8579

Options

Copy link

algora-pbc bot commented Jan 11, 2024

💡 @urbit-pilled submitted a pull request that claims the bounty. You can visit your bounty board to reward.

Copy link

algora-pbc bot commented Jan 16, 2024

@urbit-pilled: Your claim has been rewarded! 👉 Complete your Algora onboarding to collect the bounty.

Copy link

algora-pbc bot commented Jan 16, 2024

🎉🎈 @urbit-pilled has been awarded $500! 🎈🎊

@joroKr21
Copy link
Contributor

If you use intellij, the intellij zio plugin should now (or at least soon) show a diff link that will open the intellij diff view.

I think I saw that recently, although we still use ZIO 1. I guess it will go away with the change?

@adamgfraser
Copy link
Contributor

@joroKr21 That would be up to the maintainers of ZIO IntelliJ.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants