Alternate castling was forced into normal castling

[okmsbun]

I did something like this to solve this problem. If there was a castling move containing the values of the map here, I forced it to change it with its key. Before solving this problem, this needs to be solved first.

On the game analysis screen, I reduced eight castling alternatives to four. In all moves, there will be a movement just like the movement in the keys. Is this logic correct? If true, is this function the place of application?

[veloce]

I think the logic is correct, also Node looks like a good place to put this.

Since the alternative castling notation is always tricky it would be good to also add a new test to the Node tests to ensure all types of castling moves produces the same branch id.

[okmsbun]

I also added the tests. It now works like on the lichess.org website.

In the 3 pictures below, I played "e1g1", but it behaves as if I played "e1h1". So everything seems fine. It behaves the same as the Lichess.org website.

before	after	lichess.org

[veloce]

Perfect! Thank you!