fix: mongo uri spec flakiness

[AmanAgarwal041]

Description

There were two things that were making this spec file act flaky:

• The delete query wasn't working as the id was being converted to ObjectId('id') but since the id was expected to be int, it was failing. So converting the default delete query generated from crud operation to { _id: {{data_table.triggeredRow._id}}} actually started passing the whole of test.
• Second thing was, since the spec was updating some records and verifying, running it multiple times was causing the failures in the next runs, Hence instead of using the default data table from the ted suite, the data is being dumped to a new collection that is dynamically generated in the spec using the new fixture that is added in the pr. Hence, after this change the spec started passing in every multiple runs.

Fixes #35148
or
Fixes Issue URL

Warning

If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation

/ok-to-test tags="@tag.Datasource"

🔍 Cypress test results

Tip

🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10891454490
Commit: 3aef2d2
Cypress dashboard.
Tags: @tag.Datasource
Spec:

---

Mon, 16 Sep 2024 21:03:05 UTC

Communication

Should the DevRel and Marketing teams inform users about this change?

• Yes
• No

Summary by CodeRabbit

Summary by CodeRabbit

• New Features

  • Added a comprehensive dataset for MongoDB-related merchandise, improving product management and display features.

• Bug Fixes

  • Updated test case logic to focus on deletion operations and enhance validation processes.
  • Removed redundant assertions for table data verification to improve test clarity.

• Chores

  • Streamlined validation of loaded table data for clearer test output, consolidating multiple assertions into a single verification method.

[github-actions]

Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10885288503.
Cypress dashboard: Click here!
The following are new failures, please fix them before merging the PR:
To know the list of identified flaky tests - Refer here

***** Repeat Run Summary ***** Total Tests with repeat: 24 Total Passed: 24 Total Failed: 0 Total Skipped: 0 *****************************

[AmanAgarwal041]

/ci-test-limit-count run_count=20

[github-actions]

Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/10889253849.
Cypress dashboard
PR: #36249.

[github-actions]

Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10889253849.
Cypress dashboard: Click here!
The following are new failures, please fix them before merging the PR:
To know the list of identified flaky tests - Refer here

***** Repeat Run Summary ***** Total Tests with repeat: 120 Total Passed: 120 Total Failed: 0 Total Skipped: 0 *****************************