feat: 3fold, improve codestyle

- trigger draw by 3fold repetition
- delegate to Move.finalizeAfter all the work except the pieceMap update
  which remain in Variant.boardAfter
- improve Situation.piecesAfterAction codestyle
- Replay.replayMove now works
- remove notion of promotion
- add more unit tests

Author: vincentfrochot

src/main/scala/Game.scala

@@ -1237,7 +1237,7 @@ object Game {
         partialCaptures: Boolean = false
     ): Validated[String, (Game, Move)] = (orig, dest) match {
       case (Pos.Abalone(orig), Pos.Abalone(dest)) =>
-        g.apply(orig, dest, None, metrics)
+        g.apply(orig, dest)
           .toEither
           .map(t => (Abalone(t._1), Move.Abalone(t._2)))
           .toValidated

src/main/scala/Situation.scala

@@ -1287,7 +1287,7 @@ object Situation {
         partialCaptures: Boolean = false
     ): Validated[String, Move] = (from, to) match {
       case (Pos.Abalone(from), Pos.Abalone(to)) =>
-        s.move(from, to, promotion.map(_.toAbalone)).toEither.map(m => Move.Abalone(m)).toValidated
+        s.move(from, to).toEither.map(m => Move.Abalone(m)).toValidated
       case _                                    => sys.error("Not passed Abalone objects")
     }
 

src/main/scala/abalone/Board.scala

@@ -35,6 +35,8 @@ case class Board(
 
   def materialImbalance: Int = variant.materialImbalance(this)
 
+  def autoDraw: Boolean = history.threefoldRepetition && variant.repetitionEnabled
+
   override def toString = s"$variant Position after ${history.recentTurnUciString}"
 
   lazy val actors: Map[Pos, Actor] = pieces.map {

src/main/scala/abalone/Game.scala

@@ -17,10 +17,9 @@ case class Game(
   def apply(
       orig: Pos,
       dest: Pos,
-      promotion: Option[PromotableRole] = None,
       metrics: MoveMetrics = MoveMetrics()
   ): Validated[String, (Game, Move)] =
-    situation.move(orig, dest, promotion).map(_ withMetrics metrics) map { move =>
+    situation.move(orig, dest).map(_ withMetrics metrics) map { move =>
       apply(move) -> move
     }
 
@@ -38,7 +37,7 @@ case class Game(
   }
 
   def apply(uci: Uci.Move): Validated[String, (Game, Move)] =
-    apply(uci.orig, uci.dest, uci.promotion)
+    apply(uci.orig, uci.dest)
 
   private def applyClock(metrics: MoveMetrics, gameActive: Boolean, switchClock: Boolean) =
     clock.map { c =>