Bump scala chess 16.3.3 and Improve test with literal uci

app/src/main/scala/model.scala

@@ -37,9 +37,9 @@ object GameId:
   extension (id: GameId) def value: String = id
 
 object ChessCirceCodecs:
-  given Encoder[Fen.Epd] = encodeString.contramap(_.value)
-  given Decoder[Fen.Epd] = decodeString.map(Fen.Epd.apply)
-  given Encoder[Variant] = encodeString.contramap(_.name)
+  given Encoder[Fen.Full] = encodeString.contramap(_.value)
+  given Decoder[Fen.Full] = decodeString.map(Fen.Full.apply)
+  given Encoder[Variant]  = encodeString.contramap(_.name)
   given Decoder[Variant] =
     decodeString.emap: s =>
       Variant.byName(s).toRight(s"Invalid variant: $s")
@@ -59,7 +59,7 @@ object Fishnet:
   case class WorkResponse(
       work: Work,
       game_id: GameId,
-      position: Fen.Epd,
+      position: Fen.Full,
       moves: String,
       variant: Variant
   ) derives Encoder.AsObject
@@ -74,7 +74,7 @@ object Lila:
 
   case class Request(
       id: GameId,
-      initialFen: Fen.Epd,
+      initialFen: Fen.Full,
       variant: Variant,
       moves: String,
       level: Int,
@@ -100,8 +100,8 @@ object Lila:
           )
       case _ => None
 
-  def readFen(str: String): Option[Fen.Epd] =
-    Option.when(str.nonEmpty)(Fen.Epd(str))
+  def readFen(str: String): Option[Fen.Full] =
+    Option.when(str.nonEmpty)(Fen.Full(str))
 
   def readClock(s: String): Option[Clock] =
     s.split(" ", 3) match

app/src/test/scala/Arbitraries.scala

@@ -1,7 +1,7 @@
 package lila.fishnet
 
 import chess.CoreArbitraries.given
-import chess.format.Fen.Epd
+import chess.format.Fen.Full
 import org.scalacheck.Arbitrary.*
 import org.scalacheck.{ Arbitrary, Cogen, Gen }
 
@@ -34,7 +34,7 @@ object Arbitraries:
       level   <- Gen.choose(1, 8)
       clock   <- arbitrary[Option[Lila.Clock]]
       variant <- arbitrary[chess.variant.Variant]
-    yield Lila.Request(id, Epd.initial, variant, moves, level, clock)
+    yield Lila.Request(id, Full.initial, variant, moves, level, clock)
 
   given Arbitrary[Work.Task] = Arbitrary:
     for

app/src/test/scala/ExecutorTest.scala

@@ -3,6 +3,7 @@ package lila.fishnet
 import cats.effect.{ IO, Ref }
 import cats.syntax.all.*
 import chess.format.Uci
+import chess.macros.uci
 import org.typelevel.log4cats.Logger
 import org.typelevel.log4cats.noop.NoOpLogger
 import weaver.*
@@ -27,7 +28,7 @@ object ExecutorTest extends SimpleIOSuite:
   val key  = ClientKey("key")
   val key2 = ClientKey("key2")
 
-  val validMove = BestMove(Uci("e2e4").get)
+  val validMove = BestMove(uci"e2e4")
 
   test("acquire when there is no work should return none"):
     for
@@ -68,7 +69,7 @@ object ExecutorTest extends SimpleIOSuite:
       acquired <- executor.acquire(key)
       _        <- executor.move(acquired.get.id, key, validMove.some)
       response <- ref.get.map(_.head)
-    yield expect.same(response, Lila.Response(request.id, request.moves, Uci.Move("e2e4").get))
+    yield expect.same(response, Lila.Response(request.id, request.moves, uci"e2e4"))
 
   test("post move after timeout should not send move"):
     for
@@ -91,7 +92,7 @@ object ExecutorTest extends SimpleIOSuite:
       acquired <- executor.acquire(key)
       _        <- executor.move(acquired.get.id, key, validMove.some)
       response <- ref.get.map(_.head)
-    yield expect.same(response, Lila.Response(request.id, request.moves, Uci.Move("e2e4").get))
+    yield expect.same(response, Lila.Response(request.id, request.moves, uci"e2e4"))
 
   test("post null move should remove the task"):
     for

app/src/test/scala/IntegrationTest.scala

@@ -3,6 +3,7 @@ package lila.fishnet
 import cats.effect.{ IO, Ref, Resource }
 import cats.syntax.all.*
 import chess.format.Uci
+import chess.macros.uci
 import com.comcast.ip4s.*
 import com.dimafeng.testcontainers.GenericContainer
 import io.chrisdavenport.rediculous.RedisPubSub
@@ -54,7 +55,7 @@ object IntegrationTest extends IOSuite:
   test("let's play a game"): res =>
     val fishnet               = Fishnet("2.7.2", ClientKey("secret-key"))
     val fishnetAcquireRequest = Acquire(fishnet)
-    val bestMoves             = List("e7e6", "d7d5", "d8d6").traverse(Uci.apply).get
+    val bestMoves             = List(uci"e7e6", uci"d7d5", uci"d8d6")
     val postMoves             = bestMoves.map(m => PostMove(fishnet, Move(BestMove(m).some)))
 
     val gameId = "CPzkP0tq"

app/src/test/scala/http/FishnetRoutesTest.scala

@@ -66,7 +66,7 @@ object FishnetRoutesTest extends SimpleIOSuite:
     id = WorkId("workid"),
     request = Lila.Request(
       id = GameId("gameid"),
-      initialFen = chess.format.Fen.Epd("rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR"),
+      initialFen = chess.format.Fen.Full("rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR"),
       moves = "",
       variant = chess.variant.Standard,
       level = 1,

project/Dependencies.scala

@@ -10,7 +10,7 @@ object Dependencies {
     val ciris      = "3.6.0"
     val kamon      = "2.5.11"
     val kamonAgent = "1.0.18"
-    val chess      = "15.7.11"
+    val chess      = "16.3.3"
     val catsEffect = "3.5.5"
   }