-
Notifications
You must be signed in to change notification settings - Fork 21
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
Testing Suite #2
Comments
How would we like tests to work? There is a test folder currently; do we want to have tests for all components? |
@49Indium Ideally we should test things that are within our control. For example how the events are parsed from the ical, rather than testing if a message is sent. |
I've added tests (that pass) for Haiku. Will add some for the other cogs soon. |
Looking at this further, it seems that we might need to relocate some of the logic of some commands to their own functions (not within the object) to be able to test them effectively without simulating discord (see here). |
linking this library for if/when we ever get the time. This looks to be the "best" (most up-to-date and still live) discord mocking library: https://github.com/CraftSpider/dpytest |
Off the back of #1, it would be good to have a testing suite in place. Seperate issue created for discussion.
The text was updated successfully, but these errors were encountered: