From 74b7a174c2aad8213e317e97639c2f651a14d530 Mon Sep 17 00:00:00 2001 From: LQ63 Date: Tue, 21 Apr 2026 21:15:14 +0200 Subject: [PATCH] fix(rules): Serializers Added small changes --- .../scala/de/nowchess/rules/json/MoveTypeDeserializer.scala | 2 ++ .../scala/de/nowchess/rules/json/JsonSerializersTest.scala | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/modules/rule/src/main/scala/de/nowchess/rules/json/MoveTypeDeserializer.scala b/modules/rule/src/main/scala/de/nowchess/rules/json/MoveTypeDeserializer.scala index d0f0c8f..00cee39 100644 --- a/modules/rule/src/main/scala/de/nowchess/rules/json/MoveTypeDeserializer.scala +++ b/modules/rule/src/main/scala/de/nowchess/rules/json/MoveTypeDeserializer.scala @@ -6,6 +6,7 @@ import com.fasterxml.jackson.databind.{DeserializationContext, JsonDeserializer} import de.nowchess.api.move.{MoveType, PromotionPiece} class MoveTypeDeserializer extends JsonDeserializer[MoveType]: + // scalafix:off DisableSyntax.throw override def deserialize(p: JsonParser, ctx: DeserializationContext): MoveType = val node = p.getCodec.readTree[ObjectNode](p) node.get("type").asText() match @@ -15,3 +16,4 @@ class MoveTypeDeserializer extends JsonDeserializer[MoveType]: case "enPassant" => MoveType.EnPassant case "promotion" => MoveType.Promotion(PromotionPiece.valueOf(node.get("piece").asText())) case t => throw new JsonParseException(p, s"Unknown move type: $t") + // scalafix:on DisableSyntax.throw diff --git a/modules/rule/src/test/scala/de/nowchess/rules/json/JsonSerializersTest.scala b/modules/rule/src/test/scala/de/nowchess/rules/json/JsonSerializersTest.scala index a14769b..e832fcb 100644 --- a/modules/rule/src/test/scala/de/nowchess/rules/json/JsonSerializersTest.scala +++ b/modules/rule/src/test/scala/de/nowchess/rules/json/JsonSerializersTest.scala @@ -32,11 +32,13 @@ class JsonSerializersTest extends AnyFunSuite with Matchers: // ── SquareKeyDeserializer ───────────────────────────────────────── + // scalafix:off DisableSyntax.null test("SquareKeyDeserializer returns square for valid key"): new SquareKeyDeserializer().deserializeKey("e4", null) shouldBe e4 test("SquareKeyDeserializer returns null for invalid key"): new SquareKeyDeserializer().deserializeKey("z9", null) shouldBe null + // scalafix:on DisableSyntax.null // ── SquareSerializer/Deserializer ───────────────────────────────── @@ -46,8 +48,10 @@ class JsonSerializersTest extends AnyFunSuite with Matchers: test("SquareDeserializer reads valid square string"): mapper.readValue(""""e4"""", classOf[Square]) shouldBe e4 + // scalafix:off DisableSyntax.null test("SquareDeserializer returns null for invalid square string"): mapper.readValue(""""z9"""", classOf[Square]) shouldBe null + // scalafix:on DisableSyntax.null // ── MoveTypeSerializer ────────────────────────────────────────────