Merge pull request #1451 from travisbrown/update/scalacheck-1.13.3

Fix Arbitrary instances in Free tests for ScalaCheck 1.13.3
typelevel · Nov 5, 2016 · 2fd55c5 · 2fd55c5
2 parents bcc751f + 3cd666a
commit 2fd55c5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/build.sbt b/build.sbt
@@ -94,7 +94,7 @@ lazy val includeGeneratedSrc: Setting[_] = {
 
 lazy val catsSettings = buildSettings ++ commonSettings ++ publishSettings ++ scoverageSettings ++ javadocSettings
 
-lazy val scalaCheckVersion = "1.13.2"
+lazy val scalaCheckVersion = "1.13.3"
 lazy val scalaTestVersion = "3.0.0"
 lazy val disciplineVersion = "0.7.1"
 

diff --git a/free/src/test/scala/cats/free/FreeTTests.scala b/free/src/test/scala/cats/free/FreeTTests.scala
@@ -164,7 +164,7 @@ object FreeTTests extends FreeTTestsInstances {
       F.arbitrary.map(FreeT.liftF[F, G, A])
     )
 
-    val nextDepth = Gen.chooseNum(1, maxDepth - 1)
+    val nextDepth = Gen.chooseNum(1, math.max(1, maxDepth - 1))
 
     def withFlatMapped = for {
       fDepth <- nextDepth

diff --git a/free/src/test/scala/cats/free/FreeTests.scala b/free/src/test/scala/cats/free/FreeTests.scala
@@ -127,7 +127,7 @@ sealed trait FreeTestsInstances {
       A.arbitrary.map(Free.pure[F, A]),
       F.arbitrary.map(Free.liftF[F, A]))
 
-    val nextDepth = Gen.chooseNum(1, maxDepth - 1)
+    val nextDepth = Gen.chooseNum(1, math.max(1, maxDepth - 1))
 
     def withFlatMapped = for {
       fDepth <- nextDepth