-
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: updated test for DataIdentifier #32752
Conversation
WalkthroughWalkthroughThe update focuses on enhancing test reliability in the Cypress test suite by removing a redundant sleep function call in the test setup. This change aims to streamline test execution and mitigate potential flakiness associated with list loading. Additionally, interactions with Changes
Possibly related issues
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (2)
Files skipped from review as they are similar to previous changes (1)
Additional comments not posted (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/8731597355. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8731597355. |
/ci-test-limit |
2 similar comments
/ci-test-limit |
/ci-test-limit |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8732655549. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8732655107. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8732656705. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8732655549. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8732655107. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8732656705. |
@@ -25,7 +25,7 @@ | |||
"triggeredItemView": "{{{}}}", | |||
"widgetName": "List1", | |||
"requiresFlatWidgetChildren": true, | |||
"listData": "{{Api1.data}}", | |||
"listData": "[{{Api1.data}}]", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is Api.data not returning an array that we need to introduce this extra square brackets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -38,7 +38,6 @@ describe( | |||
() => { | |||
before(() => { | |||
agHelper.AddDsl("Listv2/ListV2WithNullPrimaryKey"); | |||
agHelper.Sleep(3000); //for List to load for CI flakyness |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, the RCA doesn't cover the data identifier issue and the solution for it. Could you please add some background? Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i have added more details with snapshots
/ci-test-limit |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8750959332. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8750959332. |
/ci-test-limit runId =8750959332 |
/ci-test-limit runId =8750959332 |
3 similar comments
/ci-test-limit runId =8750959332 |
/ci-test-limit runId =8750959332 |
/ci-test-limit runId =8750959332 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8751591970. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8751591447. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8751592800. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8751593637. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8751593041. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8751591970. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8751592800. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8751593637. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8751591447. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8751593041. |
DataIdentifier test fix
app/client/cypress/e2e/Regression/ClientSide/Widgets/ListV2/DataIdentifier_spec.ts
RCA:
Exeternal API was the main reason for failure
Removed unwanted wait
Summary by CodeRabbit
Summary by CodeRabbit
_dropdownControlError
to thePropertyPane
class, representing a selector for dropdown control errors.