Skip to content

[OCISDEV-843] Flaky search test - 2nd fix#13800

Open
deyankiteworks wants to merge 1 commit into
masterfrom
fix/OCISDEV-843-fix
Open

[OCISDEV-843] Flaky search test - 2nd fix#13800
deyankiteworks wants to merge 1 commit into
masterfrom
fix/OCISDEV-843-fix

Conversation

@deyankiteworks
Copy link
Copy Markdown
Member

@deyankiteworks deyankiteworks commented May 20, 2026

Summary

  • Uses resourcePage.filesList instead of resourcePage.searchList for asserting search results after pressing Enter
  • After pressing Enter, the test navigates to the full search results page which renders via ResourceTable with [data-test-resource-path] selectors — NOT the dropdown (#files-global-search-options)
  • Removes the unnecessary 2000ms waitForTimeout workaround from the first fix attempt

Related Issue

Test plan

  • Verify the "Search in personal spaces" test passes consistently in CI
  • Confirm no regressions in other search tests

🤖 Generated with Claude Code

@update-docs
Copy link
Copy Markdown

update-docs Bot commented May 20, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@deyankiteworks deyankiteworks self-assigned this May 20, 2026
@deyankiteworks deyankiteworks requested review from LukasHirt and mzner May 20, 2026 11:04
@sonarqubecloud
Copy link
Copy Markdown

await ui.userShouldSeeResources({
world,
listType: resourcePage.searchList,
listType: resourcePage.filesList,
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.

@deyankiteworks Are we sure we want to use filesList instead of searchList? Why was it passing before then?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants