style:

Author: iRingil

frontend/tests/components/ui/field.test.tsx

@@ -140,7 +140,7 @@ describe("Field", (): void => {
       </Field>,
     )
     const label: HTMLElement = screen.getByText("Email")
-    const requiredIndicator = screen.getByText("*")
+    const requiredIndicator: HTMLElement = screen.getByText("*")
     expect(label).toContainElement(requiredIndicator)
   })
 

frontend/tests/components/ui/password-input.test.tsx

@@ -30,11 +30,7 @@ vi.mock("@/components/ui/field", () => ({
     children,
     errorText,
     invalid,
-  }: {
-    children: ReactElement
-    errorText?: string
-    invalid: boolean
-  }): ReactElement => (
+  }: { children: ReactElement; errorText?: string; invalid: boolean }): ReactElement => (
     <div data-invalid={invalid}>
       {children}
       {errorText && <span>{errorText}</span>}
@@ -51,13 +47,7 @@ vi.mock("@/components/ui/input-group", () => ({
    * @param {ReactElement} [props.endElement] - The element to render at the end.
    * @returns {ReactElement} A mocked div representing the InputGroup.
    */
-  InputGroup: ({
-    children,
-    endElement,
-  }: {
-    children: ReactElement
-    endElement?: ReactElement
-  }): ReactElement => (
+  InputGroup: ({ children, endElement }: { children: ReactElement; endElement?: ReactElement }): ReactElement => (
     <div>
       {children}
       {endElement}
@@ -84,30 +74,15 @@ vi.mock("@chakra-ui/react", async (importOriginal) => {
     any // Using 'any' here is a pragmatic choice for the mock, as the props are a mix of HTML and Chakra-specific ones.
   >(function MockChakraIconButton(props, ref): ReactElement {
     // Correctly destructure and ignore non-standard props for a native button
-    const {
-      aspectRatio,
-      me,
-      size,
-      variant,
-      height,
-      color,
-      children,
-      ...rest // `rest` will now only contain valid HTML button attributes
-    } = props
-
+    const { aspectRatio, me, size, variant, height, color, children, ...rest } = props
     return (
       <button ref={ref} type="button" {...rest}>
         {children}
       </button>
     )
   })
   MockChakraIconButton.displayName = "MockChakraIconButton"
-
-  return {
-    ...original,
-    Input: MockChakraInput,
-    IconButton: MockChakraIconButton,
-  }
+  return { ...original, Input: MockChakraInput, IconButton: MockChakraIconButton }
 })
 
 // endregion
@@ -120,12 +95,8 @@ describe("PasswordInput", () => {
   it("should render the input and toggle type on click", async () => {
     const user = userEvent.setup()
     render(<PasswordInput name="password" errors={{}} />)
-
     const input = screen.getByPlaceholderText<HTMLInputElement>("password-input")
-    const toggleButton = screen.getByRole("button", {
-      name: "Toggle password visibility",
-    })
-
+    const toggleButton = screen.getByRole("button", { name: "Toggle password visibility" })
     expect(input.type).toBe("password")
     await user.click(toggleButton)
     expect(input.type).toBe("text")
@@ -134,9 +105,7 @@ describe("PasswordInput", () => {
   })
 
   it("should display an error message when an error is provided", () => {
-    const errors: FieldErrors<TestForm> = {
-      password: { type: "required", message: "This field is required" },
-    }
+    const errors: FieldErrors<TestForm> = { password: { type: "required", message: "This field is required" } }
     render(<PasswordInput name="password" errors={errors} />)
     expect(screen.getByText("This field is required")).toBeInTheDocument()
   })

frontend/tests/hooks/useAuth.test.ts

@@ -251,14 +251,10 @@ describe("useAuth", (): void => {
       const mockUser: UserPublic = { id: "1", email: "user@example.com" }
       mockedLoginApi.mockResolvedValueOnce(mockAccessTokenResponse)
       mockedReadMeApi.mockResolvedValueOnce(mockUser)
-
       const { result, rerender } = renderAuthHook()
-      // Call with the correctly structured mockLoginData
       result.current.loginMutation.mutate(mockLoginData)
-
       await waitFor((): void => expect(result.current.loginMutation.isSuccess).toBe(true))
       expect(localStorageMock.setItem).toHaveBeenCalledWith("access_token", "mock-jwt-token")
-
       rerender() // Re-render to trigger `currentUser` query if enabled after token is set
       await waitFor((): void => expect(result.current.user).toEqual(mockUser))
       expect(mockedReadMeApi).toHaveBeenCalledTimes(1)