fix(plugin25): proper winConditions

Author: xeruf

plugin2025/src/main/kotlin/sc/plugin2025/GameState.kt

@@ -11,6 +11,7 @@ import sc.plugin2025.util.HuIConstants
 import sc.plugin2025.util.HuIWinReason
 import sc.shared.InvalidMoveException
 import sc.shared.WinCondition
+import sc.shared.WinReasonTie
 import kotlin.math.pow
 import kotlin.math.sqrt
 
@@ -70,7 +71,16 @@ data class GameState @JvmOverloads constructor(
         get() = players.any { it.inGoal } && turn.mod(2) == 0 || turn / 2 >= HuIConstants.ROUND_LIMIT
 
     override val winCondition: WinCondition?
-        get() = players.filter { it.inGoal }.maxByNoEqual { -it.carrots }?.team?.let { WinCondition(it, HuIWinReason.DIFFERING_CARROTS) }
+        get() {
+            val goalies = players.filter { it.inGoal }
+            return when(goalies.size) {
+                0 -> null
+                1 -> WinCondition(goalies.single().team, HuIWinReason.GOAL)
+                else -> goalies.maxByNoEqual { -it.carrots }?.team?.let {
+                    WinCondition(it, HuIWinReason.DIFFERING_CARROTS)
+                } ?: WinCondition(null, WinReasonTie)
+            }
+        }
 
     val Hare.inGoal
         get() = position == board.size - 1

plugin2025/src/test/kotlin/sc/plugin2025/GameStateTest.kt

@@ -8,6 +8,7 @@ import sc.helpers.shouldSerializeTo
 import sc.plugin2025.util.HuIWinReason
 import sc.shared.InvalidMoveException
 import sc.shared.WinCondition
+import sc.shared.WinReasonTie
 
 class GameStateTest: FunSpec({
     test("clone correctly") {
@@ -20,7 +21,7 @@ class GameStateTest: FunSpec({
     test("let lower carrots win on tie") {
         val state = GameState(Board(arrayOf(Field.GOAL)))
         state.isOver shouldBe true
-        state.winCondition shouldBe null
+        state.winCondition shouldBe WinCondition(null, WinReasonTie)
         state.players.first().carrots = 5
         state.winCondition shouldBe WinCondition(Team.ONE, HuIWinReason.DIFFERING_CARROTS)
         state.players.last().carrots = 4
@@ -70,7 +71,7 @@ class GameStateTest: FunSpec({
             state.performMoveDirectly(Advance(2))
             state.turn shouldBe 2
             state.isOver shouldBe true
-            state.winCondition shouldBe null
+            state.winCondition shouldBe WinCondition(null, WinReasonTie)
         }
         test("round limit") {
             state.turn = 59

sdk/src/main/framework/sc/shared/WinCondition.kt

@@ -20,5 +20,8 @@ data class WinCondition(
     override fun toString(): String =
             reason.getMessage((if(reason.isRegular) winner else winner?.opponent()).toString())
 
-    override fun equals(other: Any?): Boolean = other is WinCondition && other.winner == winner
+    override fun equals(other: Any?): Boolean =
+        other is WinCondition &&
+        other.winner == winner &&
+        other.reason == reason
 }