Skip to content

Update Search Relevance Workbench Tests - #1941

Draft
wrigleyDan wants to merge 1 commit into
opensearch-project:mainfrom
wrigleyDan:main
Draft

Update Search Relevance Workbench Tests#1941
wrigleyDan wants to merge 1 commit into
opensearch-project:mainfrom
wrigleyDan:main

Conversation

@wrigleyDan

Copy link
Copy Markdown

Description

During the development of opensearch-project/dashboards-search-relevance#804 the integration tests failed due to previous changes in dashboards-search-relevance. This PR updates the tests according to these changes and fixes linting errors:

  • Route rename: singleQueryComparisonqueryAnalysis

The plugin renamed its "Single Query Comparison" template to "Query Analysis", changing the route from singleQueryComparison to queryAnalysis.

  • Removed cy.waitForLoader() calls

waitForLoader() checks for [data-test-subj="homeIcon"], which only exists in the legacy OSD header. When home:useNewHomePage = true is set in the OSD environment, the legacy header is not rendered and homeIcon never appears, causing a 60-second timeout.

Issues Resolved

n/a

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: wrigleyDan <dwrigley@opensourceconnections.com>
@wrigleyDan

Copy link
Copy Markdown
Author

I'm seeing there is some redundancy in #1940
I'll convert this to draft until that one is merged to see what's left that needs addressing.

@wrigleyDan
wrigleyDan marked this pull request as draft March 18, 2026 10:26
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.

1 participant