Skip to content

Commit 0125b32

Browse files
author
Michel Davit
committed
Avoid implicit name conflict
1 parent 45bcb1c commit 0125b32

File tree

6 files changed

+20
-16
lines changed

6 files changed

+20
-16
lines changed

scalacheck/src/main/scala-2/magnolify/scalacheck/ScalacheckMacros.scala

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,13 @@ import org.scalacheck.{Arbitrary, Cogen}
2020

2121
import scala.reflect.macros.*
2222
object ScalaCheckMacros {
23-
def genArbitraryMacro[T: c.WeakTypeTag](c: whitebox.Context): c.Tree = {
23+
def autoDerivationArbitrary[T: c.WeakTypeTag](c: whitebox.Context): c.Tree = {
2424
import c.universe._
2525
val wtt = weakTypeTag[T]
2626
q"""_root_.magnolify.scalacheck.ArbitraryDerivation.gen[$wtt]"""
2727
}
2828

29-
def genCogenMacro[T: c.WeakTypeTag](c: whitebox.Context): c.Tree = {
29+
def autoDerivationCogen[T: c.WeakTypeTag](c: whitebox.Context): c.Tree = {
3030
import c.universe._
3131
val wtt = weakTypeTag[T]
3232
q"""_root_.magnolify.scalacheck.CogenDerivation.gen[$wtt]"""
@@ -35,6 +35,7 @@ object ScalaCheckMacros {
3535
}
3636

3737
trait AutoDerivations {
38-
implicit def genArbitrary[T]: Arbitrary[T] = macro ScalaCheckMacros.genArbitraryMacro[T]
39-
implicit def genCogen[T]: Cogen[T] = macro ScalaCheckMacros.genCogenMacro[T]
38+
implicit def autoDerivationArbitrary[T]: Arbitrary[T] =
39+
macro ScalaCheckMacros.autoDerivationArbitrary[T]
40+
implicit def autoDerivationCogen[T]: Cogen[T] = macro ScalaCheckMacros.autoDerivationCogen[T]
4041
}

scalacheck/src/main/scala-3/magnolify/scalacheck/ScalacheckMacros.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import org.scalacheck.{Arbitrary, Cogen}
2121
import scala.deriving.Mirror
2222

2323
trait AutoDerivations:
24-
inline implicit def genArbitrary[T](using Mirror.Of[T]): Arbitrary[T] =
24+
inline implicit def autoDerivationArbitrary[T](using Mirror.Of[T]): Arbitrary[T] =
2525
ArbitraryDerivation.derivedMirror[T]
26-
inline implicit def genCogen[T](using Mirror.Of[T]): Cogen[T] =
26+
inline implicit def autoDerivationCogen[T](using Mirror.Of[T]): Cogen[T] =
2727
CogenDerivation.derivedMirror[T]

scalacheck/src/main/scala/magnolify/scalacheck/semiauto/package.scala

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,10 @@ package object semiauto {
2525
@deprecated("Use Gogen.gen[T] instead", "0.7.0")
2626
val CogenDerivation = magnolify.scalacheck.CogenDerivation
2727

28-
implicit def genArbitrary(a: Arbitrary.type): magnolify.scalacheck.ArbitraryDerivation.type =
28+
implicit def semiautoDerivationArbitrary(
29+
a: Arbitrary.type
30+
): magnolify.scalacheck.ArbitraryDerivation.type =
2931
magnolify.scalacheck.ArbitraryDerivation
30-
implicit def genCogen(c: Cogen.type): magnolify.scalacheck.CogenDerivation.type =
32+
implicit def semiautoDerivationCogen(c: Cogen.type): magnolify.scalacheck.CogenDerivation.type =
3133
magnolify.scalacheck.CogenDerivation
3234
}

scalacheck/src/test/scala/magnolify/scalacheck/ArbitraryDerivationSuite.scala

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ import scala.reflect._
3030

3131
class ArbitraryDerivationSuite extends MagnolifySuite {
3232
import TestArbitrary.arbSeed
33-
import magnolify.scalacheck.auto.genArbitrary
33+
import magnolify.scalacheck.auto.autoDerivationArbitrary
3434

3535
private def test[T: ClassTag](implicit t: Arbitrary[T]): Unit = {
3636
// TODO val g = ensureSerializable(t).arbitrary
@@ -73,8 +73,9 @@ class ArbitraryDerivationSuite extends MagnolifySuite {
7373

7474
// magnolia scala3 limitation:
7575
// For a recursive structures it is required to assign the derived value to an implicit variable
76-
implicit val arbNode: Arbitrary[Node] = genArbitrary
77-
implicit val arbGNode: Arbitrary[GNode[Int]] = genArbitrary
76+
import magnolify.scalacheck.semiauto.semiautoDerivationArbitrary
77+
implicit val arbNode: Arbitrary[Node] = Arbitrary.gen
78+
implicit val arbGNode: Arbitrary[GNode[Int]] = Arbitrary.gen
7879

7980
test[Node]
8081
test[GNode[Int]]

scalacheck/src/test/scala/magnolify/scalacheck/CogenDerivationSuite.scala

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ import scala.reflect.*
2727

2828
class CogenDerivationSuite extends MagnolifySuite {
2929
import TestArbitrary.arbSeed
30-
import magnolify.scalacheck.auto.genCogen
30+
import magnolify.scalacheck.auto.autoDerivationCogen
3131

3232
private def test[T: ClassTag](implicit arb: Arbitrary[T], t: Cogen[T]): Unit = {
3333
// TODO val co = ensureSerializable(t)
@@ -60,8 +60,9 @@ class CogenDerivationSuite extends MagnolifySuite {
6060

6161
// magnolia scala3 limitation:
6262
// For a recursive structures it is required to assign the derived value to an implicit variable
63-
implicit val cogenNode: Cogen[Node] = genCogen
64-
implicit val cogenGNode: Cogen[GNode[Int]] = genCogen
63+
import magnolify.scalacheck.semiauto.semiautoDerivationCogen
64+
implicit val cogenNode: Cogen[Node] = Cogen.gen
65+
implicit val cogenGNode: Cogen[GNode[Int]] = Cogen.gen
6566

6667
test[Node]
6768
test[GNode[Int]]

scalacheck/src/test/scala/magnolify/scalacheck/FunctionDerivationSuite.scala

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@ import org.scalacheck._
2424
import scala.reflect._
2525

2626
class FunctionDerivationSuite extends MagnolifySuite {
27-
import magnolify.scalacheck.auto.genCogen
28-
import magnolify.scalacheck.auto.genArbitrary
27+
import magnolify.scalacheck.auto._
2928

3029
private def test[A: ClassTag, B: ClassTag](implicit
3130
t: Arbitrary[A => B],

0 commit comments

Comments
 (0)