Renamed getLegalMoves to getMoves because, as per rules of Chess, not all the moves that method returns is legal.

This commit compiles but the tests will not yet run.
Tests are running again after rename but, of course, still failing.
Correcting mistake in kingside castling test.

The test is removing the King as well! No go!
Adding some javadocs to clarify usage of our classes.