Skip to content

Test/7 maestro e2e tests#40

Merged
mateusz1913 merged 3 commits intocallstackincubator:mainfrom
wjaszczuk:test/7-maestro-e2e-tests
Jun 10, 2025
Merged

Test/7 maestro e2e tests#40
mateusz1913 merged 3 commits intocallstackincubator:mainfrom
wjaszczuk:test/7-maestro-e2e-tests

Conversation

@wjaszczuk
Copy link
Copy Markdown
Contributor

@wjaszczuk wjaszczuk commented Jun 6, 2025

This PR introduces Maestro e2e tests.

They are run for iOS and Android for every PR targeting the main branch, and push to the main branch.
E2E test is basic. It opens the app, clicks a button, traverses through the license list, selects React Native, and then checks if it opens the proper license and goes to GitHub.

Tests run on:

  • Android:
    • API 30
    • API 34
    • API 35
  • iOS:
    • Phone SE (3rd generation) (18.0)
    • iPhone 16 Pro (18.5)

All tests record their run in MP4 and make screenshots. They are attached as test artefacts.
The tests also upload test results in the JUnit format.

This PR fixes also some potential issue, where plugin stop executing when Settings.plist was already registered in xcode project.

closes #7

Remove the bundle just before it is added to project
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 6, 2025

🦋 Changeset detected

Latest commit: 27b9be4

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@wjaszczuk
Copy link
Copy Markdown
Contributor Author

hey @mateusz1913 could you take a look?

Copy link
Copy Markdown
Collaborator

@mateusz1913 mateusz1913 left a comment

Choose a reason for hiding this comment

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

Looks awesome! Left one question and will check it manually before merging

Copy link
Copy Markdown
Collaborator

@mateusz1913 mateusz1913 left a comment

Choose a reason for hiding this comment

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

Only the branch triggers needs to be updated and we can 🚢

Comment thread .github/workflows/test-e2e-android.yaml Outdated
Comment thread .github/workflows/test-e2e-android.yaml Outdated
Comment thread .github/workflows/test-e2e-ios.yaml Outdated
Comment thread .github/workflows/test-e2e-ios.yaml Outdated
@wjaszczuk wjaszczuk force-pushed the test/7-maestro-e2e-tests branch from 2ecd53c to 27b9be4 Compare June 10, 2025 06:32
@wjaszczuk
Copy link
Copy Markdown
Contributor Author

@mateusz1913 I've rewritten the history. I amended the test commit and changed the triggers; the changelist commit is still the last one here.

@mateusz1913 mateusz1913 merged commit 58a46f1 into callstackincubator:main Jun 10, 2025
6 checks passed
@github-actions github-actions Bot mentioned this pull request Jun 10, 2025
@wjaszczuk wjaszczuk deleted the test/7-maestro-e2e-tests branch June 10, 2025 14:22
@artus9033 artus9033 mentioned this pull request Jun 25, 2025
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.

implement e2e tests

2 participants