NowChess/NowChessSystems
4
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

fix(rules): Code quality

Browse Source 
linter corrections
This commit is contained in:
LQ63
2026-04-21 18:40:49 +02:00
parent c09a618dba
commit 9f33e2e3db
3 changed files with 59 additions and 46 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/rule/src/main/scala/de/nowchess/rules/dto/DtoMapper.scala
+13 -13
View File
@@ -29,7 +29,7 @@ object DtoMapper:
case "castleKingside" => Right(MoveType.CastleKingside)
case "castleQueenside" => Right(MoveType.CastleQueenside)
case "enPassant" => Right(MoveType.EnPassant)
case "promotion" =>
case "promotion" =>
dto.promotionPiece.toRight("Missing promotion piece").flatMap(toPromotionPiece).map(MoveType.Promotion(_))
case other => Left(s"Unknown move type: $other")
@@ -51,13 +51,13 @@ object DtoMapper:
moves <- sequenceList(dto.moves.map(toMove))
initialBoard <- toBoard(dto.initialBoard)
yield GameContext(
board = board,
turn = turn,
castlingRights = toCastlingRights(dto.castlingRights),
board = board,
turn = turn,
castlingRights = toCastlingRights(dto.castlingRights),
enPassantSquare = epSquare,
halfMoveClock = dto.halfMoveClock,
moves = moves,
initialBoard = initialBoard,
halfMoveClock = dto.halfMoveClock,
moves = moves,
initialBoard = initialBoard,
)
def fromMove(move: Move): MoveDto =
@@ -71,13 +71,13 @@ object DtoMapper:
def fromGameContext(ctx: GameContext): GameContextDto =
GameContextDto(
board = fromBoard(ctx.board),
turn = ctx.turn.label,
castlingRights = fromCastlingRights(ctx.castlingRights),
board = fromBoard(ctx.board),
turn = ctx.turn.label,
castlingRights = fromCastlingRights(ctx.castlingRights),
enPassantSquare = ctx.enPassantSquare.map(_.toString),
halfMoveClock = ctx.halfMoveClock,
moves = ctx.moves.map(fromMove),
initialBoard = fromBoard(ctx.initialBoard),
halfMoveClock = ctx.halfMoveClock,
moves = ctx.moves.map(fromMove),
initialBoard = fromBoard(ctx.initialBoard),
)
private def toPromotionPiece(s: String): Either[String, PromotionPiece] = s match
modules/rule/src/test/scala/de/nowchess/rules/resource/RuleSetResourceTest.scala
+20 -14
View File
@@ -247,11 +247,13 @@ class RuleSetResourceTest:
// ── position builders ─────────────────────────────────────────────
private def buildCheckContext(): GameContext =
val board = Board(Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.E, Rank.R3) -> Piece(Color.Black, PieceType.Rook),
))
val board = Board(
Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.E, Rank.R3) -> Piece(Color.Black, PieceType.Rook),
),
)
GameContext(board, Color.White, CastlingRights.None, None, 0, List.empty, initialBoard = board)
private def buildFoolsMate(): GameContext =
@@ -263,18 +265,22 @@ class RuleSetResourceTest:
}
private def buildStalemateContext(): GameContext =
val board = Board(Map(
Square(File.H, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.F, Rank.R7) -> Piece(Color.White, PieceType.Queen),
Square(File.G, Rank.R6) -> Piece(Color.White, PieceType.King),
))
val board = Board(
Map(
Square(File.H, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.F, Rank.R7) -> Piece(Color.White, PieceType.Queen),
Square(File.G, Rank.R6) -> Piece(Color.White, PieceType.King),
),
)
GameContext(board, Color.Black, CastlingRights.None, None, 0, List.empty, initialBoard = board)
private def buildKingsOnlyContext(): GameContext =
val board = Board(Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
))
val board = Board(
Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
),
)
GameContext(board, Color.White, CastlingRights.None, None, 0, List.empty, initialBoard = board)
private def buildThreefoldContext(): GameContext =
modules/rule/src/test/scala/de/nowchess/rules/resource/RuleSetResourceUnitTest.scala
+26 -19
View File
@@ -16,16 +16,18 @@ class RuleSetResourceUnitTest extends AnyFunSuite with Matchers:
private def ctx(g: GameContext) = ContextRequest(DtoMapper.fromGameContext(g))
private def ctxSq(g: GameContext, sq: String) = ContextSquareRequest(DtoMapper.fromGameContext(g), sq)
private def ctxMv(g: GameContext, m: Move) = ContextMoveRequest(DtoMapper.fromGameContext(g), DtoMapper.fromMove(m))
private def ctxMv(g: GameContext, m: Move) = ContextMoveRequest(DtoMapper.fromGameContext(g), DtoMapper.fromMove(m))
// ── position builders ─────────────────────────────────────────────
private def checkContext(): GameContext =
val board = Board(Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.E, Rank.R3) -> Piece(Color.Black, PieceType.Rook),
))
val board = Board(
Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.E, Rank.R3) -> Piece(Color.Black, PieceType.Rook),
),
)
GameContext(board, Color.White, CastlingRights.None, None, 0, List.empty, initialBoard = board)
private def foolsMate(): GameContext =
@@ -37,18 +39,22 @@ class RuleSetResourceUnitTest extends AnyFunSuite with Matchers:
}
private def stalemateContext(): GameContext =
val board = Board(Map(
Square(File.H, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.F, Rank.R7) -> Piece(Color.White, PieceType.Queen),
Square(File.G, Rank.R6) -> Piece(Color.White, PieceType.King),
))
val board = Board(
Map(
Square(File.H, Rank.R8) -> Piece(Color.Black, PieceType.King),
Square(File.F, Rank.R7) -> Piece(Color.White, PieceType.Queen),
Square(File.G, Rank.R6) -> Piece(Color.White, PieceType.King),
),
)
GameContext(board, Color.Black, CastlingRights.None, None, 0, List.empty, initialBoard = board)
private def kingsOnlyContext(): GameContext =
val board = Board(Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
))
val board = Board(
Map(
Square(File.E, Rank.R1) -> Piece(Color.White, PieceType.King),
Square(File.E, Rank.R8) -> Piece(Color.Black, PieceType.King),
),
)
GameContext(board, Color.White, CastlingRights.None, None, 0, List.empty, initialBoard = board)
private def threefoldContext(): GameContext =
@@ -100,9 +106,9 @@ class RuleSetResourceUnitTest extends AnyFunSuite with Matchers:
test("isCheck throws BadRequestException for invalid context"):
an[BadRequestException] should be thrownBy
resource.isCheck(ctx(GameContext.initial).copy(context =
DtoMapper.fromGameContext(GameContext.initial).copy(turn = "Red"),
))
resource.isCheck(
ctx(GameContext.initial).copy(context = DtoMapper.fromGameContext(GameContext.initial).copy(turn = "Red")),
)
// ── isCheckmate ───────────────────────────────────────────────────
@@ -154,7 +160,8 @@ class RuleSetResourceUnitTest extends AnyFunSuite with Matchers:
resource.applyMove(ctxMv(GameContext.initial, move)).turn shouldBe "Black"
test("applyMove throws BadRequestException for invalid move"):
val badMove = DtoMapper.fromMove(Move(Square(File.E, Rank.R2), Square(File.E, Rank.R4)))
val badMove = DtoMapper
.fromMove(Move(Square(File.E, Rank.R2), Square(File.E, Rank.R4)))
.copy(moveType = "unknown")
an[BadRequestException] should be thrownBy
resource.applyMove(ContextMoveRequest(DtoMapper.fromGameContext(GameContext.initial), badMove))