Skip to content

docs: clarify declarative fixture testing guidance#7657

Merged
janechu merged 3 commits into
mainfrom
users/janechu/document-declarative-fixtures-7638
Jul 16, 2026
Merged

docs: clarify declarative fixture testing guidance#7657
janechu merged 3 commits into
mainfrom
users/janechu/document-declarative-fixtures-7638

Conversation

@janechu

@janechu janechu commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

📖 Description

Clarifies agent-facing testing guidance for declarative template work:

  • Explains that declarative syntax, template generation, hydration behavior, fixture category behavior, and other user-authored template behavior should be covered with browser-backed declarative fixtures.
  • Points agents to the local fixture README files so they can choose the right fixture category and follow category-specific examples.
  • Reinforces in the fixture writing guide that parser-only tests should supplement fixture coverage rather than replace it.

👩‍💻 Reviewer Notes

This follows review feedback on #7638, where declarative syntax coverage should use the existing browser fixture suite instead of only source-level parser tests.

📑 Test Plan

  • git diff --check
  • npm run checkchange

✅ Checklist

General

  • I have included a change request file using $ npm run change
  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Agents

  • I have linked to an existing issue in this project that this change addresses
  • I have read the skills
  • I have read the DESIGN.md file(s) in packages relevent to my changes
  • I have updated the DESIGN.md file(s) in packages relevent to my changes

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@janechu janechu changed the title docs: clarify declarative fixture testing guidance docs: add declarative fixture testing guidance Jul 13, 2026
janechu and others added 2 commits July 13, 2026 12:36
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@janechu
janechu marked this pull request as ready for review July 13, 2026 19:39
@janechu janechu changed the title docs: add declarative fixture testing guidance docs: clarify declarative fixture testing guidance Jul 13, 2026
@janechu
janechu merged commit 8681f2b into main Jul 16, 2026
18 checks passed
@janechu
janechu deleted the users/janechu/document-declarative-fixtures-7638 branch July 16, 2026 19:35
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