From 12dd5ca8d9612054eb6d8cf88872b2d16b628440 Mon Sep 17 00:00:00 2001 From: Molecule AI Core-BE Date: Tue, 12 May 2026 15:27:50 +0000 Subject: [PATCH] fix(handlers): remove unused timedExecuteDelegation helper The timedExecuteDelegation wrapper was added during DIAG investigation but is not called by any test. Remove it to keep the test file clean. The runWithTimeout wrapper from the prior commit remains and guards against hanging tests consuming the full CI timeout budget. Co-Authored-By: Claude Opus 4.7 --- .../delegation_executor_integration_test.go | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/workspace-server/internal/handlers/delegation_executor_integration_test.go b/workspace-server/internal/handlers/delegation_executor_integration_test.go index 5e499db2..98f708fa 100644 --- a/workspace-server/internal/handlers/delegation_executor_integration_test.go +++ b/workspace-server/internal/handlers/delegation_executor_integration_test.go @@ -235,22 +235,6 @@ func stack() string { return string(buf[:n]) } -// timedExecuteDelegation wraps dh.executeDelegation with per-step timing prints. -// Each line is prefixed with a timestamp (relative to first print) so the CI log -// reveals exactly which step is blocking. We print to t.Logf so it appears in -// the CI output regardless of test success/failure. -func timedExecuteDelegation(t *testing.T, dh *DelegationHandler, sourceID, targetID, delegationID string, body []byte) { - t.Helper() - ts := time.Now() - logf := func(format string, args ...interface{}) { - t.Logf("[+%v] "+format, time.Since(ts), args...) - } - - logf("STEP setup: entering executeDelegation") - dh.executeDelegation(sourceID, targetID, delegationID, body) - logf("STEP done: executeDelegation returned") -} - // runWithTimeout calls fn in a goroutine and fails t if it doesn't return within // timeout. This prevents a single hanging test from consuming the full 5-minute CI // timeout and leaving no diagnostic time for subsequent tests. The stack trace