Skip to content

Conversation

SrinivasanTarget
Copy link
Member

Change list

Fix payload for legacy OSS mode.

Types of changes

What types of changes are you proposing/introducing to Java client?
Put an x in the boxes that apply

  • No changes in production code.
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details

Payload for MJSWP will look like below,

{ "desiredCapabilities": { "app": "/Users/ssekar/workspace/java-client/src/test/java/io/appium/java_client/TestApp.app.zip", "platformVersion": "11.3", "automationName": "XCuiTest", "browserName": "", "platformName": "iOS", "deviceName": "iPhone 8", "launchTimeout": 500000 } } 

payload for W3c will look like below,

{ "desiredCapabilities": { "app": "/Users/ssekar/workspace/java-client/src/test/java/io/appium/java_client/TestApp.app.zip", "platformVersion": "11.3", "automationName": "XCuiTest", "browserName": "", "platformName": "iOS", "deviceName": "iPhone 8", "launchTimeout": 500000 }, "capabilities": { "alwaysMatch": { "appium:app": "/Users/ssekar/workspace/java-client/src/test/java/io/appium/java_client/TestApp.app.zip", "appium:automationName": "XCuiTest", "browserName": "", "appium:deviceName": "iPhone 8", "appium:launchTimeout": 500000, "platformName": "ios", "appium:platformVersion": "11.3" }, "firstMatch": [ { } ] } } 
// And write the first capability for gecko13
json.name(CAPABILITIES);
json.beginObject();
if(!forceMobileJSONWP) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing space

@SrinivasanTarget SrinivasanTarget merged commit 29d3473 into appium:master Jun 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants