mirror of
https://github.com/supabase/supabase.git
synced 2026-10-09 11:25:06 +03:00
## What Step 4 of the SQL editor testability plan: **mock-free hook tests** for the extracted SQL editor hooks, built on the Step 3 renderHook harness (`tests/lib/sql-editor-test-utils.tsx`) — in-memory editor port + real valtio stores + MSW. **Zero `vi.mock`.** | File | Tests | Covers | |------|-------|--------| | `useSqlEditorExecution.test.tsx` | 8 | destructive-query gating (`potentialIssues` vs. forced run), auto-limit suffixing, connection-string → `x-connection-encrypted` header, `onSuccess`/`onError` session-store writes, error-line highlight, diff-open short-circuit | | `useSqlEditorAi.test.tsx` | 7 | one-shot diff-request drain (empty vs. non-empty editor), drain-exactly-once across remounts, accept/discard diff, `onDebug` opening the assistant chat + debug prompt | | `usePrettifyQuery.test.tsx` | 2 | in-place format + write-back, diff-open no-op | | `useSnippetIdentity.test.tsx` | 2 | generated identity + store-driven loading state | | `useSnippetTitleGenerator.test.tsx` | 2 | untitled-snippet naming via the title endpoint | Every test exercises real dependencies at the seam where they're real: network via MSW, stores used real and reset per test, Monaco via the in-memory editor port. ## Test plan - [x] `pnpm test:studio -- SQLEditor` → **286/286 passing** (21 new tests included) - [x] `pnpm --filter studio typecheck` clean - [x] Confirmed zero `vi.mock` in the new files <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Added comprehensive automated coverage for SQL query formatting, snippet identity, and AI-generated titles. * Added coverage for AI-assisted SQL editing, including diff acceptance, rejection, debugging, and request handling. * Added coverage for query execution, result persistence, safety checks, replica selection, error highlighting, and diff-state behavior. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
51 lines
1.5 KiB
TypeScript
51 lines
1.5 KiB
TypeScript
import { act, waitFor } from '@testing-library/react'
|
|
import { afterEach, beforeEach, describe, expect, it } from 'vitest'
|
|
|
|
import { useSnippetTitleGenerator } from './useSnippetTitleGenerator'
|
|
import { sqlEditorState } from '@/state/sql-editor/sql-editor-state'
|
|
import {
|
|
renderSqlEditorHook,
|
|
resetSqlEditorStores,
|
|
seedSnippet,
|
|
setupSqlEditorMocks,
|
|
} from '@/tests/lib/sql-editor-test-utils'
|
|
|
|
const SNIPPET_ID = 'title-snippet'
|
|
|
|
beforeEach(() => {
|
|
resetSqlEditorStores()
|
|
// The title endpoint (POST /ai/sql/title-v2) returns { title: 'Generated title' }.
|
|
setupSqlEditorMocks()
|
|
seedSnippet({ id: SNIPPET_ID, name: 'Untitled query', sql: 'select 1;' })
|
|
})
|
|
|
|
afterEach(() => {
|
|
resetSqlEditorStores()
|
|
})
|
|
|
|
describe('useSnippetTitleGenerator', () => {
|
|
it('names an untitled snippet from the generated title and queues it for saving', async () => {
|
|
const { result } = renderSqlEditorHook(useSnippetTitleGenerator)
|
|
|
|
await act(async () => {
|
|
await result.current.setAiTitle(SNIPPET_ID, 'select 1;')
|
|
})
|
|
|
|
await waitFor(() =>
|
|
expect(sqlEditorState.snippets[SNIPPET_ID].snippet.name).toBe('Generated title')
|
|
)
|
|
expect(sqlEditorState.needsSaving.has(SNIPPET_ID)).toBe(true)
|
|
})
|
|
|
|
it('returns the generated title from generateSqlTitle', async () => {
|
|
const { result } = renderSqlEditorHook(useSnippetTitleGenerator)
|
|
|
|
let title: string | undefined
|
|
await act(async () => {
|
|
title = (await result.current.generateSqlTitle({ sql: 'select 1;' })).title
|
|
})
|
|
|
|
expect(title).toBe('Generated title')
|
|
})
|
|
})
|