Skip to content

fix(sa): increase detectorsSearch wait timeout - #2099

Open
yashgupte21 wants to merge 1 commit into
opensearch-project:3.1from
yashgupte21:fix-sa-3.1
Open

fix(sa): increase detectorsSearch wait timeout#2099
yashgupte21 wants to merge 1 commit into
opensearch-project:3.1from
yashgupte21:fix-sa-3.1

Conversation

@yashgupte21

Copy link
Copy Markdown
Contributor

Description

Increase detectorsSearch wait timeout from 300s to 500s in the SA detector tests. The 300s timeout is insufficient when running against domains with higher network latency, causing flaky failures in the beforeEach hooks.

Issues Resolved

Resolves flaky cy.wait('@detectorsSearch') timeout in 1_detectors.spec.js

(Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped  
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  plugins/security-analytics-dashboar      06:03       14       12        -        2        - │
  │    ds-plugin/1_detectors.spec.js                                                               │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        06:03       14       12        -        2        -  

✨  Done in 394.84s.

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: YashPradeep Gupte <yashprg@amazon.com>
@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

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.

1 participant