6bf8bebf51
CI / Test and Build (push) Failing after 1s
CI / Migrate Dev DB (push) Has been skipped
CI / Migrate DB (push) Has been skipped
CodeQL / Analyze actions (push) Has been cancelled
CodeQL / Analyze javascript-typescript (push) Has been cancelled
CI / Detect Version (push) Has been cancelled
CI / Detect Desktop Changes (push) Has been cancelled
CI / Build AMD64 (blacksmith-2vcpu-ubuntu-2404, ./docker/cron.Dockerfile, ubuntu-latest, ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Build AMD64 (blacksmith-2vcpu-ubuntu-2404, ./docker/db.Dockerfile, ECR_MIGRATIONS, ubuntu-latest, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build AMD64 (blacksmith-4vcpu-ubuntu-2404, ./docker/pii.Dockerfile, ECR_PII, ubuntu-latest, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build AMD64 (blacksmith-4vcpu-ubuntu-2404, ./docker/realtime.Dockerfile, ECR_REALTIME, ubuntu-latest, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build AMD64 (blacksmith-8vcpu-ubuntu-2404, ./docker/app.Dockerfile, ECR_APP, linux-x64-8-core, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/cron.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/db.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/pii.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/realtime.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-8vcpu-ubuntu-2404-arm, ./docker/app.Dockerfile, linux-arm64-8-core, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Check Docs Changes (push) Has been cancelled
Publish CLI Package / publish-npm (push) Has been cancelled
Publish Python SDK / publish-pypi (push) Has been cancelled
CI / Deploy Trigger.dev (Dev) (push) Has been cancelled
Helm Chart / Lint, test, and validate chart (push) Has been cancelled
Helm Chart / Chart version bumped (push) Has been cancelled
Publish TypeScript SDK / publish-npm (push) Has been cancelled
CI / Build Dev ECR (blacksmith-8vcpu-ubuntu-2404, ./docker/app.Dockerfile, ECR_APP, linux-x64-8-core) (push) Has been cancelled
CI / Promote Images (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-2vcpu-ubuntu-2404, ./docker/db.Dockerfile, ECR_MIGRATIONS, ubuntu-latest) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-4vcpu-ubuntu-2404, ./docker/pii.Dockerfile, ECR_PII, ubuntu-latest) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-4vcpu-ubuntu-2404, ./docker/realtime.Dockerfile, ECR_REALTIME, ubuntu-latest) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Process Docs (push) Has been cancelled
CI / Create GitHub Release (push) Has been cancelled
CI / Check Desktop Signing Secrets (push) Has been cancelled
CI / Desktop Release (push) Has been cancelled
CI / Create Desktop Prerelease (push) Has been cancelled
CI / Desktop Prerelease Build (push) Has been cancelled
CI / Publish Desktop Prerelease (push) Has been cancelled
CI / Prune Desktop Prereleases (push) Has been cancelled
Helm Chart / Install on kind and run helm test (push) Has been cancelled
55 lines
2.2 KiB
TypeScript
55 lines
2.2 KiB
TypeScript
/**
|
|
* @vitest-environment node
|
|
*/
|
|
|
|
// Renders the real key-removal expression against the real drizzle dialect.
|
|
// utils.test.ts mocks drizzle wholesale, so its assertions passed against BOTH
|
|
// previously-broken forms (`- ${keys}::text[]` and `- ${sql.param(keys)}::text[]`).
|
|
// Only a real render can tell them apart, and only the params reveal an array bind:
|
|
// the app pools set `fetch_types: false` (packages/db/db.ts), which leaves
|
|
// postgres-js unable to serialize an array bound as a single parameter.
|
|
import { describe, expect, it, vi } from 'vitest'
|
|
|
|
vi.unmock('drizzle-orm')
|
|
vi.unmock('@sim/db')
|
|
vi.unmock('@sim/db/schema')
|
|
|
|
process.env.DATABASE_URL ??= 'postgresql://user:pass@localhost:5432/test'
|
|
|
|
const { sql } = await import('drizzle-orm')
|
|
const { PgDialect } = await import('drizzle-orm/pg-core')
|
|
|
|
/** Mirrors the expression built in `updateWebhookProviderConfig`. */
|
|
function renderKeyRemoval(removedKeys: string[]) {
|
|
const merged = sql`COALESCE("provider_config"::jsonb, '{}'::jsonb) || ${JSON.stringify({ a: 1 })}::jsonb`
|
|
return new PgDialect().sqlToQuery(
|
|
sql`(${merged}) - ARRAY[${sql.join(
|
|
removedKeys.map((key) => sql`${key}`),
|
|
sql`, `
|
|
)}]::text[]`
|
|
)
|
|
}
|
|
|
|
describe('provider-config key removal SQL', () => {
|
|
// Production only ever removes one key (google-drive's pageToken reset), so the
|
|
// single-element case is the one that actually runs.
|
|
for (const keys of [['pageToken'], ['a', 'b'], ['a', 'b', 'c']]) {
|
|
it(`binds ${keys.length} key(s) as scalars inside an ARRAY constructor`, () => {
|
|
const { sql: text, params } = renderKeyRemoval(keys)
|
|
|
|
expect(text).toContain(`ARRAY[${keys.map((_, i) => `$${i + 2}`).join(', ')}]::text[]`)
|
|
expect(params).toEqual([JSON.stringify({ a: 1 }), ...keys])
|
|
for (const param of params) {
|
|
expect(Array.isArray(param)).toBe(false)
|
|
}
|
|
})
|
|
}
|
|
|
|
it('never renders a row constructor cast to text[]', () => {
|
|
// `- ($1, $2)::text[]` — what interpolating the raw JS array produced. Postgres
|
|
// rejects it: "cannot cast type record to text[]" (and 22P02 at one element).
|
|
const { sql: text } = renderKeyRemoval(['a', 'b'])
|
|
expect(text).not.toMatch(/\(\$\d+(, \$\d+)+\)::text\[\]/)
|
|
})
|
|
})
|