mirror of
https://github.com/supabase/supabase.git
synced 2026-10-14 21:59:16 +03:00
Joshen Lim
66bfc5fdc3
## Context PR here mainly breaks up the files under `ConnectSheet` to separate the functional logic so that we can write unit tests. No behavior changes intended beyond the bug fixes ## Changes involved - **Test organization:** moved all root-level `ConnectSheet` test files into `ConnectSheet/__tests__/` for consistency with other parts of the codebase that use this convention. - **Bug fix:** read replica label had a stray `}` / missing `)`, rendering as e.g. `Read Replica (us-east-1 - abc123})` instead of `Read Replica (us-east-1 - abc123)`. - **`ConnectSheet.tsx`:** extracted the "hydrate sheet state on open" `useEffect` logic (mode/field/URL param resolution from URL vs. localStorage) into a new `ConnectSheet.utils.ts`, with unit tests - **`useConnectServerEnv.ts`:** fixed two race conditions in the secret reveal/hide flow: - `toggle()` and `getValue()` could each fire a separate reveal request if triggered close together — now deduped to share one in-flight request. - `getValue()` could hide a secret that had just been explicitly revealed by a concurrent `toggle()`, due to reading a stale closure value — now reads the live state via `useLatest`. - Also stopped swallowing the original error on reveal failure (now attached via `cause`). - Added tests for the above, plus the 10s auto-hide timer (previously untested). - **`ConnectStepsSection.tsx`:** extracted `resolveContentPath` and the three inline "show notice" booleans (IPv4 addon, session pooler, self-hosted MCP) into `ConnectStepsSection.utils.ts`, matching the existing pattern for the Data API notice. Added unit tests for all of them. ## To test - [ ] Just a basic smoke test of the Connect sheet should do <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Improved connect setup hydration so saved preferences and URL values are applied more consistently when opening the sheet, including automatic URL backfilling where needed. * Refreshed connection guidance notices (IPv4 add-on, session pooler, and self-hosted MCP) with more consistent logic. * **Bug Fixes** * Fixed secret reveal behavior to keep concurrent reveal actions in sync, handle failures more safely, and ensure auto-hide works reliably. * Corrected the read-replica option label formatting. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
32 lines
1.0 KiB
TypeScript
32 lines
1.0 KiB
TypeScript
import { describe, expect, test } from 'vitest'
|
|
|
|
import { buildConnectPrompt } from '../CopyPromptAdmonition'
|
|
|
|
describe('buildConnectPrompt', () => {
|
|
test('uses redacted copy values instead of temporarily visible passwords', () => {
|
|
const container = document.createElement('div')
|
|
container.innerHTML = `
|
|
<div
|
|
data-connect-step
|
|
data-step-title="Connection string"
|
|
data-step-description="Copy the connection details for your database."
|
|
>
|
|
<div data-step-content>
|
|
<div
|
|
data-connect-copy-value="postgresql://postgres:[YOUR-PASSWORD]@db.example.supabase.co:5432/postgres"
|
|
>
|
|
<pre>postgresql://postgres:temporary-password@db.example.supabase.co:5432/postgres</pre>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
`
|
|
|
|
const prompt = buildConnectPrompt(container)
|
|
|
|
expect(prompt).toContain(
|
|
'postgresql://postgres:[YOUR-PASSWORD]@db.example.supabase.co:5432/postgres'
|
|
)
|
|
expect(prompt).not.toContain('temporary-password')
|
|
})
|
|
})
|