Skip to content

test(gui): fix and enable settings test scenarios#911

Merged
saw-jan merged 5 commits into
mainfrom
test/enable-settings-checks
May 20, 2026
Merged

test(gui): fix and enable settings test scenarios#911
saw-jan merged 5 commits into
mainfrom
test/enable-settings-checks

Conversation

@saw-jan
Copy link
Copy Markdown
Contributor

@saw-jan saw-jan commented May 18, 2026

fixed and enabled the following test scenarios

Scenario: Tabs in toolbar looks correct
Scenario: Verify various setting options in Settings tab

Fixes #849
Closes #850

Comment thread test/gui/features/tabs-settings/tabsSettings.feature
@saw-jan saw-jan force-pushed the test/enable-settings-checks branch from 0692580 to 4c960e0 Compare May 19, 2026 05:52
@@ -3,7 +3,7 @@ Feature: Visually check all tabs
I want to visually check all tabs in client
So that I can perform all the actions related to client


@smoke
Scenario: Tabs in toolbar looks correct
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.

The tests fails.
Am i doing something wrong?

  Scenario: Verify various setting options in Settings tab                          # features/tabs-settings/test.feature:17
    Given user "Alice" has been created in the server with default attributes       # steps/server_context.py:8 0.072s
    And user "Alice" has set up a client with default settings                      # steps/account_context.py:55
      Traceback (most recent call last):
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/behave/model.py", line 1991, in run
          match.run(runner.context)
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/behave/matchers.py", line 105, in run
          self.func(context, *args, **kwargs)
        File "steps/account_context.py", line 62, in step
          enter_password.login_after_setup(username, password)
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/pageObjects/EnterPassword.py", line 39, in login_after_setup
          self.oidc_relogin(username, password)
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/pageObjects/EnterPassword.py", line 33, in oidc_relogin
          authorize_via_webui(username, password)
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/helpers/WebUIHelper.py", line 12, in authorize_via_webui
          page.goto(url)
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/playwright/sync_api/_generated.py", line 9054, in goto
          self._sync(
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/playwright/_impl/_sync_base.py", line 115, in _sync
          return task.result()
                 ^^^^^^^^^^^^^
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/playwright/_impl/_page.py", line 552, in goto
          return await self._main_frame.goto(**locals_to_params(locals()))
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/playwright/_impl/_frame.py", line 153, in goto
          await self._channel.send(
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/playwright/_impl/_connection.py", line 69, in send
          return await self._connection.wrap_api_call(
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/home/prashant/Desktop/projects/oc-desktop/desktop/test/gui/venv/lib/python3.12/site-packages/playwright/_impl/_connection.py", line 559, in wrap_api_call
          raise rewrite_error(error, f"{parsed_st['apiName']}: {error}") from None
      playwright._impl._errors.Error: Page.goto: Protocol error (Page.navigate): Cannot navigate to invalid URL
      Call log:
        - navigating to "Cannot navigate to invalid URL", waiting until "load"

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.

browser login failed.

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.

I am also facing same issue

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, that's weird. Could you check your chromium version?

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.

or check with headed mode to see what happens:

browser = pw.chromium.launch(headless=True)

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.

Yes it accessible.
image

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.

This scenario is fine.

Scenario: Tabs in toolbar looks correct

But another one is failing:

Scenario: Verify various setting options in Settings tab                          # features/tabs-settings/tabsSettings.feature:17
    Given user "Alice" has been created in the server with default attributes       # steps/server_context.py:8 0.070s
    And user "Alice" has set up a client with default settings                      # steps/account_context.py:55
    And user "Alice" has set up a client with default settings                      # steps/account_context.py:55 2.801s
    When the user opens the settings tab                                            # steps/sync_context.py:86 0.217s
      ASSERT FAILED: Settings tab is not active

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.

might be similar issue: #910 (comment)

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.

Yes, that's correct in this test case as well.

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.

let's see into this issue separately.

@saw-jan saw-jan force-pushed the test/enable-settings-checks branch from 4c960e0 to 2491dd3 Compare May 19, 2026 10:46
Comment thread test/gui/features/tabs-settings/tabsSettings.feature Outdated
saw-jan added 4 commits May 20, 2026 12:32
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
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-settings-checks branch from 2491dd3 to f512770 Compare May 20, 2026 06:55
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.

LGTM 💯

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/enable-settings-checks branch from 3fd26f1 to 67889e4 Compare May 20, 2026 07:20
@saw-jan saw-jan merged commit 77b191d into main May 20, 2026
10 of 12 checks passed
@saw-jan saw-jan deleted the test/enable-settings-checks branch May 20, 2026 07:53
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.

[QA] Failing gui test

3 participants