Skip to content

refactor(test): align tabs test structure with rules from rfc#4180

Open
LKuchno wants to merge 7 commits into
mainfrom
@lkuchno/test-structure-fix
Open

refactor(test): align tabs test structure with rules from rfc#4180
LKuchno wants to merge 7 commits into
mainfrom
@lkuchno/test-structure-fix

Conversation

@LKuchno

@LKuchno LKuchno commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR cleans up the tabs single-feature test suite so directory names, e2e
file names, and scenario-description metadata follow the project's test
organization conventions. It is a test-infrastructure-only change — no library
source or runtime behavior is touched.

Changes

  • Renamed the tabs-screen-orientation test directory to
    test-tabs-screen-orientation (both index.tsx and scenario-description.ts)
    to match the test-* naming convention, and updated its registration key
    accordingly.
  • Renamed the e2e spec test-tabs-bottom-accessory-ios.e2e.ts
    test-tabs-bottom-accessory-layout-ios.e2e.ts so the e2e file mirrors its
    test screen directory.
  • Reordered the scenario imports/registrations in the tabs index.ts for a
    consistent, logical order.
  • Added missing details metadata to the scenario-description.ts files for
    test-tabs-tab-bar-hidden and
    test-tabs-tab-bar-experimental-user-interface-style-ios.
  • Fixed the key in the test-tabs-more-navigation-controller-ios
    scenario-description to carry the missing -ios suffix.
  • Updated scenario.md metadata (e2e test section reference) for
    test-tabs-layout-appearances-ios to match the new e2e file naming.

New Tabs screen order:
Simulator Screenshot - iPhone 17 Pro - 2026-06-18 at 10 41 58

@LKuchno LKuchno changed the title test(tabs): align tabs test structure with naming conventions refactor(test): align tabs test structure with rules from rfc Jun 18, 2026
@LKuchno LKuchno added the area:tabs Issue related to bottom tabs label Jun 18, 2026
Comment on lines +5 to +6
key: 'test-tabs-screen-orientation',
platforms: ['ios'],

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it iOS specific? I don't think it is. Maybe it's yet unimplemented for Android, however the test is intended for both platforms, for Android it just fails.

@LKuchno LKuchno Jun 18, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, it's currently unimplemented on Android tracked here: #868.

I plan to add the 'android' tag to #1208 once it's actually working. There are also separate iOS and iPad-specific issues that will require scenario updates. For now, I'm preparing a basic scenario to verify 3 out of the 10 possible options (according to the docs).

I feel it might be misleading to place the Android icon next to a screen that doesn't work on that platform yet. However, I can add it if preferred, trusting that the note about the current iOS-only implementation in the scenario description will be enough context.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

…work on both platform just waiting for android implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:tabs Issue related to bottom tabs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants