Skip to content

Add parameterized surface test for all 6 RN components#98

Open
EmilioBejasa wants to merge 1 commit intomainfrom
all-components-surface-test-v2
Open

Add parameterized surface test for all 6 RN components#98
EmilioBejasa wants to merge 1 commit intomainfrom
all-components-surface-test-v2

Conversation

@EmilioBejasa
Copy link
Copy Markdown
Collaborator

Summary

  • Registers all 6 RN components with AppRegistry in index.js so they can be started as surfaces in tests
  • Adds ComponentSurfaceTest, a @Parameterized test that runs constructViewTest once per component, asserting the surface view is non-null after start()

Test plan

  • CI passes all 6 parameterized test cases (MyFeature, SwitchFeature, TimerFeature, CoinFlipFeature, Button, Header)

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@EmilioBejasa EmilioBejasa requested a review from tdrhq March 31, 2026 21:56
Copy link
Copy Markdown
Contributor

@tdrhq tdrhq left a comment

Choose a reason for hiding this comment

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

Before we do this change, go back to focussing on just taking a screenshot in IsolatedTest. Once that screenshot test is working in isolation then we can come back to this change.

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