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

QAWTO-214: update appium python client dependency #404

Merged

Conversation

hesteban-tuenti
Copy link
Contributor

As part of the maintenance of the toolium dependency libraries, the update of the appium python client is going to be update to the version 4.
In order to update to this version we identified that the following actions needed to be updated to be compatible with the w3C webdriver:
https://jira.tid.es/browse/QAWTO-178

In order to be fully W3C compatible we need to change the set_value method used in the InputText pagelement to send_keys

test send_keys ios

test get from value ios

restore set_value

restore send_keys

set_text

test set value
@hesteban-tuenti hesteban-tuenti self-assigned this Oct 29, 2024
Copy link

codeclimate bot commented Oct 29, 2024

Analysis results are not available for those commits

View more on Code Climate.

@hesteban-tuenti
Copy link
Contributor Author

reviso los tests cuando tenga un momento

requirements.txt Outdated Show resolved Hide resolved
@hesteban-tuenti
Copy link
Contributor Author

hesteban-tuenti commented Nov 8, 2024

I have tested the new mobile_actions features (Android e iOS) which tests all of the detected changes in Appium 3 , and 4 and also the change in the InputText done in this PR with the following versions:

  • Appium-Python-Client= 2.11.1 - Selenium=4.11.2
  • Appium-Python-Client= 3.2.1 - Selenium=4.25
  • Appium-Python-Client= 4.2.1 Selenium=4.25

And the results have been OK, so I guess we don't break the compatibility backwards and don't need a mayor version.

rgonalo
rgonalo previously approved these changes Nov 20, 2024
@rgonalo rgonalo merged commit 7137397 into Telefonica:master Nov 25, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants