Skip to content

Conversation

kligarski
Copy link
Contributor

@kligarski kligarski commented Apr 30, 2025

Description

Check which example screens from issues/PRs can be used in e2e testing for tests Test648, Test649 and implement them if possible for Fabric.

Test648

Skipped because we can't check new animation in any meaningful way.

Test649

Test created, only on iOS (headerLargeTitle is supported only on iOS). It checks if headerLargeTitle isn't collapsed when coming back from a screen without headerLargeTitle. I changed Test649 screen to match original issue.

Changes

  • add Test649
  • change header to normal size on Second screen in Test649 to match original issue
  • add comments for every test screen from this PR in apps/src/tests/index.ts with the reason for (not) implementing e2e test for it

Test code and steps to reproduce

CI

Checklist

  • Ensured that CI passes

Copy link
Member

@kkafar kkafar left a comment

Choose a reason for hiding this comment

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

LGTM ✅ Thanks!

@kligarski kligarski merged commit d05c4c3 into main May 5, 2025
5 checks passed
@kligarski kligarski deleted the @kligarski/add-e2e-tests-649 branch May 5, 2025 08:13
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