Fix/wipe vital app revision (#2983)

* Fixes for both apps regarding event update/delete

* Added wipe my cal test

* Added eslint ignore

* Fix test

* Merge both test into a single test

* remove duplicated line

* Using prisma from calcom pkg and removing optional chaining from current invocations

Co-authored-by: Peer Richelsen <peeroke@gmail.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

apps/web/pages/api/book/request-reschedule.ts

@@ -61,7 +61,7 @@ const handler = async (
     if (session?.user?.id) {
       userOwner = await findUserDataByUserId(session?.user.id);
     } else {
-      return res.status(501);
+      return res.status(501).end();
     }
 
     const bookingToReschedule = await prisma.booking.findFirst({
@@ -235,10 +235,10 @@ function validate(
         if (error instanceof ZodError && error?.name === "ZodError") {
           return res.status(400).json(error?.issues);
         }
-        return res.status(402);
+        return res.status(402).end();
       }
     } else {
-      return res.status(405);
+      return res.status(405).end();
     }
     await handler(req, res);
   };

apps/web/playwright/fixtures/bookings.ts

@@ -21,9 +21,11 @@ export const createBookingsFixture = (page: Page) => {
       userId: number,
       username: string | null,
       eventTypeId = -1,
-      { rescheduled = false, paid = false, status = "ACCEPTED" }: Partial<Prisma.BookingCreateInput> = {}
+      { rescheduled = false, paid = false, status = "ACCEPTED" }: Partial<Prisma.BookingCreateInput> = {},
+      startDateParam?: Date,
+      endDateParam?: Date
     ) => {
-      const startDate = dayjs().add(1, "day").toDate();
+      const startDate = startDateParam || dayjs().add(1, "day").toDate();
       const seed = `${username}:${dayjs(startDate).utc().format()}:${new Date().getTime()}`;
       const uid = translator.fromUUID(uuidv5(seed, uuidv5.URL));
       const booking = await prisma.booking.create({
@@ -31,7 +33,7 @@ export const createBookingsFixture = (page: Page) => {
           uid: uid,
           title: "30min",
           startTime: startDate,
-          endTime: dayjs().add(1, "day").add(30, "minutes").toDate(),
+          endTime: endDateParam || dayjs().add(1, "day").add(30, "minutes").toDate(),
           user: {
             connect: {
               id: userId,

apps/web/playwright/lib/appstore/wipe-my-cal.test.ts

@@ -0,0 +1,69 @@
+import { expect } from "@playwright/test";
+import dayjs from "dayjs";
+
+import prisma from "@calcom/prisma";
+
+import { test } from "../../lib/fixtures";
+
+test.describe.configure({ mode: "parallel" });
+
+test.describe("Wipe my Cal App Test", () => {
+  test("Browse upcoming bookings and validate button shows and triggering wipe my cal button", async ({
+    page,
+    users,
+    bookings,
+  }) => {
+    const pro = await users.create();
+    const [eventType] = pro.eventTypes;
+    await prisma.credential.create({
+      data: {
+        key: {},
+        type: "wipemycal_other",
+        userId: pro.id,
+        appId: "wipe-my-cal",
+      },
+    });
+    await bookings.create(
+      pro.id,
+      pro.username,
+      eventType.id,
+      {},
+      dayjs().endOf("day").subtract(29, "minutes").toDate(),
+      dayjs().endOf("day").toDate()
+    );
+    await prisma.credential.create({
+      data: {
+        key: {},
+        type: "wipemycal_other",
+        userId: pro.id,
+        appId: "wipe-my-cal",
+      },
+    });
+
+    await bookings.create(pro.id, pro.username, eventType.id, {});
+    await bookings.create(pro.id, pro.username, eventType.id, {});
+    await pro.login();
+    await page.goto("/bookings/upcoming");
+    await expect(page.locator("data-testid=wipe-today-button")).toBeVisible();
+
+    const totalUserBookings = await prisma.booking.findMany({
+      where: {
+        userId: pro.id,
+      },
+    });
+    expect(totalUserBookings?.length).toBe(3);
+    await page.locator("data-testid=wipe-today-button").click();
+    await page.locator("data-testid=send_request").click();
+    // eslint-disable-next-line playwright/no-wait-for-timeout
+    await page.waitForTimeout(250);
+    const totalUserBookingsCancelled = await prisma.booking.findMany({
+      where: {
+        userId: pro.id,
+        status: "CANCELLED",
+      },
+    });
+
+    await expect(totalUserBookingsCancelled?.length).toBe(1);
+    await users.deleteAll();
+  });
+});

packages/app-store/vital/lib/reschedule.ts

@@ -139,19 +139,6 @@ const Reschedule = async (bookingUid: string, cancellationReason: string) => {
         logger.error(error.message);
       }
     }
-    // Creating cancelled event as placeholders in calendars, remove when virtual calendar handles it
-    try {
-      const eventManager = new EventManager({
-        credentials: userOwner.credentials,
-        destinationCalendar: userOwner.destinationCalendar,
-      });
-      builder.calendarEvent.title = `Cancelled: ${builder.calendarEvent.title}`;
-      await eventManager.updateAndSetCancelledPlaceholder(builder.calendarEvent, bookingToReschedule);
-    } catch (error) {
-      if (error instanceof Error) {
-        logger.error(error.message);
-      }
-    }
 
     // Send emails
     try {

packages/app-store/wipemycalother/api/wipe.ts

@@ -21,7 +21,7 @@ const handler = async (req: NextApiRequest, res: NextApiResponse) => {
   if (!req.session?.user?.id) {
     return res.status(401).json({ message: "You must be logged in to do this" });
   }
-  let result = false;
+
   try {
     const { initialDate, endDate } = req.body;
 

packages/app-store/wipemycalother/components/confirmDialog.tsx

@@ -29,7 +29,7 @@ const wipeMyCalAction = async (props: IWipeMyCalAction) => {
   };
   try {
     const endpoint = "/api/integrations/wipemycalother/wipe";
-    return fetch(`${process.env.NEXT_PUBLIC_APP_BASE_URL}` + endpoint, {
+    return fetch(`${process.env.NEXT_PUBLIC_WEBAPP_URL}` + endpoint, {
       method: "POST",
       headers: {
         "Content-Type": "application/json",
@@ -81,7 +81,7 @@ export const ConfirmDialog = (props: IConfirmDialogWipe) => {
 
   return (
     <Dialog open={isOpenDialog} onOpenChange={setIsOpenDialog}>
-      <DialogContent>
+      <DialogContent onOpenAutoFocus={(e) => e.preventDefault()}>
         <div className="flex flex-row space-x-3">
           <div className="flex h-10 w-10 flex-shrink-0 justify-center rounded-full bg-[#FAFAFA]">
             <ClockIcon className="m-auto h-6 w-6"></ClockIcon>
@@ -99,9 +99,10 @@ export const ConfirmDialog = (props: IConfirmDialogWipe) => {
         </div>
 
         <DialogFooter>
-          <DialogClose>
-            <Button color="secondary">{t("cancel")}</Button>
-          </DialogClose>
+          <Button color="secondary" onClick={() => setIsOpenDialog(false)}>
+            {t("cancel")}
+          </Button>
+
           <Button
             data-testid="send_request"
             disabled={isLoading}

packages/app-store/wipemycalother/components/wipeMyCalActionButton.tsx

@@ -32,7 +32,9 @@ const WipeMyCalActionButton = (props: IWipeMyCalActionButtonProps) => {
       {data && isSuccess && !isLoading && credentialId && (
         <>
           <ConfirmDialog trpc={trpc} isOpenDialog={openDialog} setIsOpenDialog={setOpenDialog} />
-          <Button onClick={() => setOpenDialog(true)}>Wipe Today</Button>
+          <Button onClick={() => setOpenDialog(true)} data-testid="wipe-today-button">
+            Wipe Today
+          </Button>
         </>
       )}
     </div>

packages/app-store/wipemycalother/lib/reschedule.ts

@@ -78,7 +78,7 @@ const Reschedule = async (bookingUid: string, cancellationReason: string) => {
       },
     });
     const [mainAttendee] = bookingToReschedule.attendees;
-    // @NOTE: Should we assume attendees language?
+
     const tAttendees = await getTranslation(mainAttendee.locale ?? "en", "common");
     const usersToPeopleType = (
       users: PersonAttendeeCommonFields[],
@@ -139,19 +139,6 @@ const Reschedule = async (bookingUid: string, cancellationReason: string) => {
         logger.error(error.message);
       }
     }
-    // Creating cancelled event as placeholders in calendars, remove when virtual calendar handles it
-    try {
-      const eventManager = new EventManager({
-        credentials: userOwner.credentials,
-        destinationCalendar: userOwner.destinationCalendar,
-      });
-      builder.calendarEvent.title = `Cancelled: ${builder.calendarEvent.title}`;
-      await eventManager.updateAndSetCancelledPlaceholder(builder.calendarEvent, bookingToReschedule);
-    } catch (error) {
-      if (error instanceof Error) {
-        logger.error(error.message);
-      }
-    }
 
     // Send emails
     try {