fix: Add e2e test for routing forms webhook subscription#9401
fix: Add e2e test for routing forms webhook subscription#9401keithwillcode merged 9 commits intocalcom:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Someone is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
|
/claim #9038 Test Plan: End-to-End Testing for Routing Forms Webhook Subscription Objective: Test Scope: Test Environment: The application or system where the routing forms webhook subscription is implemented. 4.1.1 Manual Verification of Routing Forms Webhook Subscription:
4.1.2 Manual Testing of gotoRoutingLink Function: Steps:
4.2 Automated Test: 4.2.1 End-to-End Testing for Routing Forms Webhook Subscription:
Test Reporting: Test Completion Criteria:
|
CarinaWolli
left a comment
There was a problem hiding this comment.
Tests and type checks are failing because saveCurrentForm function got deleted
| await new Promise((resolve) => setTimeout(resolve, 500)); | ||
| } | ||
|
|
||
| async function saveCurrentForm(page: Page) { |
There was a problem hiding this comment.
Thanks for the tip! On it
|
The pipeline passes now. |
Reason for requested change has been resolved
What does this PR do?
Add e2e test for routing forms webhook subscription
I also moved the
gotoRoutingLinkfunction used for routing form testing into the@calcom/web/playwright/lib/testUtilsmoduleFixes #9038
Type of change
How should this be tested?
Please let me know if there are more asserts to add.
/claim #9038