-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
Add mobile: command for performing editor actions #428
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After https://github.com/appium/appium-adb/pull/354/files/47dcc042040279617083385bcee33060091273ad#diff-83add4eba9c23315341d94aec97ce76b and adding this PR's comment, normal
action worked 👍
I'll check further, but this way works 👍
[HTTP] --> POST /wd/hub/session/1bef54aa-ea68-46bb-b0da-8b64826d2688/execute/sync
[HTTP] {"script":"mobile: performEditorAction","args":[{"action":"normal"}]}
[debug] [W3C] Calling AppiumDriver.execute() with args: ["mobile: performEditorAction",[{"action":"normal"}],"1bef54aa-ea68-46bb-b0da-8b64826d2688"]
[debug] [ADB] Performing editor action: normal
[debug] [ADB] Running '/Users/kazuaki/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell settings get secure default_input_method'
[debug] [ADB] Running '/Users/kazuaki/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell ime enable io.appium.settings/.AppiumIME'
[debug] [ADB] Running '/Users/kazuaki/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell ime set io.appium.settings/.AppiumIME'
[debug] [ADB] Running '/Users/kazuaki/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell input text /normal/'
[debug] [ADB] Running '/Users/kazuaki/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell ime set io.appium.android.ime/.UnicodeIME'
[debug] [W3C] Responding to client with driver.execute() result: null
[HTTP] <-- POST /wd/hub/session/1bef54aa-ea68-46bb-b0da-8b64826d2688/execute/sync 200 1404 ms - 14
[HTTP]
startLogsBroadcast: 'mobileStartLogsBroadcast', | ||
stopLogsBroadcast: 'mobileStopLogsBroadcast', | ||
|
||
performEditorAction: 'mobilePerformEditorAction', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add https://github.com/appium/appium-uiautomator2-driver/blob/7af9866353a06f8a7f22675912082fe979811254/lib/commands/general.js#L81 as well?
Since no command error raised without performEditorAction: 'mobilePerformEditorAction',
in appium-uiautomator2-driver
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, will add it later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want the other components to be published first
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the other one is publisehd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm. appium-adb
is published.
Make sure to update: https://github.com/appium/appium/edit/master/commands-yml/commands/mobile-command.yml as well |
the PR depends on appium/appium-adb#354