Skip to content

test(gui): fix and enable add-account scenarios#876

Merged
saw-jan merged 5 commits into
test/port-to-appiumfrom
test/enable-account-tests
Apr 23, 2026
Merged

test(gui): fix and enable add-account scenarios#876
saw-jan merged 5 commits into
test/port-to-appiumfrom
test/enable-account-tests

Conversation

@saw-jan
Copy link
Copy Markdown
Contributor

@saw-jan saw-jan commented Apr 21, 2026

Part of #861

  • Fixed and enabled scenarios:
    @smoke
    Scenario: Re-add an account
  • added a data table parser
    table_raw()
    table_rows()
    table_rows_hash()
    table_hashes()

@saw-jan saw-jan self-assigned this Apr 21, 2026
@saw-jan saw-jan force-pushed the test/enable-account-tests branch from 8362d7c to 40643c1 Compare April 22, 2026 06:42
saw-jan added 2 commits April 22, 2026 13:19
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/enable-account-tests branch from 40643c1 to ac176bf Compare April 22, 2026 07:49

@smoke @skip
@smoke
Scenario: Add space manually from sync connection window
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@prashant-gurung899 @anon-pradip Please, try to run this scenario. It passed once on my machine but refuse to pass now.

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.

In my machine, it didn't pass even once.

Scenario: Add space manually from sync connection window
    Given user "Alice" has been created in the server with default attributes ... passed in 0.107s
    Given user "Alice" has created folder "simple-folder" in the server ... passed in 0.084s
    And the user has started the client ... passed in 1.046s
    And the user has entered the following account information: ... passed in 3.241s
      | server   | %local_server% |
      | user     | Alice          |
      | password | 1234           |
    When the user selects manual sync folder option in advanced section ... passed in 0.640s
    And the user syncs the "Personal" space ... failed in 0.679s
ASSERT FAILED: Next button is not enabled
Deleting folder: Alice

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

hmm, yeah. that's the error. space is not being selected.
I will skip the test for now and enable with fix later on

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this will be fixed by #879, but I need review from the dev team

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.

Cool. Then, we can merge this one.

@saw-jan saw-jan changed the title test(gui): enable manual folder sync scenario test(gui): fix and enable add-account scenarios Apr 22, 2026
@saw-jan saw-jan marked this pull request as ready for review April 22, 2026 07:53
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
saw-jan added 2 commits April 22, 2026 16:18
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Copy link
Copy Markdown
Contributor

@prashant-gurung899 prashant-gurung899 left a comment

Choose a reason for hiding this comment

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

@saw-jan saw-jan merged commit b65da6f into test/port-to-appium Apr 23, 2026
0 of 3 checks passed
@saw-jan saw-jan deleted the test/enable-account-tests branch April 23, 2026 04:14
@saw-jan saw-jan mentioned this pull request Apr 23, 2026
30 tasks
saw-jan added a commit that referenced this pull request May 12, 2026
* test(gui): enable manual folder sync scenario

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): enable account re-add scenario

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): skip manual sync scenario

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): check space selection

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): remove obsolete step files

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

---------

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
saw-jan added a commit that referenced this pull request May 13, 2026
* test(gui): enable manual folder sync scenario

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): enable account re-add scenario

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): skip manual sync scenario

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): check space selection

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test(gui): remove obsolete step files

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

---------

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
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.

2 participants