-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: fix migration spec #32920
test: fix migration spec #32920
Conversation
WalkthroughWalkthroughThis update focuses on refining the Cypress test suite for a client-side application by enhancing validations, removing inefficiencies, and improving test reliability related to widget operations and page interactions. The adjustments include reorganized validation steps, updated checks for credit limit computations, and the removal of unnecessary code and waits. Changes
Possibly related issues
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 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 as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
/ci-test-limit |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8816438273. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8816438273. |
/ci-test-limit runId=8816438273 |
1 similar comment
/ci-test-limit runId=8816438273 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818311465. |
/ci-test-limit runId=8816438273 |
5 similar comments
/ci-test-limit runId=8816438273 |
/ci-test-limit runId=8816438273 |
/ci-test-limit runId=8816438273 |
/ci-test-limit runId=8816438273 |
/ci-test-limit runId=8816438273 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818320037. |
/ci-test-limit runId=8816438273 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818321759. |
/ci-test-limit runId=8816438273 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818323555. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818324489. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818326411. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818327854. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818329561. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818330436. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8818327513. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818311465. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818326411. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818320037. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818321759. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818329561. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818323555. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818324489. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818330436. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818327854. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8818327513. |
RCA:
Migration spec was having duplicate code and click on image link and navigation back was causing issues
Solution:
1.Removed redundant code
2.stub image click event and validate rather than navigating to actual image url which is external.
Summary by CodeRabbit
window.open
to ensure new tabs open correctly.