Skip to content

Conversation

@ericelliott
Copy link
Collaborator

@ericelliott ericelliott commented Jan 13, 2026

Added UI testing strategy guidelines and constraints.


Note

Enhances the TDD guide with a clear UI testing strategy and minor cleanup.

  • Adds UI testing strategy under Default Test Utils: pure tests for Redux actions/selectors, isolate side effects (see javascript/javascript-io-network-effects.mdc), use riteway/render for component markup, Playwright for real browser interactions, and avoid @testing-library/react.
  • Minor formatting/content tweak at file end (/welcome).

Written by Cursor Bugbot for commit 1c92986. This will update automatically on new commits. Configure here.

Added UI testing strategy guidelines and constraints.
Copilot AI review requested due to automatic review settings January 13, 2026 01:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request enhances the TDD documentation by adding comprehensive UI testing strategy guidelines. The changes provide clear direction on how to test different aspects of UI code using appropriate tools for each testing layer.

Changes:

  • Added UI testing strategy section with guidelines for testing Redux actions/selectors, side effects, component rendering, and browser interactions
  • Explicitly discourages using @testing-library/react in favor of the documented patterns
  • Minor formatting correction (line ending normalization)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings January 13, 2026 02:11
@ericelliott ericelliott merged commit ec8da75 into main Jan 13, 2026
5 of 6 checks passed
@ericelliott ericelliott deleted the ericelliott-patch-2 branch January 13, 2026 02:11
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.

2 participants