[TESTID-64,80,UI] Add cypress test for autocomplete feature #9322

ananzh · 2025-02-03T15:56:23Z

Description

Add tests related to autocomplete.

Issues Resolved

This will close all the issues listed here:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20

Screenshot

NA

Changelog

-test: [TESTID-64] Add cypress test for auto query updates when switch dataset

Check List

All tests pass
- yarn test:jest
- yarn test:jest_integration
New functionality includes testing.
New functionality has been documented.
Update CHANGELOG.md
Commits are signed per the DCO using --signoff

codecov · 2025-02-03T16:27:21Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.71%. Comparing base (8cc5f84) to head (6a8c9c2).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #9322   +/-   ##
=======================================
  Coverage   61.71%   61.71%           
=======================================
  Files        3817     3817           
  Lines       91841    91841           
  Branches    14545    14545           
=======================================
+ Hits        56677    56679    +2     
+ Misses      31508    31507    -1     
+ Partials     3656     3655    -1

Flag	Coverage Δ
Linux_1	`28.98% <ø> (ø)`
Linux_2	`56.46% <ø> (ø)`
Linux_3	`39.19% <ø> (?)`
Linux_4	`28.90% <ø> (ø)`
Windows_1	`29.00% <ø> (ø)`
Windows_2	`56.41% <ø> (ø)`
Windows_3	`39.19% <ø> (-0.01%)`	⬇️
Windows_4	`28.90% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

angle943 · 2025-02-12T05:18:24Z

...ensearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_query.spec.js

+        url: PATHS.SECONDARY_ENGINE,
+        authType: 'no_auth',
+      });
+      cy.deleteAllWorkspaces();


deleteWorkSpaceByName()

angle943 · 2025-02-12T05:19:35Z

...ensearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_query.spec.js

+      (config) => {
+        describe(`${config.testName}`, () => {
+          beforeEach(() => {
+            if (config.datasetType === 'INDEX_PATTERN') {


nit: config.dataType === DataTypes.IndexPattern.name (or something like that)

angle943 · 2025-02-12T05:20:01Z

...ensearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_query.spec.js

+            });
+          });
+
+          it('should show and select suggestions progressively', function () {


very nitpick: arrow function

angle943 · 2025-02-12T05:20:54Z

...nsearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_switch.spec.js

+        url: PATHS.SECONDARY_ENGINE,
+        authType: 'no_auth',
+      });
+      cy.deleteAllWorkspaces();


deleteWorkspaceByName

angle943 · 2025-02-12T05:21:06Z

...nsearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_switch.spec.js

+    }).forEach((config) => {
+      describe(`${config.testName}`, () => {
+        beforeEach(() => {
+          if (config.datasetType === 'INDEX_PATTERN') {


nit: DataTypes.IndexPattern.name

angle943 · 2025-02-12T05:21:35Z

..._opensearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_ui.spec.js

+        url: PATHS.SECONDARY_ENGINE,
+        authType: 'no_auth',
+      });
+      cy.deleteAllWorkspaces();


deleteWorkspaceByName

angle943 · 2025-02-13T19:00:26Z

cypress/utils/apps/query_enhancements/commands.js

@@ -3,7 +3,80 @@
 * SPDX-License-Identifier: Apache-2.0
 */

-Cypress.Commands.add('setQueryEditor', (value, opts = {}, submit = true) => {


This opts is needed for the ignoreSpecialSequence or whatever it was called

angle943 · 2025-02-13T19:00:31Z

cypress/utils/apps/query_enhancements/commands.js

-    .type('{backspace}')
-    .type(value, opts);
+  clearMonacoEditor().then(() => {
+    return cy.get('.inputarea').should('be.visible').wait(200).type(value, { force: true });


pass in the option here

Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]>

sejli

Some nits, but LGTM

sejli · 2025-02-13T23:45:28Z

..._opensearch_dashboards/opensearch_dashboards/apps/query_enhancements/autocomplete_ui.spec.js

+            }
+
+            cy.waitForLoader(true);
+            cy.wait(2000);


Noticed that we are using cy.wait() for some time after we use the cy.waitForLoader() function in a few cases. Is there a reason why we need the extra wait?

Just see some flakiness when test 3 times in a row. Put a bit more time can ensure they success 3 times.

sejli · 2025-02-13T23:49:45Z

cypress/utils/apps/query_enhancements/commands.js

+    .wait(200); // Wait for focus to take effect
+};
+
+const clearMonacoEditor = () => {


One thing that usually works for me in Cypress is to use .type("{selectall}{backspace}{selectall}{backspace}") to clear inputs, not sure if it would work in this case though.

thanks. so far this is the most stable way to avoid flakiness.

opensearch-trigger-bot · 2025-02-14T00:53:08Z

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/OpenSearch-Dashboards/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/OpenSearch-Dashboards/backport-2.x
# Create a new branch
git switch --create backport/backport-9322-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 3c6c806d55b837bdb16295b6066a312c5d9a39db
# Push it to GitHub
git push --set-upstream origin backport/backport-9322-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/OpenSearch-Dashboards/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-9322-to-2.x.

…ch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>

…ch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]>

* add max recent queries tests Signed-off-by: Federico Silva <[email protected]> * refactor callback function Signed-off-by: Federico Silva <[email protected]> * add testid-42 tests Signed-off-by: Federico Silva <[email protected]> * add comments Signed-off-by: Federico Silva <[email protected]> * add spec to package.json Signed-off-by: Federico Silva <[email protected]> * fix testconfig fn name Signed-off-by: Federico Silva <[email protected]> * fix clipboard flakiness Signed-off-by: Federico Silva <[email protected]> * refactor test preparation Signed-off-by: Federico Silva <[email protected]> * fix package.json, yarn.locl and cypress.config.ts Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR #9307 created/updated Signed-off-by: Federico Silva <[email protected]> * remove element.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Refactor based on #9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * comment out tests with dependencies Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR #9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR #9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (#9384) Issue Resolved: #8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Anan <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

* Update actions/cache from v1 to v4 to address deprecation warning (#9366) * build: Update actions/cache from v1 to v4 to address deprecation warning Signed-off-by: Anan <[email protected]> * Changeset file for PR #9366 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * Handle invalid geospatial request in region map (#8759) Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Federico Silva <[email protected]> * Update DEVELOPER_GUIDE.md (#9368) When building the OpenSearch Dashboards artifact for Linux, the build may fail on certain distributions. This issue stems from the recent upgrade from node-sass to dart-sass. The new process uses an embeddable module from dart-sass to process Sass files, which relies on a platform-specific compiler. On Linux, this module depends on glibc. However, some distributions—like Alpine Linux—use musl libc instead, which is incompatible with this module. I'm updating the DEVELOPER_GUIDE documentation to help other developers choose the appropriate distro for Docker images and avoid the pitfalls I encountered. See the issue I created in Github explaining this reasoning: #9329 Also the forum where I posted the kind of error you see when building with a Linux Distribution that is incompatible with glibc: https://forum.opensearch.org/t/docker-a-error-error-worker-exitted-unexpectedly-with-code-1-last-message-bundleid-embeddable-type-running/23214 Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Federico Silva <[email protected]> * create histogram tests Signed-off-by: Federico Silva <[email protected]> * update callback, refactor tests Signed-off-by: Federico Silva <[email protected]> * fix hooks Signed-off-by: Federico Silva <[email protected]> * add comments to code, refactor preconditions Signed-off-by: Federico Silva <[email protected]> * update hooks to beforeEach and afterEach for CI/CD Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR #9290 created/updated Signed-off-by: Federico Silva <[email protected]> * add todos Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> * remove elements.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Refactor based on #9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * skip permutation for SQL Signed-off-by: Federico Silva <[email protected]> * add todo Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR #9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR #9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (#9384) Issue Resolved: #8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Junqiu Lei <[email protected]> Co-authored-by: Luis Beltrán <[email protected]> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

…ch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>

* add max recent queries tests Signed-off-by: Federico Silva <[email protected]> * refactor callback function Signed-off-by: Federico Silva <[email protected]> * add testid-42 tests Signed-off-by: Federico Silva <[email protected]> * add comments Signed-off-by: Federico Silva <[email protected]> * add spec to package.json Signed-off-by: Federico Silva <[email protected]> * fix testconfig fn name Signed-off-by: Federico Silva <[email protected]> * fix clipboard flakiness Signed-off-by: Federico Silva <[email protected]> * refactor test preparation Signed-off-by: Federico Silva <[email protected]> * fix package.json, yarn.locl and cypress.config.ts Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR opensearch-project#9307 created/updated Signed-off-by: Federico Silva <[email protected]> * remove element.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (opensearch-project#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (opensearch-project#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Refactor based on opensearch-project#9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * comment out tests with dependencies Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (opensearch-project#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (opensearch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (opensearch-project#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (opensearch-project#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR opensearch-project#9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (opensearch-project#9384) Issue Resolved: opensearch-project#8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Anan <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

* Update actions/cache from v1 to v4 to address deprecation warning (opensearch-project#9366) * build: Update actions/cache from v1 to v4 to address deprecation warning Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9366 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * Handle invalid geospatial request in region map (opensearch-project#8759) Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Federico Silva <[email protected]> * Update DEVELOPER_GUIDE.md (opensearch-project#9368) When building the OpenSearch Dashboards artifact for Linux, the build may fail on certain distributions. This issue stems from the recent upgrade from node-sass to dart-sass. The new process uses an embeddable module from dart-sass to process Sass files, which relies on a platform-specific compiler. On Linux, this module depends on glibc. However, some distributions—like Alpine Linux—use musl libc instead, which is incompatible with this module. I'm updating the DEVELOPER_GUIDE documentation to help other developers choose the appropriate distro for Docker images and avoid the pitfalls I encountered. See the issue I created in Github explaining this reasoning: opensearch-project#9329 Also the forum where I posted the kind of error you see when building with a Linux Distribution that is incompatible with glibc: https://forum.opensearch.org/t/docker-a-error-error-worker-exitted-unexpectedly-with-code-1-last-message-bundleid-embeddable-type-running/23214 Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Federico Silva <[email protected]> * create histogram tests Signed-off-by: Federico Silva <[email protected]> * update callback, refactor tests Signed-off-by: Federico Silva <[email protected]> * fix hooks Signed-off-by: Federico Silva <[email protected]> * add comments to code, refactor preconditions Signed-off-by: Federico Silva <[email protected]> * update hooks to beforeEach and afterEach for CI/CD Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR opensearch-project#9290 created/updated Signed-off-by: Federico Silva <[email protected]> * add todos Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> * remove elements.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (opensearch-project#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (opensearch-project#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Refactor based on opensearch-project#9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (opensearch-project#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * skip permutation for SQL Signed-off-by: Federico Silva <[email protected]> * add todo Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (opensearch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (opensearch-project#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (opensearch-project#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR opensearch-project#9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (opensearch-project#9384) Issue Resolved: opensearch-project#8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Junqiu Lei <[email protected]> Co-authored-by: Luis Beltrán <[email protected]> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

ananzh added the discover_2.0-test Issues that are specific to the Discover 2.0 testing initiative label Feb 3, 2025

github-actions bot added the distinguished-contributor label Feb 3, 2025

ananzh added backport 2.x and removed distinguished-contributor labels Feb 3, 2025

opensearch-changeset-bot bot added a commit to ananzh/OpenSearch-Dashboards that referenced this pull request Feb 3, 2025

Changeset file for PR opensearch-project#9322 created/updated

9c0f70e

ananzh added the v2.20.0 label Feb 3, 2025

ananzh mentioned this pull request Feb 11, 2025

[Discover Autocomplete] TESTID-80: Autocomplete Languages - DQL, SQL, PPL - Verify grammar keywords are being generated #8964

Open

angle943 reviewed Feb 12, 2025

View reviewed changes

ananzh force-pushed the TESTID-64-1 branch 3 times, most recently from b4264e2 to e4e2119 Compare February 12, 2025 19:58

opensearch-changeset-bot bot added a commit to ananzh/OpenSearch-Dashboards that referenced this pull request Feb 12, 2025

Changeset file for PR opensearch-project#9322 created/updated

2905821

ananzh force-pushed the TESTID-64-1 branch 3 times, most recently from ec36302 to 21b9d0f Compare February 13, 2025 15:27

angle943 reviewed Feb 13, 2025

View reviewed changes

ananzh force-pushed the TESTID-64-1 branch from 21b9d0f to 5f41970 Compare February 13, 2025 19:46

Changeset file for PR opensearch-project#9322 created/updated

6a8c9c2

ananzh mentioned this pull request Feb 13, 2025

[TESTID-17] Add tests for query testing and advanced settings #9384

Merged

7 tasks

sejli approved these changes Feb 13, 2025

View reviewed changes

abbyhu2000 approved these changes Feb 14, 2025

View reviewed changes

abbyhu2000 merged commit 3c6c806 into opensearch-project:main Feb 14, 2025
70 of 71 checks passed

opensearch-trigger-bot bot added the failed backport label Feb 14, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[TESTID-64,80,UI] Add cypress test for autocomplete feature #9322

[TESTID-64,80,UI] Add cypress test for autocomplete feature #9322

ananzh commented Feb 3, 2025 •

edited

Loading

codecov bot commented Feb 3, 2025 •

edited

Loading

angle943 Feb 12, 2025

angle943 Feb 12, 2025

angle943 Feb 12, 2025

angle943 Feb 12, 2025

angle943 Feb 12, 2025

angle943 Feb 12, 2025

angle943 Feb 13, 2025

ananzh Feb 13, 2025

angle943 Feb 13, 2025

ananzh Feb 13, 2025

sejli left a comment

sejli Feb 13, 2025

ananzh Feb 14, 2025

sejli Feb 13, 2025

ananzh Feb 14, 2025

opensearch-trigger-bot bot commented Feb 14, 2025

[TESTID-64,80,UI] Add cypress test for autocomplete feature #9322

[TESTID-64,80,UI] Add cypress test for autocomplete feature #9322

Conversation

ananzh commented Feb 3, 2025 • edited Loading

Description

Issues Resolved

Screenshot

Changelog

Check List

codecov bot commented Feb 3, 2025 • edited Loading

Codecov Report

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sejli left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

opensearch-trigger-bot bot commented Feb 14, 2025

ananzh commented Feb 3, 2025 •

edited

Loading

codecov bot commented Feb 3, 2025 •

edited

Loading