Skip to content

Conversation

PepperLola
Copy link
Member

Task

Symptom

Tests weren't passing.

Solution

  • Needed to convert ScoreTracker to a default import/export (not sure why it didn't work otherwise)
  • Needed to add new argument to touchmove event handler assertions
  • Needed to change mock of World to the correct getOwnRobots function

Verification

Tests pass


Before merging, ensure the following criteria are met:

  • All acceptance criteria outlined in the ticket are met.
  • Necessary test cases have been added and updated.
  • A feature toggle or safe disable path has been added (if applicable).
  • User-facing polish:
    • Ask: "Is this ready-looking?"
  • Cross-linking between Jira and GitHub:
    • PR links to the relevant Jira issue.
    • Jira ticket has a comment referencing this PR.

@PepperLola PepperLola merged commit a7f43e4 into dev Aug 23, 2025
17 checks passed
@PepperLola PepperLola deleted the jwrigh/test-fixes branch August 23, 2025 00:00
@PepperLola PepperLola mentioned this pull request Aug 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants