appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.
https://www.appsmith.com
Apache License 2.0
34.35k stars 3.72k forks source link

[Task]: Add Cypress Tests to Twilio SaaS Integration #15134

Open felixsuarez0727 opened 2 years ago

felixsuarez0727 commented 2 years ago

This is a follow-up on #13756

Test Case Issue Status
Entity Explorer: Test cases with respect to action that can be done on Entity explorer #1869 In Progress
Query Pane: Test ideas for Query Pane #1868 Pending review
Active Datasource: Tests on active datasource behaviour #1867 Pending review
Connections: Establish connection with Twilio #1866 Pending review
Query Pane: Ensure user is able to save the queries #299 Already Covered
Query Pane: Ensure user is able to add special characters into the name from Pane #298 Already Covered
Query Pane: Ensure user is able to add longer name #297 Already Covered
Query Pane: Ensure user is able change name from the pane #296 Already Covered
Query Pane: Ensure use method from the drop down #295 Already Covered
Query Pane: Ensure Long body form is added #294 Already Covered
Query Pane: Ensure the user is displayed appropriate error messages on wrong field data #293 Already Covered
Query Pane: Ensure mandatory Field are added #292 Already Covered
Active Datasource: Ensure if a Query is inactive datasource must be deleted #291 Already Covered
Active Datasource: Clicking on Delete from Action icon the action must verify if Query is assocuated with the Datasource #290 Already Covered
Connections: Ensure clicking on the back button user should be navigated to active Datasources page #289 Already Covered
Connections: Ensure on delete if the DS is associated with a query the DS must show an error and unable to delete #288 Already Covered
Connections: Ensure clicking on Delete the DS gets deleted #287 Already Covered
Connections: Ensure user is disaplyed with Auth type and user is able to select Basic Auth #286 Already Covered
felixsuarez0727 commented 2 years ago

Hi,

This task is completed, this is the PR:

https://github.com/appsmithorg/appsmith/pull/15636

Currently waiting for ui-test-results:

image.png

Thanks

felixsuarez0727 commented 2 years ago

ui-test-result is still running ...

image.png

felixsuarez0727 commented 2 years ago

Fixed the flaky tests for this, the tests seem to be stuck for sha=11bfdc7

felixsuarez0727 commented 2 years ago

image.png

felixsuarez0727 commented 2 years ago

Only one test fails when running the tests in PR

testfail.png

But locally all issues are successful

image.png

felixsuarez0727 commented 2 years ago

Two observations here:

This is pushing this issue to one more sprint.

felixsuarez0727 commented 2 years ago

image.png

Perf is failing.

felixsuarez0727 commented 2 years ago

The issue is blocked by a real error that needs to be fixed.