diff --git a/frontend/src/components/ui/button.test.tsx b/frontend/src/components/ui/button.test.tsx index 2fc7c1b..3abb228 100644 --- a/frontend/src/components/ui/button.test.tsx +++ b/frontend/src/components/ui/button.test.tsx @@ -23,9 +23,7 @@ describe("Button component", () => { render() const button = screen.getByRole("button", { name: /disabled button/i }) expect(button).toBeDisabled() - // It should have cursor-pointer but also disabled:pointer-events-none - // which prevents the cursor from changing. - // We can also add disabled:cursor-default for clarity. expect(button).toHaveClass("disabled:pointer-events-none") + expect(button).toHaveClass("disabled:cursor-default") }) }) diff --git a/frontend/src/components/ui/dropdown-menu.test.tsx b/frontend/src/components/ui/dropdown-menu.test.tsx index 347832c..c0b050a 100644 --- a/frontend/src/components/ui/dropdown-menu.test.tsx +++ b/frontend/src/components/ui/dropdown-menu.test.tsx @@ -2,7 +2,6 @@ import { render, screen } from "@testing-library/react" import { describe, expect, it } from "vitest" import { DropdownMenu, - DropdownMenuTrigger, DropdownMenuContent, DropdownMenuItem, DropdownMenuSub, @@ -41,7 +40,7 @@ describe("DropdownMenu components", () => { expect(trigger).toHaveClass("cursor-pointer") }) - it("DropdownMenuItem should not have cursor-pointer behavior when disabled", () => { + it("DropdownMenuItem should show default cursor when disabled", () => { render( @@ -52,8 +51,24 @@ describe("DropdownMenu components", () => { const item = screen.getByText("Disabled Item") // Radix sets data-disabled attribute expect(item).toHaveAttribute("data-disabled") - // If pointer-events-none is missing, we check for cursor class - // We expect the final class to have disabled:cursor-default or similar, - // or rely on pointer-events-none to block interactions. + expect(item).toHaveClass("data-[disabled]:cursor-default") + }) + + it("DropdownMenuSubTrigger should show default cursor when disabled", () => { + render( + + + + Disabled Sub Trigger + + + + + + + ) + const trigger = screen.getByText("Disabled Sub Trigger") + expect(trigger).toHaveAttribute("data-disabled") + expect(trigger).toHaveClass("data-[disabled]:cursor-default") }) })