test(Tasks): Add comprehensive test suite for Tasks component#356
test(Tasks): Add comprehensive test suite for Tasks component#356ShivaGupta-14 wants to merge 1 commit into
Conversation
|
Thank you for opening this PR! Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools. Please take a moment to:
More information on how to conduct a self review: This helps make the review process smoother and gives us a clearer understanding of your thought process. Once you've added your self-review, we'll continue from our side. Thank you! |
e2bd188 to
3b4e2e2
Compare
- Add tests for Keyboard Navigation (ArrowUp, ArrowDown, boundary stops)
- Add tests for Hotkey Shortcuts ('a', 'c', 'd', 'e', 'f', 'p', 'r', 's', 't')
- Add tests for complete/delete hotkeys when dialog is already open
- Add tests for Sync button click and error toast on sync failure
- Add tests for LocalStorage persistence (tasksPerPage, lastSyncTime)
- Add tests for sync time auto-update interval
- Add tests for error handling (add task, edit task failures)
- Add tests for mouse hover handlers (mouseEnter, mouseLeave)
- Updates: CCExtractor#300
3b4e2e2 to
c502366
Compare
|
@ShivaGupta-14 can not review this PR as there are too many changes, please open separate PRs for each sub component if possible (or a set of sub components). Closing for now. |
|
ok, I will open separate PRs |
Description
Updates: #300
Checklist
npx prettier --write .(for formatting)gofmt -w .(for Go backend)npm test(for JS/TS testing)Additional Notes
Screenshots