From b9d977339b612ffa8d38df5bd3fa3333e9353e3a Mon Sep 17 00:00:00 2001 From: Molecule AI Core-BE Date: Tue, 12 May 2026 11:24:19 +0000 Subject: [PATCH] fix(handlers): use valid UUIDs for workspace seeds in integration tests workspaces.id is UUID-typed. The string IDs like "ws-source-159-integration" caused: pq: invalid input syntax for type uuid Fix: use real UUIDs (AAAAAAAA-AAAA-AAAA-AAAA-AAAAAAAAAAAA / BBBBBBBB-BBBB-BBBB-BBBB-BBBBBBBBBBBB) matching the pattern in delegation_ledger_integration_test.go. Also add the required 'name' column (NOT NULL) to the INSERT. Co-Authored-By: Claude Opus 4.7 --- .../delegation_executor_integration_test.go | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/workspace-server/internal/handlers/delegation_executor_integration_test.go b/workspace-server/internal/handlers/delegation_executor_integration_test.go index f7d62c0d..d6649002 100644 --- a/workspace-server/internal/handlers/delegation_executor_integration_test.go +++ b/workspace-server/internal/handlers/delegation_executor_integration_test.go @@ -43,8 +43,8 @@ import ( // Each test gets a fresh table state. const testDelegationID = "del-159-test-integration" -const testSourceID = "ws-source-159-integration" -const testTargetID = "ws-target-159-integration" +const testSourceID = "aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa" +const testTargetID = "bbbbbbbb-bbbb-bbbb-bbbb-bbbbbbbbbbbb" // setupIntegrationFixtures inserts the rows executeDelegation requires: // - workspaces: source and target (siblings, parent_id=NULL so CanCommunicate=true) @@ -58,14 +58,15 @@ func setupIntegrationFixtures(t *testing.T, conn *sql.DB) func() { // We INSERT ... ON CONFLICT DO NOTHING so parallel test runs don't conflict. for _, ws := range []struct { id string + name string parentID *string // nil means NULL }{ - {testSourceID, nil}, - {testTargetID, nil}, + {testSourceID, "test-source", nil}, + {testTargetID, "test-target", nil}, } { if _, err := conn.ExecContext(context.Background(), - `INSERT INTO workspaces (id, parent_id) VALUES ($1, $2) ON CONFLICT (id) DO NOTHING`, - ws.id, ws.parentID, + `INSERT INTO workspaces (id, name, parent_id) VALUES ($1::uuid, $2, $3) ON CONFLICT (id) DO NOTHING`, + ws.id, ws.name, ws.parentID, ); err != nil { t.Fatalf("seed workspace %s: %v", ws.id, err) }