From a115a66f9a92ea336c1445acdbb2573d4314cd63 Mon Sep 17 00:00:00 2001 From: rabbitblood Date: Mon, 20 Apr 2026 01:13:22 -0700 Subject: [PATCH] Fix TestExtended_WorkspaceDelete missing sqlmock expectations The Delete handler acquired token revocation and schedule disable queries but this test was never updated, causing sqlmock strict mode to reject the unexpected ExecQuery calls. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../internal/handlers/handlers_extended_test.go | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/workspace-server/internal/handlers/handlers_extended_test.go b/workspace-server/internal/handlers/handlers_extended_test.go index f3cbbb27..564f3a35 100644 --- a/workspace-server/internal/handlers/handlers_extended_test.go +++ b/workspace-server/internal/handlers/handlers_extended_test.go @@ -35,6 +35,14 @@ func TestExtended_WorkspaceDelete(t *testing.T) { mock.ExpectExec("DELETE FROM canvas_layouts WHERE workspace_id = ANY"). WillReturnResult(sqlmock.NewResult(1, 1)) + // Token revocation for deleted workspaces. + mock.ExpectExec("UPDATE workspace_auth_tokens SET revoked_at"). + WillReturnResult(sqlmock.NewResult(0, 0)) + + // #1027: cascade-disable schedules for deleted workspaces. + mock.ExpectExec("UPDATE workspace_schedules SET enabled = false"). + WillReturnResult(sqlmock.NewResult(0, 0)) + // Expect RecordAndBroadcast INSERT for WORKSPACE_REMOVED mock.ExpectExec("INSERT INTO structure_events"). WillReturnResult(sqlmock.NewResult(0, 1))