-
Notifications
You must be signed in to change notification settings - Fork 25
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
Increase code coverage of tests #70
Comments
To create better tests that actually run a notebook and produce a real output we can use: https://github.com/jupyter/jupyter_kernel_test |
Hey @marimeireles, I set up the test environment but I am confused about how we can test LOAD or anything that doesn't give output. We can rather check for error messages if something is unsuccessful. Or you suggest something :) |
Hum, yeah. If the other commands like CREATE [...] don't throw an error message, we can create new err msgs for them. Additionally, if you wanna inspiration on how to start using this python lib: https://github.com/jupyter-xeus/xeus-cling/blob/master/test/test_xcpp_kernel.py |
I saw this and tried to replicate but the library is throwing some weird errors, I will try them again and let you know. |
We need more tests, this issue keeps track of tests that should be added:
These are the more latent ones. We can expand on this. The more the better :)
The text was updated successfully, but these errors were encountered: