Skip to content

Commit

Permalink
Fixes flaky test and removes local publishing of plugin dependencies (#…
Browse files Browse the repository at this point in the history
…135)

* Fixes 'bucket level monitor can be created by extraction query' flaky test

* Removes local publishing of plugin dependencies for github cypress tests

Signed-off-by: Clay Downs <[email protected]>
  • Loading branch information
downsrob authored Oct 20, 2021
1 parent 0ba3b22 commit 5522f78
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 20 deletions.
20 changes: 0 additions & 20 deletions .github/workflows/cypress-workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,26 +24,6 @@ jobs:
with:
# TODO: Parse this from alerting plugin
java-version: 14
# dependencies: OpenSearch
- name: Checkout OpenSearch
uses: actions/checkout@v2
with:
repository: opensearch-project/OpenSearch
path: OpenSearch
ref: '1.x'
- name: Build OpenSearch
working-directory: ./OpenSearch
run: ./gradlew publishToMavenLocal
# dependencies: common-utils
- name: Checkout common-utils
uses: actions/checkout@v2
with:
repository: opensearch-project/common-utils
path: common-utils
ref: 'main'
- name: Build common-utils
working-directory: ./common-utils
run: ./gradlew publishToMavenLocal -Dopensearch.version=${{ env.OPENSEARCH_VERSION }}
- name: Checkout
uses: actions/checkout@v2
with:
Expand Down
1 change: 1 addition & 0 deletions cypress/integration/bucket_level_monitor_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,7 @@ describe('Bucket-Level Monitors', () => {
cy.get('[data-test-subj="extractionQueryCodeEditor"]').within(() => {
// If possible, a data-test-subj attribute should be added to access the code editor input directly
cy.get('.ace_text-input')
.focus()
.clear({ force: true })
.type(JSON.stringify(sampleAggregationQuery), {
force: true,
Expand Down

0 comments on commit 5522f78

Please sign in to comment.