test: use present voice in test descriptions

Author: danielwaltz

test/unit/core.test.ts

@@ -235,7 +235,7 @@ describe("Core", () => {
         adhesive.cleanup();
       });
 
-      it("should handle updating frozen options from disabled state", () => {
+      it("handles updating frozen options from disabled state", () => {
         const adhesive = createInitializedAdhesive();
 
         adhesive.updateOptions({ enabled: false });
@@ -263,7 +263,7 @@ describe("Core", () => {
         adhesive.cleanup();
       });
 
-      it("should handle creating instance with enabled:false then updating options", () => {
+      it("handles creating instance with enabled:false then updating options", () => {
         // Create an instance that starts disabled (uses frozen options)
         const adhesive = Adhesive.create({
           targetEl: targetElement,
@@ -291,7 +291,7 @@ describe("Core", () => {
         adhesive.cleanup();
       });
 
-      it("should preserve all default options when updating frozen options", () => {
+      it("preserves all default options when updating frozen options", () => {
         // Create an instance that starts disabled (uses frozen options with defaults)
         const adhesive = Adhesive.create({
           targetEl: targetElement,
@@ -320,7 +320,7 @@ describe("Core", () => {
         adhesive.cleanup();
       });
 
-      it("should preserve all option properties when unfreezing frozen options", () => {
+      it("preserves all option properties when unfreezing frozen options", () => {
         // This test specifically verifies that no options are lost during the unfreezing process
         const adhesive = Adhesive.create({
           targetEl: targetElement,
@@ -354,7 +354,7 @@ describe("Core", () => {
         adhesive.cleanup();
       });
 
-      it("should handle multiple consecutive updates from disabled state", () => {
+      it("handle multiple consecutive updates from disabled state", () => {
         const adhesive = createInitializedAdhesive();
 
         // Disable first

test/unit/react.test.tsx

@@ -16,17 +16,14 @@ import {
   TEST_Z_INDEXES,
 } from "../utils/shared-test-helpers.js";
 
-// Create the mock Adhesive instance
 const mockAdhesiveInstance = createMockAdhesive();
 
-// Mock the Adhesive class
 vi.mock("@adhesivejs/core", () => ({
   Adhesive: {
     create: vi.fn(() => mockAdhesiveInstance),
   },
 }));
 
-// Enhanced test component using useAdhesive hook
 function TestHookComponent() {
   const targetRef = useRef<HTMLDivElement>(null);
   const boundingRef = useRef<HTMLDivElement>(null);
@@ -97,7 +94,7 @@ describe("React Integration", () => {
 
   describe("useAdhesive Hook", () => {
     describe("initialization and rendering", () => {
-      it("should render without errors", () => {
+      it("renders without errors", () => {
         renderTestComponent();
         const { stickyElement, toggleEnabledButton } = getTestElements();
 
@@ -106,14 +103,14 @@ describe("React Integration", () => {
         expect(screen.getByText("Toggle (enabled)")).toBeInTheDocument();
       });
 
-      it("should create Adhesive instance on mount", async () => {
+      it("creates Adhesive instance on mount", async () => {
         renderTestComponent();
 
         const { Adhesive } = await import("@adhesivejs/core");
         expect(Adhesive.create).toHaveBeenCalledTimes(1);
       });
 
-      it("should not create multiple instances on re-renders", async () => {
+      it("does not create multiple instances on re-renders", async () => {
         const { rerender } = renderTestComponent();
 
         rerender(<TestHookComponent />);
@@ -125,7 +122,7 @@ describe("React Integration", () => {
     });
 
     describe("state management", () => {
-      it("should handle enabled toggle correctly", async () => {
+      it("handles enabled toggle correctly", async () => {
         const user = userEvent.setup();
         renderTestComponent();
         const { toggleEnabledButton } = getTestElements();
@@ -142,7 +139,7 @@ describe("React Integration", () => {
         expect(screen.getByText("Toggle (enabled)")).toBeInTheDocument();
       });
 
-      it("should handle position changes correctly", async () => {
+      it("handles position changes correctly", async () => {
         const user = userEvent.setup();
         renderTestComponent();
         const { togglePositionButton } = getTestElements();
@@ -159,7 +156,7 @@ describe("React Integration", () => {
         expect(screen.getByText("Position: top")).toBeInTheDocument();
       });
 
-      it("should handle offset changes correctly", async () => {
+      it("handles offset changes correctly", async () => {
         const user = userEvent.setup();
         renderTestComponent();
         const { toggleOffsetButton } = getTestElements();
@@ -176,7 +173,7 @@ describe("React Integration", () => {
         expect(screen.getByText("Offset: 10")).toBeInTheDocument();
       });
 
-      it("should call updateOptions when options change", async () => {
+      it("calls updateOptions when options change", async () => {
         const user = userEvent.setup();
         renderTestComponent();
         const {
@@ -215,7 +212,7 @@ describe("React Integration", () => {
     });
 
     describe("cleanup and lifecycle", () => {
-      it("should cleanup on unmount without errors", () => {
+      it("cleans up on unmount without errors", () => {
         const { unmount } = renderTestComponent();
         const { stickyElement } = getTestElements();
 
@@ -224,7 +221,7 @@ describe("React Integration", () => {
         expect(() => unmount()).not.toThrow();
       });
 
-      it("should handle disabled state properly", () => {
+      it("handles disabled state properly", () => {
         function DisabledComponent() {
           const targetRef = useRef<HTMLDivElement>(null);
           const boundingRef = useRef<HTMLDivElement>(null);
@@ -248,21 +245,21 @@ describe("React Integration", () => {
 
   describe("AdhesiveContainer Component", () => {
     describe("basic rendering", () => {
-      it("should render children correctly", () => {
+      it("renders children correctly", () => {
         renderContainer();
 
         expect(screen.getByTestId("container-child")).toBeInTheDocument();
         expect(screen.getByText("Container Content")).toBeInTheDocument();
       });
 
-      it("should handle empty children", () => {
+      it("handles empty children", () => {
         render(<AdhesiveContainer />);
         // Should not throw errors
       });
     });
 
     describe("styling and customization", () => {
-      it("should apply custom class names", async () => {
+      it("applies custom class names", async () => {
         const customProps = {
           className: "custom-class",
           outerClassName: CUSTOM_CLASS_NAMES.outerClassName,
@@ -292,7 +289,7 @@ describe("React Integration", () => {
         );
       });
 
-      it("should handle z-index styling", () => {
+      it("handles z-index styling", () => {
         renderContainer({ zIndex: TEST_Z_INDEXES[3] });
 
         expect(screen.getByTestId("container-child")).toBeInTheDocument();
@@ -301,7 +298,7 @@ describe("React Integration", () => {
 
     describe("configuration options", () => {
       configurationTestCases.forEach(({ name, props }) => {
-        it(`should handle ${name} correctly`, () => {
+        it(`handles ${name} correctly`, () => {
           const { rerender } = renderContainer(props);
 
           expect(screen.getByTestId("container-child")).toBeInTheDocument();
@@ -317,7 +314,7 @@ describe("React Integration", () => {
         });
       });
 
-      it("should handle bounding element with string selector", () => {
+      it("handles bounding element with string selector", () => {
         render(
           <div className="bounding-container">
             <AdhesiveContainer boundingEl=".bounding-container">