-
-
Notifications
You must be signed in to change notification settings - Fork 70
Add Unit Tests for handling key events in processing:core
#965
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
Comments
Thanks for opening this @Rishab87. Which existing issues are you referring to? |
I'm referring to issues like #779 |
I've also created a PR for this issue, can you please review it? |
Hi @SableRaf, just a follow up, since this is done, so should I start adding other unit tests too? |
Hi @Rishab87. Yes, go ahead! Please open a new PR since the first one already got approved for the 4.4.1 milestone. Thank you for your contributions 💙 |
Yes I'll open a seperate issue and a PR which can close it. |
I saw a few open issues related to key events not behaving as expected. It would make sense to add unit tests for handling key events to ensure these events work as expected. The tests should cover all possible cases, especially those mentioned in the existing issues.
The text was updated successfully, but these errors were encountered: