Skip to content

test(gui): port remove account connection scenario to appium#878

Merged
saw-jan merged 2 commits into
test/port-to-appiumfrom
port-acc-connection-suite-to-appium
Apr 24, 2026
Merged

test(gui): port remove account connection scenario to appium#878
saw-jan merged 2 commits into
test/port-to-appiumfrom
port-acc-connection-suite-to-appium

Conversation

@prashant-gurung899
Copy link
Copy Markdown
Contributor

@prashant-gurung899 prashant-gurung899 commented Apr 22, 2026

Part of #861

Enable scenario:

@smoke
Scenario: remove an account connection

@prashant-gurung899 prashant-gurung899 changed the base branch from main to test/port-to-appium April 22, 2026 07:16
@prashant-gurung899 prashant-gurung899 changed the title test(gui)Port acc connection suite to appium test(gui): Port acc connection suite to appium Apr 22, 2026
@prashant-gurung899 prashant-gurung899 force-pushed the port-acc-connection-suite-to-appium branch 5 times, most recently from 8cdc8c6 to 8cf2c60 Compare April 23, 2026 08:33
@saw-jan saw-jan force-pushed the port-acc-connection-suite-to-appium branch from 8cf2c60 to b4aeca0 Compare April 24, 2026 07:46
Comment thread test/gui/features/add-account/account.feature Outdated
@saw-jan saw-jan force-pushed the port-acc-connection-suite-to-appium branch 2 times, most recently from 61af882 to 6adc8f8 Compare April 24, 2026 08:39
@saw-jan saw-jan force-pushed the port-acc-connection-suite-to-appium branch from 6adc8f8 to d077316 Compare April 24, 2026 08:40
@saw-jan saw-jan changed the title test(gui): Port acc connection suite to appium test(gui): port remove account connection scenario to appium Apr 24, 2026
@saw-jan saw-jan marked this pull request as ready for review April 24, 2026 08:41
Comment on lines +56 to +61
# ISSUE: https://github.com/opencloud-eu/desktop/pull/879
# Cannot activate account tab by click event
# Select the account tab using keyboard events as a workaround
# TODO: Remove the workaround and uncomment 'click' action
# account_tab.click()
account_tab.send_keys(Keys.TAB)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

fix for this is in #879

@saw-jan saw-jan requested a review from anon-pradip April 24, 2026 10:39
@prashant-gurung899
Copy link
Copy Markdown
Contributor Author

LGTM 👍
Thank you for the help.

@saw-jan saw-jan merged commit dc231b2 into test/port-to-appium Apr 24, 2026
@saw-jan saw-jan deleted the port-acc-connection-suite-to-appium branch April 24, 2026 10:54
saw-jan added a commit that referenced this pull request May 12, 2026
* port removeAccountConnection test suite to appium

Signed-off-by: prashant-gurung899 <[email protected]>

* test: fix account selection

Signed-off-by: Saw-jan <[email protected]>

---------

Signed-off-by: prashant-gurung899 <[email protected]>
Signed-off-by: Saw-jan <[email protected]>
Co-authored-by: Saw-jan <[email protected]>
saw-jan added a commit that referenced this pull request May 13, 2026
* port removeAccountConnection test suite to appium

Signed-off-by: prashant-gurung899 <[email protected]>

* test: fix account selection

Signed-off-by: Saw-jan <[email protected]>

---------

Signed-off-by: prashant-gurung899 <[email protected]>
Signed-off-by: Saw-jan <[email protected]>
Co-authored-by: Saw-jan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants