From e160f304b1f51ebf140f6c82ac9244cae01fc132 Mon Sep 17 00:00:00 2001 From: Max Date: Fri, 30 Jan 2026 20:09:28 +0800 Subject: [PATCH] Update Sandbox Integration Test to Reflect Command Structure Changes - Modified the command verification in the sandbox integration test to assert the new command structure, which now starts with "bash" and includes specific arguments for executing CCR commands. - Enhanced assertions to check for the presence of required command components, improving the robustness of the test. --- agent/assistant/sandbox_integration_test.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/agent/assistant/sandbox_integration_test.go b/agent/assistant/sandbox_integration_test.go index c4214e92..4852f9dd 100644 --- a/agent/assistant/sandbox_integration_test.go +++ b/agent/assistant/sandbox_integration_test.go @@ -78,8 +78,12 @@ func TestClaudeCommandBuilding(t *testing.T) { require.NoError(t, err) // Verify command structure + // Command is now: ["bash", "-c", "nohup ccr start ... && ccr code ..."] assert.NotEmpty(t, cmd) - assert.Equal(t, "ccr-run", cmd[0], "Command should start with ccr-run") + assert.Equal(t, "bash", cmd[0], "Command should start with bash") + assert.Equal(t, "-c", cmd[1], "Second arg should be -c") + assert.Contains(t, cmd[2], "ccr code", "Bash command should contain ccr code") + assert.Contains(t, cmd[2], "--permission-mode", "Should include permission mode") t.Logf("Built command: %v", cmd) // Verify environment variables