Skip to content
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

[Task]: Fix flakiness of cypress/e2e/Regression/ServerSide/ApiTests/API_MultiPart_Spec.ts #35137

Closed
NilanshBansal opened this issue Jul 24, 2024 · 3 comments · Fixed by #35927
Assignees
Labels
cypress-flaky-fix This label is auto-added when a PR which only has Cypress fixes are merged to release Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod Stability Pod For all issues/tasks to be prioritized under Stability pod Task A simple Todo

Comments

@NilanshBansal
Copy link
Contributor

SubTasks

Failing Tests:
7. Checks MultiPart form data for a File Type upload + Bug 12476, 8. Checks MultiPart form data for a Array Type upload results in API error

This test case relies on cloudinary which is currently throwing an error. This is being caused due to being dependent on external API.

Failure Runs: https://internal.appsmith.com/app/cypress-dashboard/run-details-65890b3c81d7400d08fa9ee5?branch=master&workflowId=8718186679&attempt=3&selectiontype=test&testsstatus=failed&specsstatus=fail

@NilanshBansal NilanshBansal added the Task A simple Todo label Jul 24, 2024
@github-actions github-actions bot added the potential-duplicate This label marks issues that are potential duplicates of already open issues label Jul 24, 2024

This comment was marked as resolved.

@NilanshBansal
Copy link
Contributor Author

@NilanshBansal NilanshBansal added Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. cypress-flaky-fix This label is auto-added when a PR which only has Cypress fixes are merged to release labels Jul 25, 2024
@github-actions github-actions bot added Integrations Product Issues related to a specific integration Stability Pod For all issues/tasks to be prioritized under Stability pod labels Jul 25, 2024
@sneha122
Copy link
Contributor

Adding estimate as 2 as we have reliance external Cloudinary API, in order to get rid of this dependency, we may have to add a new file upload API functionality in TED server and use API from there. Since we have never actively added any datasources or APIs in TED server, we would need to get more context on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cypress-flaky-fix This label is auto-added when a PR which only has Cypress fixes are merged to release Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod Stability Pod For all issues/tasks to be prioritized under Stability pod Task A simple Todo
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants