Skip to content

test(gui): fix and enable activity filter scenario#910

Merged
saw-jan merged 3 commits into
mainfrom
test/fix-filter-click
May 20, 2026
Merged

test(gui): fix and enable activity filter scenario#910
saw-jan merged 3 commits into
mainfrom
test/fix-filter-click

Conversation

@saw-jan
Copy link
Copy Markdown
Contributor

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

Fix the activity filter selection and enable the scenario

@saw-jan saw-jan self-assigned this May 18, 2026
@saw-jan saw-jan changed the title test: fix and enable activity filter scenario test(gui): fix and enable activity filter scenario May 18, 2026
@saw-jan saw-jan force-pushed the test/fix-filter-click branch from 51dac14 to 8f6e93e Compare May 18, 2026 12:01
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/fix-filter-click branch from 8f6e93e to 1cd3af0 Compare May 19, 2026 04:29

@smoke @skip
@smoke
Scenario: filter synced activities
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 test is failing in my local machine.
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.

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, this should be fixed. THe CI failure is different one which I fixed very recently.
Let me check.

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.

Could you pull this again and test? It's working for me

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.

It is working fine on my system

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 test is passing now.
image

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.

I think, the test failed because there's popup in the first screen

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.

Wait, the test is kind of flaky.
The test failes when the client opens up on the first(left) screen.
But passes when the client opens up on the second(right) screen.

@saw-jan

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 think, the test failed because there's popup in the first screen

Yes, you are right.

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.

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/fix-filter-click branch from 1cd3af0 to a3b2b49 Compare May 19, 2026 10:45
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.

Approving this PR as per our discussion. We will work on finding the solution if the problem occurs more often(even in CI).

@saw-jan saw-jan merged commit c279415 into main May 20, 2026
12 checks passed
@saw-jan saw-jan deleted the test/fix-filter-click branch May 20, 2026 05:02
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