Skip to content

tidy: delete @Ignore'd ScreenshotTest#108

Open
EmilioBejasa wants to merge 1 commit intomainfrom
tidy-remove-dead-screenshot-test
Open

tidy: delete @Ignore'd ScreenshotTest#108
EmilioBejasa wants to merge 1 commit intomainfrom
tidy-remove-dead-screenshot-test

Conversation

@EmilioBejasa
Copy link
Copy Markdown
Collaborator

Summary

Deletes ScreenshotTest.kt (~510 lines). The entire class was annotated @org.junit.Ignore so none of its tests ever ran. It modelled component states using native Android views (LinearLayout, TextView, etc.) which predates the React Native surface approach.

Pure structural change — no tests removed from the active suite.

🤖 Generated with Claude Code

The entire class was annotated @org.junit.Ignore so none of its tests
ever ran. It modelled component states using native Android views (LinearLayout,
TextView, etc.) which predates the React Native surface approach and is
no longer the direction we're taking.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@EmilioBejasa EmilioBejasa requested a review from tdrhq April 8, 2026 19:18
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.

again, distraction from #103. But it's simple enough that it's okay to merge.

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