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: update Ruby recorder code #1173

Merged
merged 2 commits into from
Nov 5, 2023

Conversation

eglitise
Copy link
Collaborator

@eglitise eglitise commented Nov 3, 2023

This PR updates the recorder-generated code for the Ruby lib core client.

  • Fix uiautomator locator strategy
  • Do not add quotes around dictionary keys
  • Fix executeScript
  • Add activateApp and terminateApp
  • Change many methods deprecated in ruby_lib_core 7.3 to their mobile extension equivalents
    • Certain mobile extensions like mobile: setConnectivity first require UI changes
  • Reorder and group driver command implementations

@eglitise eglitise added enhancement New feature or request fix This resolves a user-facing problem labels Nov 3, 2023
app/renderer/lib/client-frameworks/ruby.js Outdated Show resolved Hide resolved
app/renderer/lib/client-frameworks/ruby.js Outdated Show resolved Hide resolved
app/renderer/lib/client-frameworks/ruby.js Outdated Show resolved Hide resolved
app/renderer/lib/client-frameworks/ruby.js Outdated Show resolved Hide resolved
app/renderer/lib/client-frameworks/ruby.js Outdated Show resolved Hide resolved
app/renderer/lib/client-frameworks/ruby.js Outdated Show resolved Hide resolved
Copy link
Member

@KazuCocoa KazuCocoa left a comment

Choose a reason for hiding this comment

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

thank you!

}

codeFor_rotateDevice () {
return `# Not supported: rotateDevice`;
Copy link
Member

Choose a reason for hiding this comment

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

Not for this issue itself. but maybe we no longer function this method itself.
https://github.com/search?q=org%3Aappium%20mobileRotation&type=code

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, they should likely be replaced with getRotation/setRotation. Not sure if the current drivers implement it though.

@eglitise eglitise merged commit ac2e83b into appium:main Nov 5, 2023
5 checks passed
@eglitise eglitise deleted the update-ruby-recorder-code branch November 5, 2023 20:05
shiva-guntoju pushed a commit to shiva-guntoju/appium-inspector that referenced this pull request Feb 2, 2024
* feat: update Ruby recorder code

* chore: address review comments
laib3 pushed a commit to laib3/appium-inspector that referenced this pull request Nov 16, 2024
* feat: update Ruby recorder code

* chore: address review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fix This resolves a user-facing problem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants