switchboard/core
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Releases 9 Wiki Activity

Fix OpenAPI tests failing on Postgres CI
All checks were successful
CI/CD / detect-changes (pull_request) Successful in 4s
Details
CI/CD / test-frontend (pull_request) Successful in 5s
Details
CI/CD / test-go-pg (pull_request) Successful in 2m42s
Details
CI/CD / test-sqlite (pull_request) Successful in 2m47s
Details
CI/CD / build-and-deploy (pull_request) Successful in 1m39s
Details

Browse Source 
openapiTestStores() was hardcoded to sqlite.NewStores() regardless of
the DB backend. When CI runs against Postgres, the SQLite ? placeholders
cause "syntax error at or near ','". Use the same IsSQLite() check
pattern as packages_bundled_test.go.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Jeffrey Smith
2026-03-31 11:02:58 +00:00
parent e4b5f4e8a8
commit fad32bd48a
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
server/handlers/openapi_test.go
View File

@@ -7,7 +7,8 @@ import (
"switchboard-core/database" "switchboard-core/database"
"switchboard-core/store" "switchboard-core/store"
"switchboard-core/store/sqlite" postgres "switchboard-core/store/postgres"
sqlite "switchboard-core/store/sqlite"
) )
// Minimal static spec for testing — valid OpenAPI 3.0 structure. // Minimal static spec for testing — valid OpenAPI 3.0 structure.
@@ -32,7 +33,10 @@ func openapiTestStores(t *testing.T) store.Stores {
t.Helper() t.Helper()
database.RequireTestDB(t) database.RequireTestDB(t)
database.TruncateAll(t) database.TruncateAll(t)
return sqlite.NewStores(database.TestDB) if database.IsSQLite() {
return sqlite.NewStores(database.TestDB)
}
return postgres.NewStores(database.TestDB)
} }
// ── Test: Zero extensions → kernel spec only ────────────────── // ── Test: Zero extensions → kernel spec only ──────────────────