Skip to content

test: Stabilize basic e2e test suite - #2928

Merged
mykola-mokhnach merged 3 commits into
appium:masterfrom
mykola-mokhnach:test1
Jul 30, 2026
Merged

test: Stabilize basic e2e test suite#2928
mykola-mokhnach merged 3 commits into
appium:masterfrom
mykola-mokhnach:test1

Conversation

@mykola-mokhnach

Copy link
Copy Markdown
Contributor

No description provided.

@mykola-mokhnach
mykola-mokhnach marked this pull request as draft July 30, 2026 18:20
node silently ignores CLI flags placed after a positional argument, so
--test-concurrency=1 in the functional-test workflow was never actually
applied (it came after the glob pattern). This let basic's spec files
race each other for the same simulator session, tripping the
session-claim-handler's obsolete-session termination against an
in-flight WDA launch.

Have the CI step call the corresponding e2e-test:<root> npm script
instead of duplicating the node --test invocation, so there's a single
source of truth for these flags.
@mykola-mokhnach
mykola-mokhnach marked this pull request as ready for review July 30, 2026 19:09
@mykola-mokhnach
mykola-mokhnach merged commit 028e937 into appium:master Jul 30, 2026
9 of 10 checks passed
@mykola-mokhnach
mykola-mokhnach deleted the test1 branch July 30, 2026 19:42
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 12.1.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants