-
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: Tree_Select_2_spec - Added toggle JS assertions before updating trigger functions #32873
Conversation
WalkthroughThe changes involve enhancing Cypress testing for the TreeSelect widget and PropertyPane class. This includes adding functionality to toggle JavaScript mode and verifying its state, as well as refining tests to assert the JavaScript toggle's state based on specific conditions. Changes
Possibly related issues
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/8797277577. |
/ci-test-limit runId=8797277577 |
1 similar comment
/ci-test-limit runId=8797277577 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8798774725. |
/ci-test-limit runId=8797277577 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8799264881. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8799264881. |
/ci-test-limit runId=8797277577 |
2 similar comments
/ci-test-limit runId=8797277577 |
/ci-test-limit runId=8797277577 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8799742043. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8799743036. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8799744307. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8799744307. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8799742043. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8799743036. |
/ci-test-limit runId=8797277577 |
/ci-test-limit runId=8797277577 |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8891684703. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8891683021. |
/ci-test-limit runId=8891293893 |
4 similar comments
/ci-test-limit runId=8891293893 |
/ci-test-limit runId=8891293893 |
/ci-test-limit runId=8891293893 |
/ci-test-limit runId=8891293893 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8892123017. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8892122669. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8892124968. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8892124096. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8892125118. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8892124096. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8892125118. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8892124968. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8892122669. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8892123017. |
@ApekshaBhosale : The PR title and description needs to be updated. The code changes LGTM. |
Description
Spec has been flaky because when JS toggle is on a property in edit mode it should remain after entering edit mode from deployed mode. In this PR, we are going to check if JS mode on for each trigger paths and if not, toggle JS mode before adding trigger functions in each test.
We were asserting external URL(google.com) for
Navigate To
functionality and navigating back from external URL usingBrowserNavigation
to app was creating flakiness. Now we will be stubbing the URL for same window.Link - Spec 36 https://www.notion.so/appsmith/Cypress-tests-analysis-4090efa5e3064a2e87f262d3c399a339
Phase 1
Summary by CodeRabbit