From 45a11ab7dade52a44cb3138c6e9520afbae74e4b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 2 Feb 2026 00:59:42 +0000 Subject: [PATCH 1/4] Initial plan From 2f16e202beb058f8aa1095567b1d12b321c17594 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 2 Feb 2026 01:02:25 +0000 Subject: [PATCH 2/4] Unskip all 12 e2e tests Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com> --- e2e/allday/create-allday-event-keyboard.spec.ts | 2 +- e2e/allday/create-allday-event-mouse.spec.ts | 2 +- e2e/allday/delete-allday-event-keyboard.spec.ts | 2 +- e2e/allday/delete-allday-event-mouse.spec.ts | 2 +- e2e/allday/update-allday-event-keyboard.spec.ts | 2 +- e2e/allday/update-allday-event-mouse.spec.ts | 2 +- e2e/someday/delete-someday-event-keyboard.spec.ts | 2 +- e2e/someday/delete-someday-event-mouse.spec.ts | 2 +- e2e/someday/update-someday-event-keyboard.spec.ts | 2 +- e2e/someday/update-someday-event-mouse.spec.ts | 2 +- e2e/timed/delete-event-mouse.spec.ts | 2 +- e2e/timed/update-event-keyboard.spec.ts | 2 +- 12 files changed, 12 insertions(+), 12 deletions(-) diff --git a/e2e/allday/create-allday-event-keyboard.spec.ts b/e2e/allday/create-allday-event-keyboard.spec.ts index 6b0e8de19..74655bc06 100644 --- a/e2e/allday/create-allday-event-keyboard.spec.ts +++ b/e2e/allday/create-allday-event-keyboard.spec.ts @@ -7,7 +7,7 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; -test.skip("should create an all-day event using keyboard interaction", async ({ +test("should create an all-day event using keyboard interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/allday/create-allday-event-mouse.spec.ts b/e2e/allday/create-allday-event-mouse.spec.ts index 424519a54..469db6ac7 100644 --- a/e2e/allday/create-allday-event-mouse.spec.ts +++ b/e2e/allday/create-allday-event-mouse.spec.ts @@ -7,7 +7,7 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; -test.skip("should create an all-day event using mouse interaction", async ({ +test("should create an all-day event using mouse interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/allday/delete-allday-event-keyboard.spec.ts b/e2e/allday/delete-allday-event-keyboard.spec.ts index 9a8be017c..02d0294af 100644 --- a/e2e/allday/delete-allday-event-keyboard.spec.ts +++ b/e2e/allday/delete-allday-event-keyboard.spec.ts @@ -10,7 +10,7 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; -test.skip("should delete an all-day event using keyboard interaction", async ({ +test("should delete an all-day event using keyboard interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/allday/delete-allday-event-mouse.spec.ts b/e2e/allday/delete-allday-event-mouse.spec.ts index f6fea5d70..6ece2a259 100644 --- a/e2e/allday/delete-allday-event-mouse.spec.ts +++ b/e2e/allday/delete-allday-event-mouse.spec.ts @@ -9,7 +9,7 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; -test.skip("should delete an all-day event using mouse interaction", async ({ +test("should delete an all-day event using mouse interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/allday/update-allday-event-keyboard.spec.ts b/e2e/allday/update-allday-event-keyboard.spec.ts index ddc339fec..0a415d3d9 100644 --- a/e2e/allday/update-allday-event-keyboard.spec.ts +++ b/e2e/allday/update-allday-event-keyboard.spec.ts @@ -10,7 +10,7 @@ import { updateEventTitle, } from "../utils/event-test-utils"; -test.skip("should update an all-day event using keyboard interaction", async ({ +test("should update an all-day event using keyboard interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/allday/update-allday-event-mouse.spec.ts b/e2e/allday/update-allday-event-mouse.spec.ts index 76c55d54c..7be1e3d69 100644 --- a/e2e/allday/update-allday-event-mouse.spec.ts +++ b/e2e/allday/update-allday-event-mouse.spec.ts @@ -14,7 +14,7 @@ test.skip( "Mouse flows are desktop-only in week view.", ); -test.skip("should update an all-day event using mouse interaction", async ({ +test("should update an all-day event using mouse interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/someday/delete-someday-event-keyboard.spec.ts b/e2e/someday/delete-someday-event-keyboard.spec.ts index 7eded7039..99de9ef32 100644 --- a/e2e/someday/delete-someday-event-keyboard.spec.ts +++ b/e2e/someday/delete-someday-event-keyboard.spec.ts @@ -12,7 +12,7 @@ import { test.skip(({ isMobile }) => isMobile, "Someday sidebar is desktop-only."); -test.skip("should delete a someday event using keyboard interaction", async ({ +test("should delete a someday event using keyboard interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/someday/delete-someday-event-mouse.spec.ts b/e2e/someday/delete-someday-event-mouse.spec.ts index f7dd89846..dd3f36925 100644 --- a/e2e/someday/delete-someday-event-mouse.spec.ts +++ b/e2e/someday/delete-someday-event-mouse.spec.ts @@ -11,7 +11,7 @@ import { test.skip(({ isMobile }) => isMobile, "Someday sidebar is desktop-only."); -test.skip("should delete a someday event using mouse interaction", async ({ +test("should delete a someday event using mouse interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/someday/update-someday-event-keyboard.spec.ts b/e2e/someday/update-someday-event-keyboard.spec.ts index 68ca31df7..1ccb58f25 100644 --- a/e2e/someday/update-someday-event-keyboard.spec.ts +++ b/e2e/someday/update-someday-event-keyboard.spec.ts @@ -12,7 +12,7 @@ import { test.skip(({ isMobile }) => isMobile, "Someday sidebar is desktop-only."); -test.skip("should update a someday event using keyboard interaction", async ({ +test("should update a someday event using keyboard interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/someday/update-someday-event-mouse.spec.ts b/e2e/someday/update-someday-event-mouse.spec.ts index dcc6e370f..0e4261944 100644 --- a/e2e/someday/update-someday-event-mouse.spec.ts +++ b/e2e/someday/update-someday-event-mouse.spec.ts @@ -11,7 +11,7 @@ import { test.skip(({ isMobile }) => isMobile, "Someday sidebar is desktop-only."); -test.skip("should update a someday event using mouse interaction", async ({ +test("should update a someday event using mouse interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/timed/delete-event-mouse.spec.ts b/e2e/timed/delete-event-mouse.spec.ts index 0e7f8eaeb..18012ec10 100644 --- a/e2e/timed/delete-event-mouse.spec.ts +++ b/e2e/timed/delete-event-mouse.spec.ts @@ -10,7 +10,7 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; -test.skip("should delete a timed event using mouse interaction", async ({ +test("should delete a timed event using mouse interaction", async ({ page, }) => { await prepareCalendarPage(page); diff --git a/e2e/timed/update-event-keyboard.spec.ts b/e2e/timed/update-event-keyboard.spec.ts index 096b83cf5..d87e44a7c 100644 --- a/e2e/timed/update-event-keyboard.spec.ts +++ b/e2e/timed/update-event-keyboard.spec.ts @@ -10,7 +10,7 @@ import { updateEventTitle, } from "../utils/event-test-utils"; -test.skip("should update a timed event using keyboard interaction", async ({ +test("should update a timed event using keyboard interaction", async ({ page, }) => { await prepareCalendarPage(page); From a454227ab327c52ce9af8fdd2d3d45d1772ad1cd Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 2 Feb 2026 01:13:00 +0000 Subject: [PATCH 3/4] Add mobile skip conditions to all-day and timed tests Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com> --- e2e/allday/create-allday-event-keyboard.spec.ts | 2 ++ e2e/allday/create-allday-event-mouse.spec.ts | 5 +++++ e2e/allday/delete-allday-event-keyboard.spec.ts | 2 ++ e2e/allday/delete-allday-event-mouse.spec.ts | 5 +++++ e2e/allday/update-allday-event-keyboard.spec.ts | 2 ++ e2e/timed/delete-event-mouse.spec.ts | 5 +++++ e2e/timed/update-event-keyboard.spec.ts | 2 ++ 7 files changed, 23 insertions(+) diff --git a/e2e/allday/create-allday-event-keyboard.spec.ts b/e2e/allday/create-allday-event-keyboard.spec.ts index 74655bc06..92455b6fc 100644 --- a/e2e/allday/create-allday-event-keyboard.spec.ts +++ b/e2e/allday/create-allday-event-keyboard.spec.ts @@ -7,6 +7,8 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; +test.skip(({ isMobile }) => isMobile, "Keyboard shortcuts are desktop-only."); + test("should create an all-day event using keyboard interaction", async ({ page, }) => { diff --git a/e2e/allday/create-allday-event-mouse.spec.ts b/e2e/allday/create-allday-event-mouse.spec.ts index 469db6ac7..67acc64b6 100644 --- a/e2e/allday/create-allday-event-mouse.spec.ts +++ b/e2e/allday/create-allday-event-mouse.spec.ts @@ -7,6 +7,11 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; +test.skip( + ({ isMobile }) => isMobile, + "Mouse flows are desktop-only in week view.", +); + test("should create an all-day event using mouse interaction", async ({ page, }) => { diff --git a/e2e/allday/delete-allday-event-keyboard.spec.ts b/e2e/allday/delete-allday-event-keyboard.spec.ts index 02d0294af..4bc04ad9c 100644 --- a/e2e/allday/delete-allday-event-keyboard.spec.ts +++ b/e2e/allday/delete-allday-event-keyboard.spec.ts @@ -10,6 +10,8 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; +test.skip(({ isMobile }) => isMobile, "Keyboard shortcuts are desktop-only."); + test("should delete an all-day event using keyboard interaction", async ({ page, }) => { diff --git a/e2e/allday/delete-allday-event-mouse.spec.ts b/e2e/allday/delete-allday-event-mouse.spec.ts index 6ece2a259..dfa0ce96c 100644 --- a/e2e/allday/delete-allday-event-mouse.spec.ts +++ b/e2e/allday/delete-allday-event-mouse.spec.ts @@ -9,6 +9,11 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; +test.skip( + ({ isMobile }) => isMobile, + "Mouse flows are desktop-only in week view.", +); + test("should delete an all-day event using mouse interaction", async ({ page, }) => { diff --git a/e2e/allday/update-allday-event-keyboard.spec.ts b/e2e/allday/update-allday-event-keyboard.spec.ts index 0a415d3d9..60a555169 100644 --- a/e2e/allday/update-allday-event-keyboard.spec.ts +++ b/e2e/allday/update-allday-event-keyboard.spec.ts @@ -10,6 +10,8 @@ import { updateEventTitle, } from "../utils/event-test-utils"; +test.skip(({ isMobile }) => isMobile, "Keyboard shortcuts are desktop-only."); + test("should update an all-day event using keyboard interaction", async ({ page, }) => { diff --git a/e2e/timed/delete-event-mouse.spec.ts b/e2e/timed/delete-event-mouse.spec.ts index 18012ec10..dc8ead1cf 100644 --- a/e2e/timed/delete-event-mouse.spec.ts +++ b/e2e/timed/delete-event-mouse.spec.ts @@ -10,6 +10,11 @@ import { prepareCalendarPage, } from "../utils/event-test-utils"; +test.skip( + ({ isMobile }) => isMobile, + "Mouse flows are desktop-only in week view.", +); + test("should delete a timed event using mouse interaction", async ({ page, }) => { diff --git a/e2e/timed/update-event-keyboard.spec.ts b/e2e/timed/update-event-keyboard.spec.ts index d87e44a7c..bb5ac9a2e 100644 --- a/e2e/timed/update-event-keyboard.spec.ts +++ b/e2e/timed/update-event-keyboard.spec.ts @@ -10,6 +10,8 @@ import { updateEventTitle, } from "../utils/event-test-utils"; +test.skip(({ isMobile }) => isMobile, "Keyboard shortcuts are desktop-only."); + test("should update a timed event using keyboard interaction", async ({ page, }) => { From 22c3b1c114d4a7bfc4feba821d256f600960a82b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 2 Feb 2026 01:15:50 +0000 Subject: [PATCH 4/4] Fix all-day mouse tests and add timing fix for someday keyboard test Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com> --- e2e/allday/delete-allday-event-mouse.spec.ts | 3 ++- e2e/allday/update-allday-event-mouse.spec.ts | 3 ++- e2e/someday/update-someday-event-keyboard.spec.ts | 1 + 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/e2e/allday/delete-allday-event-mouse.spec.ts b/e2e/allday/delete-allday-event-mouse.spec.ts index dfa0ce96c..f4293fb06 100644 --- a/e2e/allday/delete-allday-event-mouse.spec.ts +++ b/e2e/allday/delete-allday-event-mouse.spec.ts @@ -6,6 +6,7 @@ import { expectAllDayEventVisible, fillTitleAndSaveWithMouse, openAllDayEventFormWithMouse, + openEventForEditingWithMouse, prepareCalendarPage, } from "../utils/event-test-utils"; @@ -24,7 +25,7 @@ test("should delete an all-day event using mouse interaction", async ({ await fillTitleAndSaveWithMouse(page, title); await expectAllDayEventVisible(page, title); - await page.locator("#allDayRow").getByRole("button", { name: title }).click(); + await openEventForEditingWithMouse(page, title); await deleteEventWithMouse(page); await expectAllDayEventMissing(page, title); diff --git a/e2e/allday/update-allday-event-mouse.spec.ts b/e2e/allday/update-allday-event-mouse.spec.ts index 7be1e3d69..6b8f70b9f 100644 --- a/e2e/allday/update-allday-event-mouse.spec.ts +++ b/e2e/allday/update-allday-event-mouse.spec.ts @@ -5,6 +5,7 @@ import { expectAllDayEventVisible, fillTitleAndSaveWithMouse, openAllDayEventFormWithMouse, + openEventForEditingWithMouse, prepareCalendarPage, updateEventTitle, } from "../utils/event-test-utils"; @@ -24,7 +25,7 @@ test("should update an all-day event using mouse interaction", async ({ await fillTitleAndSaveWithMouse(page, title); await expectAllDayEventVisible(page, title); - await page.locator("#allDayRow").getByRole("button", { name: title }).click(); + await openEventForEditingWithMouse(page, title); const updatedTitle = updateEventTitle("All-Day Event"); await fillTitleAndSaveWithMouse(page, updatedTitle); diff --git a/e2e/someday/update-someday-event-keyboard.spec.ts b/e2e/someday/update-someday-event-keyboard.spec.ts index 1ccb58f25..e08cf05b3 100644 --- a/e2e/someday/update-someday-event-keyboard.spec.ts +++ b/e2e/someday/update-someday-event-keyboard.spec.ts @@ -21,6 +21,7 @@ test("should update a someday event using keyboard interaction", async ({ await openSomedayEventFormWithKeyboard(page); await fillTitleAndSaveWithKeyboard(page, title); await expectSomedayEventVisible(page, title); + await page.waitForTimeout(1000); await openEventForEditingWithKeyboard(page, title);