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

[Test] [Data Explorer] Add new cypress test for testing discover related advanced settings #5269

Open
abbyhu2000 opened this issue Oct 10, 2023 · 3 comments
Assignees
Labels
data explorer Issues related to the Data Explorer project de-angular de-angularize work good first issue Good for newcomers test:functional

Comments

@abbyhu2000
Copy link
Member

Describe the bug
Add new cypress tests to verify all settings work properly for data explorer.
Screenshot 2023-10-10 at 12 36 39 PM

@abbyhu2000 abbyhu2000 added good first issue Good for newcomers test:functional de-angular de-angularize work data explorer Issues related to the Data Explorer project labels Oct 10, 2023
@ananzh ananzh removed the untriaged label Oct 11, 2023
@shawnkchan
Copy link

shawnkchan commented Oct 25, 2023

Hi, I would like to help write the test for this! May I know where I should be writing the file in and if there are any examples I could follow?

@abbyhu2000
Copy link
Member Author

Hi @shawnkchan Yes. You should write in cypress in the functional test repo: https://github.com/opensearch-project/opensearch-dashboards-functional-test

I recently had a PR that adds a list of cypress test for discover/data explorer: opensearch-project/opensearch-dashboards-functional-test#862

You could take a look at some of the example tests. This test is one of the test i added for one of the advanced settings pageOnLoad for discover: https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/862/files#diff-3eb70f4c04881cd28f1ddef02e8f912f3a6124daf6fd805674c9d4bf2796207bR189

I recommend you can create a new file in cypress/integration/core-opensearch-dashboards/opensearch-dashboards/apps/data_explorer, maybe call it discover_advanced_setting.spec.js and write your tests there. Let me know if you need more info. Thanks!

@ananzh
Copy link
Member

ananzh commented Feb 27, 2024

added this in the test meta issue #5713

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data explorer Issues related to the Data Explorer project de-angular de-angularize work good first issue Good for newcomers test:functional
Projects
None yet
Development

No branches or pull requests

3 participants