HIGHQualitysmallNext.js + Supabase Standards

Test server actions in isolation

Server actions are async functions. Test them directly without rendering UI. Mock requireAuth() and the Supabase client.

Why This Matters

Testing server actions through UI rendering is slow, flaky, and doesn't isolate the mutation logic from rendering concerns.

Related Rules

Use route groups to organize app sections

Architecture

HIGH

Colocate page files with their route segment

Architecture

MEDIUM

Use (select auth.uid()) instead of auth.uid() in policies

Performance

MEDIUM

Use kebab-case for file and directory names

Quality

MEDIUM

Catch this automatically on every PR

BeforeMerge scans your pull requests against this rule and 3+ others. Get actionable feedback before code ships.

Join Waitlist Browse All Rules

Why This Matters

Server actions are pure async functions — test them like any other function.

Good

test("createRule requires auth", async () => {
  vi.mocked(requireAuth).mockRejectedValue(new Error("Unauthorized"))
  await expect(createRule(mockData)).rejects.toThrow("Unauthorized")
})
 
test("createRule inserts with org scope", async () => {
  vi.mocked(requireAuth).mockResolvedValue({ orgId: "org-1", userId: "user-1" })
  await createRule(mockData)
  expect(mockInsert).toHaveBeenCalledWith(
    expect.objectContaining({ organization_id: "org-1" })
  )
})

Test server actions in isolation

Why This Matters

Tags

Related Rules

Catch this automatically on every PR

Why This Matters

Good