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
Fix prettifier not being used for property check #2173
Fix prettifier not being used for property check #2173
Conversation
153c852
to
aab0ba6
Compare
@cheeseng I have re-opened the PR and also updated it so it targets |
@mdedetrich Both scalatest and scalatest+scalacheck PR fix looks good to me, but I am not sure if they are the cause/only cause for the problem you are trying to address, because both of them should affect scala-js/native build only. The problem you encounter happens in jvm build also right? |
This specific PR doesn't address my problem since as you pointed out it was JVM but theoretically speaking JS/Native should have the exact same problem due to having the same bug so I re-opened it. |
I see, I'll study a bit further and see if I can find the root cause of the problem. |
Sorry I accidentally closed this, I didn't realize I closed this until I see your message on gitter. |
So in PR #2155 I fixed the
Prettifier
so that it works on case classes however I noticed that its still not being applied even when usingimplicit val prettifier: Prettifier = Prettifier.truncateAt(SizeLimit(10))
, i.e.This keeps on going for ages, i.e. its not being truncated. This is when using scalatestplus-scalacheck with property based tests. After going through the code I think that I found the culprit is was in
decorateStringToValue
, one of them uses a prettifier but the other completely ignores. Not sure if this is an oversight or if I am looking in the wrong places.Would also like to add a test for this to verify this but not sure where is the appropriate place to put this.
References: #2174