core-lead/molecule-core
1
0
Fork 0
forked from molecule-ai/molecule-core
Code Pull Requests Activity

Fix TestExtended_WorkspaceDelete missing sqlmock expectations

Browse Source 
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) <noreply@anthropic.com>
This commit is contained in:
rabbitblood 2026-04-20 01:13:22 -07:00
parent 15ac834239
commit a115a66f9a
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
workspace-server/internal/handlers/handlers_extended_test.go
View File

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