test(gui): port to appium for sync resources feature#947
Conversation
Asmitapaudel
force-pushed
the
test/port-syncResource
branch
from
aa15d5a to
618abea
Compare
| if ( | ||
| p_element.get_attribute("checked") == 'true' | ||
| and p_element.rect["x"] > parent_position | ||
| if p_element.rect["x"] >= parent_position and ( |
There was a problem hiding this comment.
When selecting folders, deselect_all_remote_folders() unchecks everything first, so the checked == 'true' condition never matched and parent_element stayed None. When unselecting, the check is still needed to pick the correct parent when multiple elements with the same name exist in the tree. Now the checked state is only enforced when unselecting.
Asmitapaudel
changed the title
Asmitapaudel
force-pushed
the
test/port-syncResource
branch
from
618abea to
bc7b2c3
Compare
|
|
Asmitapaudel
force-pushed
the
test/port-syncResource
branch
9 times, most recently
from
be29c29 to
77aa667
Compare
| SyncConnection.FOLDER_SYNC_CONNECTION_MENU_BUTTON.selector.format( | ||
| sync_folder=sync_folder, | ||
| sync_path=get_config("currentUserSyncPath"), | ||
| sync_path=get_config("currentUserSyncPath").rstrip("/") + "/", |
There was a problem hiding this comment.
Ensures the sync path always ends with / so the locator works consistently.
Asmitapaudel
force-pushed
the
test/port-syncResource
branch
2 times, most recently
from
3571821 to
74c462f
Compare
| with zipfile.ZipFile(zip_file_name, 'w') as zipped_file: | ||
| for resource in resources: | ||
| zipped_file.write(resource) | ||
| try: |
There was a problem hiding this comment.
Fixed FileNotFoundError in after_scenario cleanup. The create_zip() function was permanently changing the working directory without restoring it, causing subsequent cleanup operations to fail. Added try-finally block to ensure the original directory is always restored.
There was a problem hiding this comment.
Please, also link the error that you are referring to.
I guess you are referring to this error:
https://ci.opencloud.rocks/repos/11/pipeline/96/44#L848
Asmitapaudel
force-pushed
the
test/port-syncResource
branch
from
74c462f to
6af9aac
Compare
4 participants
Part of: #861
This PR enables the following syncResources test: