nullprop/Chess-Challenge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix stalemate detection after generating capture moves in a position with no captures available

Browse source 
#418
This commit is contained in:
Sebastian Lague 2023-08-01 17:49:40 +02:00
parent bcd8cbb423
commit 1315551493
2 changed files with 21 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Chess-Challenge/src/API/Board.cs
View file

@ -174,8 +174,12 @@ namespace ChessChallenge.API
{
bool includeQuietMoves = !capturesOnly;
moveGen.GenerateMoves(ref moveList, board, includeQuietMoves);
hasCachedMoveCount = true;
cachedMoveCount = moveList.Length;
if (!capturesOnly)
{
hasCachedMoveCount = true;
cachedMoveCount = moveList.Length;
}
}

13
Chess-Challenge/src/Framework/Application/Helpers/Tester.cs
View file

@ -368,6 +368,19 @@ namespace ChessChallenge.Application
Assert(!boardAPI.IsDraw(), "Draw wrong");
boardAPI.MakeMove(new API.Move("f5f7", boardAPI));
Assert(boardAPI.IsDraw(), "Draw wrong");
// Test stalemate bug when generating captures
board.LoadPosition("8/8/3k4/8/8/3K4/4Q3/8 w - - 0 1");
boardAPI = new API.Board(board);
var captures = boardAPI.GetLegalMoves(capturesOnly: true);
Assert(captures.Length == 0, "Wrong capture count");
Assert(!boardAPI.IsInStalemate(), "Stalemate wrong");
// Test stalemate bug when generating captures (non alloc)
board.LoadPosition("8/8/3k4/8/8/3K4/4Q3/8 w - - 0 1");
boardAPI = new API.Board(board);
Span<API.Move> captureSpan = stackalloc API.Move[64];
boardAPI.GetLegalMovesNonAlloc(ref captureSpan, capturesOnly: true);
Assert(captureSpan.Length == 0, "Wrong capture count");
Assert(!boardAPI.IsInStalemate(), "Stalemate wrong");
// Insufficient material
board = new Chess.Board();