-
Notifications
You must be signed in to change notification settings - Fork 443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add tests for specific functions #920
Labels
good first issue
Good for newcomers
Comments
MarkZH
pushed a commit
that referenced
this issue
Jun 21, 2024
Add some tests for external moves. A lot more should be added before #920 can be closed. Also fixes a bug in engine_wrapper found while creating these tests.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now, the only tests are integration tests that perform a complete run of
lichess_bot_main()
to check that there are no errors communicating with an engine and with a mocked version of the lichess.org servers. Individual functions could use unit tests, such as:get_lichess_cloud_move()
Lichess
class methods to make sure they still return expected responses (bot scopes, profile data, bot opponent lists, etc.)The text was updated successfully, but these errors were encountered: