Skip to content

Completed lessons and puzzles test files#111

Merged
AidenRangel12 merged 2 commits intomainfrom
aiden-rangel
Jan 22, 2026
Merged

Completed lessons and puzzles test files#111
AidenRangel12 merged 2 commits intomainfrom
aiden-rangel

Conversation

@AidenRangel12
Copy link
Collaborator

Completed Lessons test files, specifically Lessons.test.tsx, and Lessons-overlay.test.tsx, and completed Puzzles test file, Puzzles.test.tsx.

@F-Hejazi F-Hejazi self-requested a review January 9, 2026 17:58
Copy link
Collaborator

@F-Hejazi F-Hejazi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Aiden,
Thanks for working on these tests! Your work establishes a good foundation, but there are some gaps in coverage that we need to address. Some must-fix test items include drag-and-drop interactions, pawn promotion, lesson/puzzle completion flows, and socket connection/disconnection handling. Once those are covered, we'd also want to test for edge cases like network failures and invalid FEN strings.
Feel free to continue on this branch if you'd like, otherwise go ahead and merge and I'll document the remaining work in follow-up tickets.

@yalrubaye
Copy link
Collaborator

Meets expectation.

@AidenRangel12 AidenRangel12 merged commit 04c4c00 into main Jan 22, 2026
1 check passed
@F-Hejazi F-Hejazi deleted the aiden-rangel branch January 27, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants