-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
fix: remove static waits from Cypress tests in ClientSide folder #38576
base: release
Are you sure you want to change the base?
fix: remove static waits from Cypress tests in ClientSide folder #38576
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Remove static waits from Cypress tests in ClientSide folder
This PR removes static wait times (cy.wait()) from Cypress tests in the ClientSide folder and replaces them with proper element visibility checks. This improves test reliability by waiting for specific conditions rather than arbitrary time periods.
Changes made:
Testing:
Link to Devin run: https://app.devin.ai/sessions/8d53afea35c34302a8ba45abbdec59d4
Automation
/ok-to-test tags="@tag.All"
🔍 Cypress test results
<- Each wait replacement uses appropriate element This is an auto-generated comment: Cypress test results -->
Caution
If you modify the content in this section, you are likely to disrupt the CI result for your PR.
<- Each wait replacement uses appropriate element end of auto-generated comment: Cypress test results -->
Caution
🔴 🔴 🔴 Some tests have failed.
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/12709636884
Commit: 09b3bd6
Cypress dashboard.
Tags: @tag.All
Spec:
The following are new failures, please fix them before merging the PR:
Fri, 10 Jan 2025 13:53:13 UTC