Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add appPath to handleCreateSession #230

Merged

Conversation

proAlexandr
Copy link
Contributor

TL;DR: Allows to explicitly specify path to .app application that will be opened by driver.

Context:
During development you can have many build of the same application with the same bundleId on your machine. You have no control which one will be opened by [[XCUIApplication alloc] initWithBundleIdentifier:bundleID]. So you can end up testing wrong build (for example, production build from /Application folder instead of dev build that includes local changes).

This PR introduces appPath argument for /session request to specify a path to .app application and be sure that webdriver launches exactly the same build that you need.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 6, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@mykola-mokhnach
Copy link
Contributor

@proAlexandr Please sign the CLA

@proAlexandr proAlexandr force-pushed the feat-session-create-app-path branch from a84beb3 to cd90662 Compare September 7, 2023 09:19
@proAlexandr
Copy link
Contributor Author

Hi @mykola-mokhnach. Thanks a lot for review!
I have signed SLA and updated the PR according to comments.

@proAlexandr proAlexandr force-pushed the feat-session-create-app-path branch from cd90662 to 3f40a05 Compare September 7, 2023 09:21
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@proAlexandr proAlexandr force-pushed the feat-session-create-app-path branch from 5537c4e to f6400f9 Compare September 8, 2023 12:42
@mykola-mokhnach mykola-mokhnach merged commit 7b7cc17 into appium:master Sep 8, 2023
8 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 8, 2023
## [1.8.0](v1.7.4...v1.8.0) (2023-09-08)

### Features

* add appPath to handleCreateSession ([#230](#230)) ([7b7cc17](7b7cc17))
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

🎉 This PR is included in version 1.8.0 🎉

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