cleanup(test): remove org booking integration coverage#29019
cleanup(test): remove org booking integration coverage#29019romitg2 merged 1 commit intocalcom:mainfrom
Conversation
|
Hey there and thank you for opening this pull request! 👋🏼 We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted. Details: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
💤 Files with no reviewable changes (7)
📝 WalkthroughWalkthroughThis change removes integration test files and test cases from the booking-audit and bookings packages. Specifically, six integration test files in the booking-audit service are deleted entirely, removing test coverage for ACCEPTED, ATTENDEE_ADDED, CANCELLED, CREATED, LOCATION_CHANGED, and NO_SHOW_UPDATED booking actions. Additionally, organization watchlist-related integration tests are removed from the spam-booking test suite. These deletions eliminate approximately 1,356 lines of test code across the affected modules. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What this does
spam-booking.integration-test.tsVerification
VITEST_MODE=integration yarn test packages/features/bookings/lib/handleNewBooking/test/spam-booking.integration-test.ts yarn type-check:ci --forceBoth commands passed locally.