molecule-ai/molecule-core
41
0
Fork 2
Code Issues 44 Pull Requests 15 Actions 15 Packages Projects Releases Wiki Activity

fix(handlers): use valid UUIDs for workspace seeds in integration tests

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Molecule AI · core-be 2026-05-12 11:24:19 +00:00
parent b2064cab2b
commit b9d977339b
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
workspace-server/internal/handlers/delegation_executor_integration_test.go
View File

@ -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)
}