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

[MD] func testing for data source management listing #314

Conversation

mpabba3003
Copy link
Contributor

@mpabba3003 mpabba3003 commented Sep 30, 2022

[MD] func testing for data source management listing

Description

Cypress functional tests for datasource management

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.

ohltyler and others added 4 commits September 27, 2022 16:31
Signed-off-by: Tyler Ohlsen <ohltyler@amazon.com>
…ct#291)

Enables access to easily see the import custom maps.
Also added more query strings to easily set and build a link to
the test results page to be easily copied and shared.

Issue:
n/a

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
Signed-off-by: Tyler Ohlsen <ohltyler@amazon.com>

Signed-off-by: Tyler Ohlsen <ohltyler@amazon.com>
@mpabba3003 mpabba3003 requested a review from a team as a code owner September 30, 2022 15:39
@mpabba3003 mpabba3003 changed the title [MD] func testing for data source management listing & es lint on few existing files [MD] func testing for data source management listing Sep 30, 2022
@seraphjiang
Copy link
Member

could we create PR against main?

@tianleh
Copy link
Member

tianleh commented Oct 3, 2022

Please take a look at #317 you may need to move the location of the test file and also add a feature flag. See this example https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/main/cypress/integration/plugins/security/audit_log_spec.js#L11

@tianleh
Copy link
Member

tianleh commented Oct 3, 2022

Please share the Cypress video for this test

AM1TB and others added 7 commits October 5, 2022 15:01
```experimentalNetworkStubbing``` option was removed and made the default behavior from version 6.0.0.
Ref: https://docs.cypress.io/guides/references/changelog#6-0-0

Issue resolved:
opensearch-project#309

Signed-off-by: am1tb <amit.banerjee.jsr@gmail.com>
…h-project#317)

* Add instructions for tests for experimental features

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add more

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add todo

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* enhance readability

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update format

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* capitalize

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add example

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add cypress configuration example

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
…roject#319)

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>
* Adds basic visBuilder test

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* Adds env variable

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* updates cypress bundle snapshot workflow

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>
* add Emeritus Maintainers

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update package-lock.json

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add founder Aviv

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update node

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* revert

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: mpabba3003 <amazonmanideep@gmail.com>
Signed-off-by: mpabba3003 <amazonmanideep@gmail.com>
@mpabba3003
Copy link
Contributor Author

Closing this PR. I will raise a new PR against Main branch.

@mpabba3003 mpabba3003 closed this Oct 14, 2022
@mpabba3003
Copy link
Contributor Author

@seraphjiang @tianleh A new PR #325 has been created which includes resolution for all comments on this PR

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.

8 participants