Skip to content

Hang while running test suite for scalatestplus-scalacheck (possible regression) #909

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

Closed
griggt opened this issue Jul 28, 2022 · 3 comments

Comments

@griggt
Copy link

griggt commented Jul 28, 2022

Running the test suite for scalatestplus-scalacheck appears to hang indefinitely (while shrinking?) when built against the latest commit on the scalacheck main branch (7749c38).

The same test suite passes within about 5 seconds if built against the previous merge commit (dc93d79), so it seems related to #838.

This issue was noticed while updating projects in the Scala 3 community build (scala/scala3#15728 (comment)) however it is also reproducible on Scala 2.13.

Reproduction Steps

  1. publishLocal scalacheck @ 7749c38
  2. clone scalatest/scalatestplus-scalacheck@32e7dc6
  3. update build.sbt of scalatestplus-scalacheck to use the locally published scalacheck:
-    "org.scalacheck" %%% "scalacheck" % "1.16.0",
+    "org.scalacheck" %%% "scalacheck" % "1.17-7749c38-SNAPSHOT",
  1. run sbt scalatestPlusScalaCheckJVM/test
@armanbilge
Copy link
Member

Thanks for the report. To ease reproduction the mentioned snapshot is actually published.
https://s01.oss.sonatype.org/content/repositories/snapshots/org/scalacheck/scalacheck_2.13/1.17-7749c38-SNAPSHOT/

@rossabaker
Copy link
Member

I've been meaning to do a release, but we'll hold for this fix. Thanks for catching it!

AurelienRichez added a commit to AurelienRichez/scalacheck that referenced this issue Jul 28, 2022
AurelienRichez added a commit to AurelienRichez/scalacheck that referenced this issue Jul 28, 2022
rossabaker added a commit that referenced this issue Aug 2, 2022
fix wrong match in shrinker (fixes #909)
@rossabaker
Copy link
Member

I haven't tested with scalatestplus-scalacheck, but it should be fixed in 1.17-8689ae7-SNAPSHOT.

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

No branches or pull requests

3 participants