-
Notifications
You must be signed in to change notification settings - Fork 18
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
Integrate with mocha and write tests #9
Comments
How about Jasmine? |
Go ahead, I don't think I have a strong preference for mocha. Something is better than nothing. |
Do you want unit tests of actual JS functions, or e2e tests of actually running the code on some chess games? |
You can start with unit tests and later add integration tests with actual chess engines. |
I am writing JS unit tests, and running them using karma. Karma requires a recent Node.js version. From their docs:
Are you OK with bumping |
I decided to just use
|
Here is the initial PR #26 . |
Thanks 👍. Merged. |
No description provided.
The text was updated successfully, but these errors were encountered: