From 265bd49f3a53c249ee9cd743c3c3509d0322a966 Mon Sep 17 00:00:00 2001 From: Molecule AI Infra-Runtime-BE Date: Thu, 14 May 2026 03:28:28 +0000 Subject: [PATCH] fix(handlers): update executeDelegation calls in integration tests executeDelegation signature changed from 5 params to 4 params on staging (ctx removed). Update all 5 integration test call sites in delegation_executor_integration_test.go to match. Companion fix for PR #916 (fix/904-handler-test-blockers). Co-Authored-By: Claude Opus 4.7 --- .../delegation_executor_integration_test.go | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/workspace-server/internal/handlers/delegation_executor_integration_test.go b/workspace-server/internal/handlers/delegation_executor_integration_test.go index 43625d4a..c8de3bd3 100644 --- a/workspace-server/internal/handlers/delegation_executor_integration_test.go +++ b/workspace-server/internal/handlers/delegation_executor_integration_test.go @@ -303,7 +303,8 @@ func TestIntegration_ExecuteDelegation_DeliveryConfirmedProxyError_TreatsAsSucce start := time.Now() runWithTimeout(t, 30*time.Second, func(ctx context.Context) { - dh.executeDelegation(ctx, testSourceID, testTargetID, testDelegationID, a2aBody) + _ = ctx // ctx unused: executeDelegation manages its own 30-min timeout internally + dh.executeDelegation(testSourceID, testTargetID, testDelegationID, a2aBody) }) t.Logf("executeDelegation took %v", time.Since(start)) @@ -355,7 +356,8 @@ func TestIntegration_ExecuteDelegation_ProxyErrorNon2xx_RemainsFailed(t *testing }) start := time.Now() runWithTimeout(t, 30*time.Second, func(ctx context.Context) { - dh.executeDelegation(ctx, testSourceID, testTargetID, testDelegationID, a2aBody) + _ = ctx // ctx unused: executeDelegation manages its own 30-min timeout internally + dh.executeDelegation(testSourceID, testTargetID, testDelegationID, a2aBody) }) t.Logf("executeDelegation took %v", time.Since(start)) @@ -404,7 +406,8 @@ func TestIntegration_ExecuteDelegation_ProxyErrorEmptyBody_RemainsFailed(t *test }) start := time.Now() runWithTimeout(t, 30*time.Second, func(ctx context.Context) { - dh.executeDelegation(ctx, testSourceID, testTargetID, testDelegationID, a2aBody) + _ = ctx // ctx unused: executeDelegation manages its own 30-min timeout internally + dh.executeDelegation(testSourceID, testTargetID, testDelegationID, a2aBody) }) t.Logf("executeDelegation took %v", time.Since(start)) @@ -452,7 +455,8 @@ func TestIntegration_ExecuteDelegation_CleanProxyResponse_Unchanged(t *testing.T }) start := time.Now() runWithTimeout(t, 30*time.Second, func(ctx context.Context) { - dh.executeDelegation(ctx, testSourceID, testTargetID, testDelegationID, a2aBody) + _ = ctx // ctx unused: executeDelegation manages its own 30-min timeout internally + dh.executeDelegation(testSourceID, testTargetID, testDelegationID, a2aBody) }) t.Logf("executeDelegation took %v", time.Since(start)) @@ -497,7 +501,8 @@ func TestIntegration_ExecuteDelegation_RedisDown_FallsBackToDB(t *testing.T) { }) start := time.Now() runWithTimeout(t, 30*time.Second, func(ctx context.Context) { - dh.executeDelegation(ctx, testSourceID, testTargetID, testDelegationID, a2aBody) + _ = ctx // ctx unused: executeDelegation manages its own 30-min timeout internally + dh.executeDelegation(testSourceID, testTargetID, testDelegationID, a2aBody) }) t.Logf("executeDelegation took %v", time.Since(start)) -- 2.45.2