Skip to content

fix: remove Appium logLevel arg to prevent orphaned processes - #42

Merged
tai2 merged 2 commits into
mainfrom
add-e2e-scenarios
Apr 17, 2026
Merged

fix: remove Appium logLevel arg to prevent orphaned processes#42
tai2 merged 2 commits into
mainfrom
add-e2e-scenarios

Conversation

@tai2

@tai2 tai2 commented Apr 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Removed logLevel: "warn" from the Appium service args in wdio.ios.conf.ts
  • The warn log level suppressed the info-level startup message ("Appium REST http interface listener started") that @wdio/appium-service watches for to detect readiness
  • This caused the service to always time out, lose its process reference, and never clean up Appium on exit — leaving orphaned node appium and xcodebuild processes after every test run

Test plan

  • Ran pnpm e2e:ios --mochaOpts.grep "discards recording when paused mid-recording" and confirmed:
    • Appium starts successfully ("Appium started with ID: ...")
    • Cleanup runs on exit ("Killing entire Appium tree" / "Process and its children successfully terminated")
    • No orphaned processes remain after the test

🤖 Generated with Claude Code

tai2 and others added 2 commits April 17, 2026 16:00
…st playback, multiple recordings, and pause during recording

Add 8 new E2E tests across 5 suites covering previously untested user flows.
Add accessibility labels to settings button, reset button, empty state, and
recordings count for E2E discoverability. Move E2E banner below nav bar to
avoid overlapping header controls.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The `logLevel: "warn"` arg suppressed Appium's info-level startup
message that @wdio/appium-service uses to detect readiness. This caused
the service to always time out, lose its process reference, and skip
cleanup on exit — leaving orphaned Appium and xcodebuild processes
after every test run.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tai2
tai2 merged commit ce7de51 into main Apr 17, 2026
1 check passed
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.

1 participant