From d8524707f01df9d179dd587d3e4227520f60ee55 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Fri, 22 May 2026 07:08:35 +0000 Subject: [PATCH 1/2] =?UTF-8?q?=F0=9F=A7=B9=20fix:=20remove=20any=20types?= =?UTF-8?q?=20and=20eslint-disable=20in=20LayoutEditor.test.tsx?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com> --- .../__tests__/LayoutEditor.test.tsx | 32 +++++++------------ 1 file changed, 12 insertions(+), 20 deletions(-) diff --git a/src/components/__tests__/LayoutEditor.test.tsx b/src/components/__tests__/LayoutEditor.test.tsx index fb53039..a426713 100644 --- a/src/components/__tests__/LayoutEditor.test.tsx +++ b/src/components/__tests__/LayoutEditor.test.tsx @@ -22,8 +22,7 @@ vi.mock("@dnd-kit/core", async (importOriginal) => { useSensor: vi.fn(() => ({})), useDroppable: ({ id }: { id: string }) => ({ setNodeRef: vi.fn(), - // eslint-disable-next-line @typescript-eslint/no-explicit-any - isOver: (window as any).mockIsOverId === id, + isOver: (window as unknown as { mockIsOverId?: string }).mockIsOverId === id, }), PointerSensor: vi.fn(), KeyboardSensor: vi.fn(), @@ -58,16 +57,16 @@ const defaultLayout: CardLayout = { }; describe("LayoutEditor", () => { + type MockWindow = typeof window & { triggerDragEnd?: (event: unknown) => void; mockIsOverId?: string; }; + const mockWindow = window as unknown as MockWindow; let mockOnLayoutChange: ReturnType; let mockOnToggleVisibility: ReturnType; beforeEach(() => { mockOnLayoutChange = vi.fn(); mockOnToggleVisibility = vi.fn(); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - (window as any).triggerDragEnd = undefined; - // eslint-disable-next-line @typescript-eslint/no-explicit-any - (window as any).mockIsOverId = undefined; + mockWindow.triggerDragEnd = undefined; + mockWindow.mockIsOverId = undefined; }); it("renders blocks in their respective columns", () => { @@ -119,8 +118,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; expect(triggerDragEnd).toBeDefined(); // Drag 'avatar' to 'right' column @@ -149,8 +147,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; // Drag 'avatar' over 'topLanguages' triggerDragEnd({ @@ -178,8 +175,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; triggerDragEnd({ active: { id: "avatar" }, over: null, @@ -200,8 +196,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; triggerDragEnd({ active: { id: "avatar" }, over: { id: "avatar" }, @@ -222,8 +217,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; triggerDragEnd({ active: { id: "avatar" }, over: { id: "non-existent-block" }, @@ -250,8 +244,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; // Drag 'avatar' over 'stats' (downwards) triggerDragEnd({ @@ -278,8 +271,7 @@ describe("LayoutEditor", () => { const dndContext = screen.getByTestId("dnd-context"); fireEvent.click(dndContext); - // eslint-disable-next-line @typescript-eslint/no-explicit-any - const triggerDragEnd = (window as any).triggerDragEnd; + const triggerDragEnd = mockWindow.triggerDragEnd; // Drag 'avatar' to empty column 'right' triggerDragEnd({ From e0963f0c4af8dbae4d2d748819c29c846291dda7 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Fri, 22 May 2026 07:14:28 +0000 Subject: [PATCH 2/2] =?UTF-8?q?=F0=9F=A7=B9=20fix:=20remove=20any=20types?= =?UTF-8?q?=20and=20eslint-disable=20in=20LayoutEditor.test.tsx?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com> --- src/components/__tests__/LayoutEditor.test.tsx | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/components/__tests__/LayoutEditor.test.tsx b/src/components/__tests__/LayoutEditor.test.tsx index a426713..df7ff60 100644 --- a/src/components/__tests__/LayoutEditor.test.tsx +++ b/src/components/__tests__/LayoutEditor.test.tsx @@ -122,7 +122,7 @@ describe("LayoutEditor", () => { expect(triggerDragEnd).toBeDefined(); // Drag 'avatar' to 'right' column - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: { id: "right" }, }); @@ -150,7 +150,7 @@ describe("LayoutEditor", () => { const triggerDragEnd = mockWindow.triggerDragEnd; // Drag 'avatar' over 'topLanguages' - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: { id: "topLanguages" }, }); @@ -176,7 +176,7 @@ describe("LayoutEditor", () => { fireEvent.click(dndContext); const triggerDragEnd = mockWindow.triggerDragEnd; - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: null, }); @@ -197,7 +197,7 @@ describe("LayoutEditor", () => { fireEvent.click(dndContext); const triggerDragEnd = mockWindow.triggerDragEnd; - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: { id: "avatar" }, }); @@ -218,7 +218,7 @@ describe("LayoutEditor", () => { fireEvent.click(dndContext); const triggerDragEnd = mockWindow.triggerDragEnd; - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: { id: "non-existent-block" }, }); @@ -247,7 +247,7 @@ describe("LayoutEditor", () => { const triggerDragEnd = mockWindow.triggerDragEnd; // Drag 'avatar' over 'stats' (downwards) - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: { id: "stats" }, }); @@ -274,7 +274,7 @@ describe("LayoutEditor", () => { const triggerDragEnd = mockWindow.triggerDragEnd; // Drag 'avatar' to empty column 'right' - triggerDragEnd({ + triggerDragEnd!({ active: { id: "avatar" }, over: { id: "right" }, });