Skip to content

chore(test): document non-automatable Stack v5 e2e scenarios - #4528

Open
LKuchno wants to merge 2 commits into
mainfrom
@lkuchno/e2e-test-coverage-stack-v5
Open

chore(test): document non-automatable Stack v5 e2e scenarios#4528
LKuchno wants to merge 2 commits into
mainfrom
@lkuchno/e2e-test-coverage-stack-v5

Conversation

@LKuchno

@LKuchno LKuchno commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

This PR documents why two Stack v5 test scenarios cannot be reliably automated with Detox and updates their e2e coverage status from 'tbd' to 'incomplete'. Both scenarios involve visual assertions and interaction state observation beyond Detox's capabilities, so they remain manual tests.

Changes include updates to scenario-description.ts and scenario.md files, replacing placeholder "TBD" text with explicit explanations of what cannot be automated and why. The test-stack-subviews-ios scenario also receives formatting improvements for better readability.

Changes

  • test-stack-subviews-ios: Documented that visual verification against Dev Tools element-inspector overlay cannot be accessed by Detox, keeping all checks manual; reformatted scenario steps with improved spacing and added "Known Issues/Important Observations" subsection
  • test-stack-toolbar-menu-icon-android: Documented that drawable and tint color assertions per interaction state (normal/pressed/focused/disabled) are pixel-level visual checks Detox cannot express; pressed/focused states additionally require mid-gesture observation and hardware keyboard focus support

@LKuchno
LKuchno requested a review from kmichalikk August 20, 2026 09:02
@LKuchno
LKuchno force-pushed the @lkuchno/e2e-test-coverage-stack-v5 branch from e3e391d to cd64760 Compare August 20, 2026 13:37

@kmichalikk kmichalikk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Left one comment, otherwise good to go

details: 'Tests header config and subview customization.',
platforms: ['ios'],
e2eCoverage: 'tbd',
e2eCoverage: 'incomplete',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe we should use "n/a" instead of "incomplete" in these situations?

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