fix(plugin23): board xml test

Author: mosasini

plugin/src/main/kotlin/sc/plugin2023/Board.kt

@@ -7,15 +7,11 @@ import sc.framework.deepCopy
 import kotlin.random.Random
 import sc.plugin2023.util.PenguinConstants as Constants
 
-/**
- * Klasse welche eine Spielbrett darstellt. Bestehend aus einem
- * zweidimensionalen Array aus Feldern
- *
- * @author soed
- */
+/** Spielbrett aus einem zweidimensionalen Array aus Feldern. */
 @XStreamAlias(value = "board")
-class Board(override val gameField: MutableTwoDBoard<Field> = generateFields()):
-    RectangularBoard<Field>(gameField), IBoard {
+class Board(
+    gameField: MutableTwoDBoard<Field> = generateFields()
+): RectangularBoard<Field>(gameField), IBoard {
 
     constructor(board: Board): this(board.gameField.deepCopy())
 

plugin/src/main/kotlin/sc/plugin2023/util/XStreamClasses.kt

@@ -2,15 +2,14 @@ package sc.plugin2023.util
 
 import sc.api.plugins.Coordinates
 import sc.networking.XStreamProvider
-import sc.plugin2023.Board
-import sc.plugin2023.GameState
-import sc.plugin2023.Move
+import sc.plugin2023.*
 
 class XStreamClasses: XStreamProvider {
 
     override val classesToRegister =
         listOf(
             Board::class.java,
+            Field::class.java,
             GameState::class.java,
             Move::class.java,
             Coordinates::class.java,

plugin/src/test/kotlin/sc/plugin2023/BoardTest.kt

@@ -11,8 +11,10 @@ import io.kotest.matchers.string.*
 import sc.api.plugins.Coordinates
 import sc.api.plugins.Team
 import sc.helpers.shouldSerializeTo
-import sc.helpers.testXStream
+import sc.networking.XStreamProvider
 import sc.plugin2023.util.PenguinConstants
+import sc.plugin2023.util.XStreamClasses
+import sc.protocol.LobbyProtocol
 import sc.y
 
 class BoardTest: FunSpec({
@@ -60,20 +62,23 @@ class BoardTest: FunSpec({
             board.possibleMovesFrom(0 y 0) shouldHaveSize PenguinConstants.BOARD_SIZE - 1
         }
     }
-    xcontext("XML Serialization") {
-        test("empty Board") {
+    val xstream = XStreamProvider.basic()
+    LobbyProtocol.registerAdditionalMessages(xstream, XStreamClasses().classesToRegister)
+    context("XML Serialization") {
+        xtest("empty Board") {
+            // TODO shouldSerializeTo needs to be uncoupled from general testXStream
             Board(arrayOf()) shouldSerializeTo """
               <board/>
             """.trimIndent()
         }
         test("random Board length") {
-            testXStream.toXML(Board()) shouldHaveLineCount 82
+            xstream.toXML(Board()) shouldHaveLineCount 82
         }
         test("Board with content") {
             val fieldTwo = "<field>TWO</field>"
-            testXStream.fromXML(fieldTwo) shouldBe Field(penguin = Team.TWO)
-            testXStream.fromXML("<board><list>$fieldTwo</list>") shouldBe makeSimpleBoard(Field(penguin = Team.TWO))
-            testXStream.toXML(makeBoard(0 y 0 to 1)) shouldContainOnlyOnce fieldTwo
+            xstream.fromXML(fieldTwo) shouldBe Field(penguin = Team.TWO)
+            // testXStream.fromXML("<board><list>$fieldTwo</list>") shouldBe makeSimpleBoard(Field(penguin = Team.TWO))
+            xstream.toXML(makeBoard(0 y 0 to 1)) shouldContainOnlyOnce fieldTwo
         }
     }
 })