Repository navigation
Add Unit Tests for handling key events in processing:core #965
Description
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.
Activity
SableRaf commentedon Mar 8, 2025
Thanks for opening this @Rishab87.
Which existing issues are you referring to?
Rishab87 commentedon Mar 8, 2025
I'm referring to issues like #779
Rishab87 commentedon Mar 8, 2025
I've also created a PR for this issue, can you please review it?
Rishab87 commentedon Mar 19, 2025
Hi @SableRaf, just a follow up, since this is done, so should I start adding other unit tests too?
SableRaf commentedon Mar 19, 2025
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 💙
Rishab87 commentedon Mar 20, 2025
Yes I'll open a seperate issue and a PR which can close it.