Skip to content

refactor: modularize Appium local runner#41

Merged
fadi-george merged 5 commits into
mainfrom
fg/clean-up
Jun 23, 2026
Merged

refactor: modularize Appium local runner#41
fadi-george merged 5 commits into
mainfrom
fg/clean-up

Conversation

@fadi-george

Copy link
Copy Markdown
Contributor

Description

Modularizes the Appium local runner so build, config, runtime preparation, and SDK artifact setup live behind smaller script modules. Also tightens --pods handling for matrix runs and fixes Cordova iOS project-based builds.

Details

  • Split run-local.sh into sourced modules for config, build, runtime preparation, shared helpers, and local SDK artifacts.
  • Added run-all.sh --pods routing so it only forwards --pods to Flutter, Cordova, and Capacitor, with one warning for unsupported selected SDKs.
  • Updated Cordova iOS build selection to use App.xcworkspace when present and fall back to App.xcodeproj for SwiftPM/project-based demos.
  • No breaking changes intended; existing run-local.sh flags continue to work.

Testing

  • bash -n appium/scripts/run-all.sh appium/scripts/run-local.sh appium/scripts/run-local/build.sh appium/scripts/run-local/runtime.sh
  • appium/scripts/run-all.sh --help
  • appium/scripts/run-all.sh --sdks=android --platform=ios --pods
  • ./run-local.sh --sdk=cordova --platform=ios --pods completed successfully with 34 passing tests.

Made with Cursor

@fadi-george fadi-george requested a review from a team as a code owner June 23, 2026 16:48
@fadi-george fadi-george merged commit 1310adc into main Jun 23, 2026
2 checks passed
@fadi-george fadi-george deleted the fg/clean-up branch June 23, 2026 19:20
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