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

bug: appium 2.0.0-beta53 seems to ignore appium:options object #18191

Closed
1 task done
radoslaw-karasinski-sauce opened this issue Feb 13, 2023 · 1 comment · Fixed by #18195
Closed
1 task done

bug: appium 2.0.0-beta53 seems to ignore appium:options object #18191

radoslaw-karasinski-sauce opened this issue Feb 13, 2023 · 1 comment · Fixed by #18195
Assignees
Labels
Bug a problem that needs fixing

Comments

@radoslaw-karasinski-sauce
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

derivedDataPath and usePrebuiltWDA setting seems to be ignored in appium 2.0.0-beta53 - xcodebuild test-without-building call doesn't use provided values.

Appium 2.0.0-beta53 seems to also ignore other caps provided in appium:options, i.e. connectHardwareKeyboard. This method was working for 2.0.0-beta44.

Appium beta53 CLI call: Appium command is: /Volumes/Sauce/selenium/appium-v2.0.0-beta53/node-darwin --max_old_space_size=1024 /Volumes/Sauce/selenium/appium-v2.0.0-beta53 --log-timestamp --callback-address 127.0.0.1 --address 127.0.0.1 --port 4443 --log-no-colors --default-capabilities {"appium:options": {"usePrebuiltWDA": true, "preventWDAAttachments": true, "orientation": "PORTRAIT", "launchTimeout": 180000, "showIOSLog": false, "noReset": true, "eventTimings": true, "backendRetries": 4, "derivedDataPath": "/Volumes/Sauce/wda/wda-v2.0.0-beta53-xcode13.0/WebDriverAgent/DerivedData/WebdriverAgent", "bootstrapPath": "/Volumes/Sauce/wda/wda-v2.0.0-beta53-xcode13.0/WebDriverAgent", "maxTypingFrequency": 8, "newCommandTimeout": 0, "connectHardwareKeyboard": true, "simulatorPasteboardAutomaticSync": "on", "app": "/var/folders/z9/rg8m2_gs5yq64gd5h71df2cc0000kr/T/tmpC2WXHT/TestApp7.2.app.zip", "keepKeyChains": true}} --relaxed-security --base-path /wd/hub

Appium beta44 CLI call: Appium command is: /Volumes/Sauce/selenium/appium-v2.0.0-beta44-separate-homes/node-darwin --max_old_space_size=1024 /Volumes/Sauce/selenium/appium-v2.0.0-beta44-separate-homes --log-timestamp --callback-address 127.0.0.1 --address 127.0.0.1 --port 4443 --log-no-colors --default-capabilities {"appium:options": {"usePrebuiltWDA": true, "preventWDAAttachments": true, "orientation": "PORTRAIT", "launchTimeout": 180000, "showIOSLog": false, "noReset": true, "eventTimings": true, "backendRetries": 4, "derivedDataPath": "/Volumes/Sauce/wda/wda-v2.0.0-beta44-xcode14.0/WebDriverAgent/DerivedData/WebdriverAgent", "maxTypingFrequency": 8, "newCommandTimeout": 0, "connectHardwareKeyboard": true, "simulatorPasteboardAutomaticSync": "on", "app": "/var/folders/z9/rg8m2_gs5yq64gd5h71df2cc0000kr/T/tmpkPWe1U/TestApp7.2.app.zip", "keepKeyChains": true}} --relaxed-security --base-path /wd/hub

Expected Behavior

Appium should use provided derivedDataPath to pass it to xcodebuild test-without-building call.

Minimal Reproducible Example

Start any session with derivedDataPath and usePrebuiltWDA inside appium:options caps.

Environment

All informations in attached gist.

Link to Appium Logs

https://gist.github.com/radoslaw-karasinski-sauce/3bac813bf3080f05cae22cd15ff0a8e7

Futher Information

No response

@radoslaw-karasinski-sauce radoslaw-karasinski-sauce added Bug a problem that needs fixing Needs Triage bugs which are not yet confirmed labels Feb 13, 2023
@radoslaw-karasinski-sauce radoslaw-karasinski-sauce changed the title bug: appium 2.0.0-beta53 seems to ignore derivedDataPath bug: appium 2.0.0-beta53 seems to ignore appium:options object Feb 13, 2023
@mykola-mokhnach mykola-mokhnach removed the Needs Triage bugs which are not yet confirmed label Feb 13, 2023
@jlipps
Copy link
Member

jlipps commented Feb 14, 2023

this should be fixed in the next release. previously there was no intentional support for using appium:options together with --default-capabilities; it only worked accidentally! (though of course it should be possible to combine these features). the reason it regressed is because we changed where appium:options was processed to fix another issue. now we have tests for both cases to prevent further regressions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug a problem that needs fixing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants