Skip to content

chore: remove superseded WebdriverIO e2e suite - #468

Open
bscanlan wants to merge 1 commit into
mainfrom
chore/remove-superseded-wdio-e2e-suite
Open

chore: remove superseded WebdriverIO e2e suite#468
bscanlan wants to merge 1 commit into
mainfrom
chore/remove-superseded-wdio-e2e-suite

Conversation

@bscanlan

Copy link
Copy Markdown
Member

Why?

Nothing runs the WebdriverIO end-to-end suite. CI's UI test jobs use the Maestro
suite instead, and both of those jobs are commented out anyway. Its lockfile has
gone stale and now accounts for most of the repo's open dependency alerts, none
of which ship, because the examples are left out of the published package.

How?

Deletes the suite along with the two stale references left behind in the CI
config and the lint ignore list.

Generated with Claude Code

examples/example/e2e is a WebdriverIO/Appium suite that nothing runs. The
UI tests CircleCI builds are the Maestro ones in examples/example/test --
both android-e2e-test and ios-e2e-test build and run from that directory,
and both jobs are currently commented out of the workflow anyway.

The only reference to examples/example/e2e in CI was a stray
working_directory on the ios job's "Install Maestro" step, which just pipes
a curl to bash and does not care where it runs; the android job runs the
same step with no working_directory. Removed for parity.

Its yarn v1 lockfile has gone stale and accounts for most of the repo's
outstanding dependency update noise. None of it ships: package.json's files
array excludes examples/ from the published package.

Also drops the now-dead eslint ignorePatterns entry for examples/*/e2e/**.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A7XtZXJWpdNvW49CcERj8b
@bscanlan
bscanlan enabled auto-merge (squash) August 30, 2026 15:41
Comment thread .circleci/config.yml
Comment on lines 155 to 165
ios-e2e-test:
<<: *macos_config
steps:
- attach_project

- <<: *setup_env_file

- run:
working_directory: examples/example/e2e
name: Install Maestro
command: curl -Ls "https://get.maestro.mobile.dev" | bash

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.

We can remove this entire stage all together for both iOS and Android

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