Skip to content

docs(testing): clarify extension integration homes - #134

Merged
racitores merged 1 commit into
mainfrom
docs/extension-integration-homes
Aug 25, 2026
Merged

docs(testing): clarify extension integration homes#134
racitores merged 1 commit into
mainfrom
docs/extension-integration-homes

Conversation

@racitores

Copy link
Copy Markdown
Contributor

Description

Clarify the existing MetaMask Extension integration boundaries without introducing a new testing layer or runner.

  • Route full UI + real Redux scenarios to test/integration/.
  • Route composed MetaMaskController and child-controller wiring to the existing Node tests under app/scripts/.
  • Keep isolated controller logic in unit tests and real browser/extension boundaries in E2E.
  • Document that the composed-background tests remain on the unit Jest command despite owning an integration boundary.

Type of Change

  • New skill
  • Skill improvement/update
  • Bug fix
  • Documentation update
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING.md guidelines
  • My skill follows the SKILL_TEMPLATE.md format
  • I have tested this skill with an AI agent
  • My skill does not contain any secrets, private keys, or sensitive data
  • I have added appropriate documentation
  • My changes don't break existing skills

Testing

  • yarn audit:skills (0 errors; pre-existing warnings only)
  • yarn test (65 tests passed)
  • git diff --check

Additional Context

This deliberately does not add a Mobile-style component-view or controller integration harness. It documents two existing homes within Extension's current three-layer model and prevents controller-composition coverage from defaulting to E2E.

Made with Cursor

Route UI/state and composed-background coverage through existing Extension test locations so agents avoid unnecessary E2E tests or new layers.

Co-authored-by: Cursor <cursoragent@cursor.com>
@racitores
racitores marked this pull request as ready for review August 24, 2026 17:03
@racitores
racitores enabled auto-merge (squash) August 25, 2026 08:00
@racitores
racitores merged commit 1193e1e into main Aug 25, 2026
29 checks passed
@racitores
racitores deleted the docs/extension-integration-homes branch August 25, 2026 08:15
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