Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins #725

Conversation

ohltyler
Copy link
Member

@ohltyler ohltyler commented Jun 21, 2023

Description

Using draft PR #683 as a jumpstart in this PR

Adding initial tests for augment-vis / feature-anywhere feature. The added test suite here is mostly regression testing to ensure a variety of visualization types render as expected on a dashboard, and the new feature doesn't break anything. It is intended to only be run as part of core/vanilla OSD, since we expect dashboards to behave differently when the onboarded plugins (AD/Alerting) are part of the cluster (e.g., 'Anomaly Detection'/'Alerting' options show up as options under eligible visualizations on a dashboard).

Primarily focusing on helper functions that can be consumed by the downstream plugins to bootstrap their test cluster environments. This includes:

  • ingesting sample data (can be modified to fit plugin use-cases later)
  • creating a visualization with configured metrics
  • creating a dashboard
  • creating a single function to do everything (ingest data, create dashboard, create visualizations). this way, plugins can use this and then immediately add on their actions to start testing, such as navigating to the respective plugin's option under an eligible visualization's context menu to begin creating anomaly detectors / monitors

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: Jovan Cvetkovic <[email protected]>
@ohltyler ohltyler marked this pull request as draft June 21, 2023 19:37
@ohltyler ohltyler marked this pull request as ready for review June 23, 2023 00:22
@ohltyler ohltyler changed the title Feature anywhere initial tests Feature anywhere] Add test for vanilla OSD + helper fns for plugins Jun 23, 2023
@ohltyler ohltyler changed the title Feature anywhere] Add test for vanilla OSD + helper fns for plugins [Vis Augmenter / Feature Anywhere] Add test for vanilla OSD + helper fns for plugins Jun 23, 2023
@ohltyler ohltyler changed the title [Vis Augmenter / Feature Anywhere] Add test for vanilla OSD + helper fns for plugins [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins Jun 23, 2023
@ohltyler ohltyler changed the base branch from main to feature/feature-anywhere June 26, 2023 16:12
Signed-off-by: Tyler Ohlsen <[email protected]>
@ohltyler
Copy link
Member Author

Results on my local consistently passing:

ohltyler@b0f1d86ddbb0 opensearch-dashboards-functional-test % yarn cypress:run-without-security --spec 'cypress/integration/core-opensearch-dashboards/opensearch-dashboards/apps/vis-augmenter/*'
yarn run v1.22.19
$ env TZ=America/Los_Angeles NO_COLOR=1 cypress run --headless --env SECURITY_ENABLED=false --spec 'cypress/integration/core-opensearch-dashboards/opensearch-dashboards/apps/vis-augmenter/*'
Couldn't determine Mocha version

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:        9.5.4                                                                          │
  │ Browser:        Electron 94 (headless)                                                         │
  │ Node Version:   v14.20.0 (/Users/ohltyler/.nvm/versions/node/v14.20.0/bin/node)                │
  │ Specs:          1 found (core-opensearch-dashboards/opensearch-dashboards/apps/vis-augmenter/d │
  │                 ashboard_spec.js)                                                              │
  │ Searched:       cypress/integration/core-opensearch-dashboards/opensearch-dashboards/apps/vis- │
  │                 augmenter/*                                                                    │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


────────────────────────────────────────────────────────────────────────────────────────────────────

  Running:  core-opensearch-dashboards/opensearch-dashboards/apps/vis-augmenter/            (1 of 1)
            dashboard_spec.js
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db

Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating
Couldn't determine Mocha version


  Vis augmenter - existing dashboards work as expected
    dashboard with ineligible, eligible, and vega visualizations
      ✓ All visualizations should be visible (84140ms)
      ✓ View events option does not exist for any visualization (11653ms)
      ✓ Validate non-vega visualizations are not rendered with vega under the hood (10173ms)


  3 passing (2m)


  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        3                                                                                │
  │ Passing:      3                                                                                │
  │ Failing:      0                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  0                                                                                │
  │ Video:        true                                                                             │
  │ Duration:     1 minute, 53 seconds                                                             │
  │ Spec Ran:     core-opensearch-dashboards/opensearch-dashboards/apps/vis-augmenter/dashboard_sp │
  │               ec.js                                                                            │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


  (Video)

  -  Started processing:  Compressing to 32 CRF
  -  Finished processing: /Users/ohltyler/workspace/opensearch/feature-anywhere/legac    (6 seconds)
                          y-test/opensearch-dashboards-functional-test/cypress/videos
                          /core-opensearch-dashboards/opensearch-dashboards/apps/vis-
                          augmenter/dashboard_spec.js.mp4


====================================================================================================

  (Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  core-opensearch-dashboards/opensear      01:53        3        3        -        -        - │
  │    ch-dashboards/apps/vis-augmenter/da                                                         │
  │    shboard_spec.js                                                                             │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        01:53        3        3        -        -        -

ohltyler added 2 commits June 27, 2023 15:35
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
@peterzhuamazon peterzhuamazon merged commit d65bf9f into opensearch-project:feature/feature-anywhere Jun 27, 2023
kavilla pushed a commit that referenced this pull request Jul 11, 2023
#739)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 11, 2023
#739)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 07a67d7)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 11, 2023
#739)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 07a67d7)
kavilla pushed a commit that referenced this pull request Jul 11, 2023
#739) (#746)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 07a67d7)

Co-authored-by: Tyler Ohlsen <[email protected]>
kavilla pushed a commit that referenced this pull request Jul 11, 2023
#739) (#747)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 07a67d7)

Co-authored-by: Tyler Ohlsen <[email protected]>
ohltyler added a commit to ohltyler/opensearch-dashboards-functional-test that referenced this pull request Jul 13, 2023
ohltyler added a commit to ohltyler/opensearch-dashboards-functional-test that referenced this pull request Jul 13, 2023
SuZhou-Joe pushed a commit that referenced this pull request Jul 13, 2023
* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 13, 2023
* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 39444c1)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 13, 2023
* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 39444c1)
SuZhou-Joe pushed a commit that referenced this pull request Jul 18, 2023
* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 39444c1)

Co-authored-by: Tyler Ohlsen <[email protected]>
SuZhou-Joe pushed a commit that referenced this pull request Jul 18, 2023
* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit 39444c1)

Co-authored-by: Tyler Ohlsen <[email protected]>
leanneeliatra pushed a commit to leanneeliatra/opensearch-dashboards-functional-test-fork that referenced this pull request Sep 29, 2023
opensearch-project#739)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (opensearch-project#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (opensearch-project#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: [email protected] <[email protected]>
leanneeliatra pushed a commit to leanneeliatra/opensearch-dashboards-functional-test-fork that referenced this pull request Sep 29, 2023
* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (opensearch-project#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (opensearch-project#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: [email protected] <[email protected]>
ruanyl added a commit that referenced this pull request Oct 10, 2023
…ain tenant in shortlink (#736)

* Test to ensure when a shortlink is copied, tenant is changed and short link is visited. The tenant from the link is visited.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* After link copy, allow tenant change

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Addressing comments, removing function to it's own file and removing unneeded code in before.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Spelling error rectified
Signed-off-by: leanneeliatra <[email protected]>

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Linting errors resolved

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Update cypress/utils/commands.js

Co-authored-by: Yulong Ruan <[email protected]>
Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Changed data-test-subj to getElementByTestId for consitency

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Switch tenant updated to use interceps and should() for improved test performance

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* switch tenant changed to use intercepts and should() for increaced test performance.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Adding commen to explain how the addtion of the should() helps

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Global declaration of the tenant removed. The clearing of session storage is not needed actually, cy.visit() does this for us

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Additition of 'createDashboard' method to allow programatic dashboard creation. Also improvements to the test.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* When testing we are looking for the private tenant.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* check changed to allow for both 'private' and __user__

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* linting errors resolved Signed-off-by: leanneeliatra <[email protected]>

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* merging in changes on remote

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Adding back --headless parameter.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* undoing package-lock changes.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Use library from release tag instead of git reference (#706)

The module installed from the OSD test library seems to update
or not based on caching issues.

Related proposal:
opensearch-project/opensearch-dashboards-test-library#36

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* update testSplitTables test (#731)

Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* [Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Revert "[Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739)" (#748)

This reverts commit 07a67d7.

Signed-off-by: manasvinibs <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Test sample data with multiple data source enabled on local cluster (#756)

Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* [Vis Augmenter] Add tests in OSD & AD plugin (#752)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix: CVE of tough-cookie and word-wrap (#763)

Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* feat: use "overrides" to install desired version of tough-cookie and word-wrap. (#772)

* feat: use npm v8 to install and initialize package-lock

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add Node version requirement in DEVELOPER_GUIDE

Signed-off-by: SuZhou-Joe <[email protected]>

---------

Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Improve date selection across versions of OUI (#778)

Signed-off-by: Miki <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* feat: sync cypress test from notifications-dashboards (#776) (#784)

* feat: sync cypress test from notifications-dashboards

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: rename commands

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add larger wait time

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add refresh before delete all notifications configs

Signed-off-by: SuZhou-Joe <[email protected]>

---------

Signed-off-by: SuZhou-Joe <[email protected]>
(cherry picked from commit 81f70c3)
Signed-off-by: [email protected] <[email protected]>

* [Table Visualizations] Tests cleanup (#785)

Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Bump dependency on opensearch-dashboards-test-library (#790)

Signed-off-by: Miki <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* [Table Visualizations] Test Update (#787)

* fix-test
* simplify selector

---------

Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix discover (#803)

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Change to toast message in Reports (#578)

* Change to toast message in Reports download spec

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

* Update test

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

* Fix lint

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

---------

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix discover (#807)

Signed-off-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* feat: add test cases for remote models (#813)

Signed-off-by: Lin Wang <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix workbench download text and csv schema (#814) (#817)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 46c0882)

Co-authored-by: Joshua Li <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* update trace analytics cypress tests according to observability changes (#775)

* update ftr

Signed-off-by: Derek Ho <[email protected]>

* revert file

Signed-off-by: Derek Ho <[email protected]>

* update with trace analytics accordian changes

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Bump semver from 7.3.7 to 7.5.4 (#822)

Bumps [semver](https://github.com/npm/node-semver) from 7.3.7 to 7.5.4.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v7.3.7...v7.5.4)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: [email protected] <[email protected]>

* [Dashboard] Retry visbuilder dashboard test (#823)

Seeing it pass in video replays but occassionally doesn't pass.
However, it doesn't pass with security enabled so I'd imagine it
could be related to the refreshing of the session.

Issue:
opensearch-project/OpenSearch-Dashboards#4947

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Add new tests for alerting dashboards (#832)

Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Update source element in import_vector_map_tab.spec.js (#844)

* Update source element in import_vector_map_tab.spec.js

Signed-off-by: Junqiu Lei <[email protected]>

* Update source element in import_vector_map_tab.spec.js

Signed-off-by: Junqiu Lei <[email protected]>

---------

Signed-off-by: Junqiu Lei <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Use index pattern id to find page in import_vector_map_tab.spec.js (#847)

Signed-off-by: Junqiu Lei <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Buffer `waitForLoader` before checking for icon (#857) (#859)

Some async calls occur after navigating and sometimes navigating too quickly
prevents the calls from having correct data and therefore the homeIcon is in
a bad state.

This adds the ability to buffer before checking for the homeIcon.

Defaulted to 0 ms, but for `yarn cypress:run-with-security` configures
WAIT_FOR_LOADER_BUFFER_MS to be 500 ms

Issue resolved:
opensearch-project/OpenSearch-Dashboards#5028

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit e7c7e59)

Co-authored-by: Kawika Avilla <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Overwrite removed and moved to after in unit test

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Update dashboard.spec.js

Signed-off-by: leanneeliatra <[email protected]>

* Update dashboard_sample_data_spec.js

remove unneeded change

Signed-off-by: leanneeliatra <[email protected]>

---------

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>
Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: manasvinibs <[email protected]>
Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: Miki <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: Lin Wang <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: Junqiu Lei <[email protected]>
Co-authored-by: Yulong Ruan <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
Co-authored-by: Sirazh Gabdullin <[email protected]>
Co-authored-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Manasvini B Suryanarayana <[email protected]>
Co-authored-by: Kristen Tian <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Lin Wang <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ashish Agrawal <[email protected]>
Co-authored-by: Junqiu Lei <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 10, 2023
…ain tenant in shortlink (#736)

* Test to ensure when a shortlink is copied, tenant is changed and short link is visited. The tenant from the link is visited.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* After link copy, allow tenant change

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Addressing comments, removing function to it's own file and removing unneeded code in before.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Spelling error rectified
Signed-off-by: leanneeliatra <[email protected]>

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Linting errors resolved

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Update cypress/utils/commands.js

Co-authored-by: Yulong Ruan <[email protected]>
Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Changed data-test-subj to getElementByTestId for consitency

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Switch tenant updated to use interceps and should() for improved test performance

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* switch tenant changed to use intercepts and should() for increaced test performance.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Adding commen to explain how the addtion of the should() helps

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Global declaration of the tenant removed. The clearing of session storage is not needed actually, cy.visit() does this for us

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Additition of 'createDashboard' method to allow programatic dashboard creation. Also improvements to the test.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* When testing we are looking for the private tenant.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* check changed to allow for both 'private' and __user__

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* linting errors resolved Signed-off-by: leanneeliatra <[email protected]>

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* merging in changes on remote

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Adding back --headless parameter.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* undoing package-lock changes.

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Use library from release tag instead of git reference (#706)

The module installed from the OSD test library seems to update
or not based on caching issues.

Related proposal:
opensearch-project/opensearch-dashboards-test-library#36

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* update testSplitTables test (#731)

Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* [Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Revert "[Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739)" (#748)

This reverts commit 07a67d7.

Signed-off-by: manasvinibs <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Test sample data with multiple data source enabled on local cluster (#756)

Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* [Vis Augmenter] Add tests in OSD & AD plugin (#752)

* [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Remove unnecessary test case

Signed-off-by: Tyler Ohlsen <[email protected]>

* Optimize getters

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727)

* feature anywhere initial tests

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Add test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add AD vis augmenter tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* More refactoring

Signed-off-by: Tyler Ohlsen <[email protected]>

* More tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add test for AD cleanup scenario

Signed-off-by: Tyler Ohlsen <[email protected]>

* Set up saved obj test suite

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add reminder TODO

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add tests regarding saved obj visibility

Signed-off-by: Tyler Ohlsen <[email protected]>

* Add view events tests

Signed-off-by: Tyler Ohlsen <[email protected]>

* cleanup

Signed-off-by: Tyler Ohlsen <[email protected]>

* remove import

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>

* Fix NPE

Signed-off-by: Tyler Ohlsen <[email protected]>

* Fix bug of non-empty dashboard; remove unnecessary test

Signed-off-by: Tyler Ohlsen <[email protected]>

* Simplify dashboard creation

Signed-off-by: Tyler Ohlsen <[email protected]>

* Update undefined check

Signed-off-by: Tyler Ohlsen <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix: CVE of tough-cookie and word-wrap (#763)

Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* feat: use "overrides" to install desired version of tough-cookie and word-wrap. (#772)

* feat: use npm v8 to install and initialize package-lock

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add Node version requirement in DEVELOPER_GUIDE

Signed-off-by: SuZhou-Joe <[email protected]>

---------

Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Improve date selection across versions of OUI (#778)

Signed-off-by: Miki <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* feat: sync cypress test from notifications-dashboards (#776) (#784)

* feat: sync cypress test from notifications-dashboards

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: rename commands

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add larger wait time

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add refresh before delete all notifications configs

Signed-off-by: SuZhou-Joe <[email protected]>

---------

Signed-off-by: SuZhou-Joe <[email protected]>
(cherry picked from commit 81f70c3)
Signed-off-by: [email protected] <[email protected]>

* [Table Visualizations] Tests cleanup (#785)

Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Bump dependency on opensearch-dashboards-test-library (#790)

Signed-off-by: Miki <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* [Table Visualizations] Test Update (#787)

* fix-test
* simplify selector

---------

Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix discover (#803)

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Change to toast message in Reports (#578)

* Change to toast message in Reports download spec

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

* Update test

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

* Fix lint

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

---------

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix discover (#807)

Signed-off-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* feat: add test cases for remote models (#813)

Signed-off-by: Lin Wang <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* fix workbench download text and csv schema (#814) (#817)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 46c0882)

Co-authored-by: Joshua Li <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* update trace analytics cypress tests according to observability changes (#775)

* update ftr

Signed-off-by: Derek Ho <[email protected]>

* revert file

Signed-off-by: Derek Ho <[email protected]>

* update with trace analytics accordian changes

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Bump semver from 7.3.7 to 7.5.4 (#822)

Bumps [semver](https://github.com/npm/node-semver) from 7.3.7 to 7.5.4.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v7.3.7...v7.5.4)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: [email protected] <[email protected]>

* [Dashboard] Retry visbuilder dashboard test (#823)

Seeing it pass in video replays but occassionally doesn't pass.
However, it doesn't pass with security enabled so I'd imagine it
could be related to the refreshing of the session.

Issue:
opensearch-project/OpenSearch-Dashboards#4947

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Add new tests for alerting dashboards (#832)

Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Update source element in import_vector_map_tab.spec.js (#844)

* Update source element in import_vector_map_tab.spec.js

Signed-off-by: Junqiu Lei <[email protected]>

* Update source element in import_vector_map_tab.spec.js

Signed-off-by: Junqiu Lei <[email protected]>

---------

Signed-off-by: Junqiu Lei <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Use index pattern id to find page in import_vector_map_tab.spec.js (#847)

Signed-off-by: Junqiu Lei <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Buffer `waitForLoader` before checking for icon (#857) (#859)

Some async calls occur after navigating and sometimes navigating too quickly
prevents the calls from having correct data and therefore the homeIcon is in
a bad state.

This adds the ability to buffer before checking for the homeIcon.

Defaulted to 0 ms, but for `yarn cypress:run-with-security` configures
WAIT_FOR_LOADER_BUFFER_MS to be 500 ms

Issue resolved:
opensearch-project/OpenSearch-Dashboards#5028

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit e7c7e59)

Co-authored-by: Kawika Avilla <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Overwrite removed and moved to after in unit test

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>

* Update dashboard.spec.js

Signed-off-by: leanneeliatra <[email protected]>

* Update dashboard_sample_data_spec.js

remove unneeded change

Signed-off-by: leanneeliatra <[email protected]>

---------

Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: [email protected] <[email protected]>
Signed-off-by: leanneeliatra <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Sirazh Gabdullin <[email protected]>
Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: manasvinibs <[email protected]>
Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: Miki <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: Lin Wang <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: Junqiu Lei <[email protected]>
Co-authored-by: Yulong Ruan <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
Co-authored-by: Sirazh Gabdullin <[email protected]>
Co-authored-by: Tyler Ohlsen <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Manasvini B Suryanarayana <[email protected]>
Co-authored-by: Kristen Tian <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Lin Wang <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ashish Agrawal <[email protected]>
Co-authored-by: Junqiu Lei <[email protected]>
(cherry picked from commit 21bf25f)
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.

4 participants