From 4e1f54b28dd09239e32723e20544e1fb309b950e Mon Sep 17 00:00:00 2001 From: Jeffrey Smith Date: Mon, 30 Mar 2026 20:42:36 +0000 Subject: [PATCH] Fix upgrade test bool literal for Postgres Use `true` instead of `1` for BOOLEAN column insert in TestUpgrade_SchemaMultiColumnAdd. Postgres BOOLEAN rejects integer literals; `true` works on both PG and SQLite. Co-Authored-By: Claude Opus 4.6 (1M context) --- server/handlers/upgrade_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/server/handlers/upgrade_test.go b/server/handlers/upgrade_test.go index 23089ec..7d1c1ea 100644 --- a/server/handlers/upgrade_test.go +++ b/server/handlers/upgrade_test.go @@ -236,8 +236,9 @@ func TestUpgrade_SchemaMultiColumnAdd(t *testing.T) { } // Verify new row can use all columns + // Use 'true' for bool — works on both Postgres (BOOLEAN) and SQLite (INTEGER, coerced to 1) _, err = database.TestDB.ExecContext(ctx, - fmt.Sprintf("INSERT INTO %s (id, name, priority, score, active) VALUES ('r2', 'new', 3, 4.5, 1)", physical)) + fmt.Sprintf("INSERT INTO %s (id, name, priority, score, active) VALUES ('r2', 'new', 3, 4.5, true)", physical)) if err != nil { t.Fatal("insert with new columns failed:", err) }