molecule-ai/molecule-core
41
0
Fork 2
Code Issues 43 Pull Requests 16 Actions 1 Packages Projects Releases Wiki Activity

fix(handlers): remove unused timedExecuteDelegation helper

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Molecule AI · core-be 2026-05-12 15:27:50 +00:00
parent 05fcf90816
commit 12dd5ca8d9
1 changed files with 0 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

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