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

toRightAssociated test failure in CI #3568

Closed
travisbrown opened this issue Aug 14, 2020 · 2 comments · Fixed by #3569
Closed

toRightAssociated test failure in CI #3568

travisbrown opened this issue Aug 14, 2020 · 2 comments · Fixed by #3569

Comments

@travisbrown
Copy link
Contributor

Just noticed this test failure in this job for #3567, probably related to #3527:

==> X cats.tests.AndThenSuite.toRightAssociated works  0.202s munit.FailException: Failing seed: fSK8i_ADMdWh0P7wdLaXzD1SVThALHwqzii_jRMBtHE=
7051You can reproduce this failure by adding the following override to your suite:
7052
7053  override val scalaCheckInitialSeed = "H6sIq1EUUvxUiPa11mJ4PoKuyBKd3swtk-oTfbaaH6D="
7054
7055Exception raised on property evaluation.
7056> ARG_0: AndThen$201277246
7057> Exception: java.lang.StackOverflowError: null
7058    at munit.ScalaCheckSuite.propToTry(ScalaCheckSuite.scala:89)
7059Caused by: java.lang.StackOverflowError
7060    at cats.data.AndThen$Concat.equals(AndThen.scala:163)
7061    at cats.data.AndThen$Concat.equals(AndThen.scala:163)
7062    at cats.data.AndThen$Concat.equals(AndThen.scala:163)
7063    at cats.data.AndThen$Concat.equals(AndThen.scala:163)

I don't personally think this should block the next release candidate, but we should look into it. @johnynek, any idea?

@johnynek
Copy link
Contributor

I'll try to debug today and send a PR.

@travisbrown
Copy link
Contributor Author

@johnynek Thanks! For what it's worth I spent five minutes trying to reproduce the error locally by setting the seed and the test didn't fail, so it might just be a matter of lower -Xss settings or whatever on Travis CI?

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

Successfully merging a pull request may close this issue.

2 participants