opencode/yao
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

test(robot): extend timeout for execution status checks in E2E tests

Browse source 
- Increased the timeout duration from 60 seconds to 120 seconds for waiting on execution statuses in multiple E2E test cases, ensuring more reliable test outcomes during longer execution scenarios.
This commit is contained in:
Max 2026-03-24 23:04:26 +08:00
parent 38a3316336
commit 68c29ca1c0
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
agent/robot/api/e2e_suspend_test.go
View file

@ -94,7 +94,7 @@ func TestE2ENormalExecutionNoSuspend(t *testing.T) {
result := triggerSuspendRobot(t, ctx, memberID, "Write a one-sentence greeting") result := triggerSuspendRobot(t, ctx, memberID, "Write a one-sentence greeting")
exec := waitForStatus(t, result.ExecutionID, exec := waitForStatus(t, result.ExecutionID,
[]types.ExecStatus{types.ExecCompleted, types.ExecFailed}, 60*time.Second) []types.ExecStatus{types.ExecCompleted, types.ExecFailed}, 120*time.Second)
require.NotNil(t, exec, "Execution should exist and reach terminal state") require.NotNil(t, exec, "Execution should exist and reach terminal state")
if exec.Status == types.ExecFailed { if exec.Status == types.ExecFailed {
@ -137,7 +137,7 @@ func TestE2ESuspendResumeFlow(t *testing.T) {
// Step 2: Wait for the execution to reach waiting status // Step 2: Wait for the execution to reach waiting status
exec := waitForStatus(t, execID, exec := waitForStatus(t, execID,
[]types.ExecStatus{types.ExecWaiting, types.ExecCompleted, types.ExecFailed}, 60*time.Second) []types.ExecStatus{types.ExecWaiting, types.ExecCompleted, types.ExecFailed}, 120*time.Second)
require.NotNil(t, exec, "Execution should exist") require.NotNil(t, exec, "Execution should exist")
require.Equal(t, types.ExecWaiting, exec.Status, "Execution should be in waiting status") require.Equal(t, types.ExecWaiting, exec.Status, "Execution should be in waiting status")
@ -193,7 +193,7 @@ func TestE2EReplyShortcut(t *testing.T) {
result := triggerSuspendRobot(t, ctx, memberID, "Check inventory levels") result := triggerSuspendRobot(t, ctx, memberID, "Check inventory levels")
exec := waitForStatus(t, result.ExecutionID, exec := waitForStatus(t, result.ExecutionID,
[]types.ExecStatus{types.ExecWaiting}, 60*time.Second) []types.ExecStatus{types.ExecWaiting}, 120*time.Second)
require.NotNil(t, exec, "Execution should reach waiting status") require.NotNil(t, exec, "Execution should reach waiting status")
require.Equal(t, types.ExecWaiting, exec.Status) require.Equal(t, types.ExecWaiting, exec.Status)
@ -229,7 +229,7 @@ func TestE2EResumeContextPersistence(t *testing.T) {
result := triggerSuspendRobot(t, ctx, memberID, "Analyze user behavior") result := triggerSuspendRobot(t, ctx, memberID, "Analyze user behavior")
exec := waitForStatus(t, result.ExecutionID, exec := waitForStatus(t, result.ExecutionID,
[]types.ExecStatus{types.ExecWaiting, types.ExecCompleted, types.ExecFailed}, 60*time.Second) []types.ExecStatus{types.ExecWaiting, types.ExecCompleted, types.ExecFailed}, 120*time.Second)
require.NotNil(t, exec) require.NotNil(t, exec)
if exec.Status != types.ExecWaiting { if exec.Status != types.ExecWaiting {
@ -298,7 +298,7 @@ func TestE2EInteractWithNonWaitingExecution(t *testing.T) {
// Wait for completion // Wait for completion
exec := waitForStatus(t, result.ExecutionID, exec := waitForStatus(t, result.ExecutionID,
[]types.ExecStatus{types.ExecCompleted, types.ExecFailed}, 60*time.Second) []types.ExecStatus{types.ExecCompleted, types.ExecFailed}, 120*time.Second)
require.NotNil(t, exec, "Execution should reach terminal state") require.NotNil(t, exec, "Execution should reach terminal state")
// Try to interact with the completed execution // Try to interact with the completed execution