test: confirm variables with space still arrives

zimeg · zimeg · commit 28ff7f1b11ab · 2026-03-23T12:18:08.000-07:00
diff --git a/internal/hooks/hook_executor_default_test.go b/internal/hooks/hook_executor_default_test.go
@@ -60,7 +60,8 @@ func Test_Hook_Execute_Default_Protocol(t *testing.T) {
 			opts: HookExecOpts{
 				Hook: HookScript{Name: "happypath", Command: "echo {}"},
 				Env: map[string]string{
-					"BATMAN": "robin hood",
+					"BATMAN": "robin",
+					"WHOAMI": "lumpy space princess",
 					"YIN":    "yang",
 				},
 				Exec: &MockExec{
@@ -74,7 +75,8 @@ func Test_Hook_Execute_Default_Protocol(t *testing.T) {
 				response, err := executor.Execute(ctx, opts)
 				require.Equal(t, "test output", response)
 				require.Equal(t, nil, err)
-				require.Contains(t, opts.Exec.(*MockExec).mockCommand.Env, `BATMAN=robin hood`)
+				require.Contains(t, opts.Exec.(*MockExec).mockCommand.Env, `BATMAN=robin`)
+				require.Contains(t, opts.Exec.(*MockExec).mockCommand.Env, `WHOAMI=lumpy space princess`)
 				require.Contains(t, opts.Exec.(*MockExec).mockCommand.Env, `YIN=yang`)
 			},
 		},
diff --git a/internal/hooks/hook_executor_v2_test.go b/internal/hooks/hook_executor_v2_test.go
@@ -55,7 +55,8 @@ func Test_Hook_Execute_V2_Protocol(t *testing.T) {
 			opts: HookExecOpts{
 				Hook: HookScript{Name: "happypath", Command: "echo {}"},
 				Env: map[string]string{
-					"BATMAN": "robin hood",
+					"BATMAN": "robin",
+					"WHOAMI": "lumpy space princess",
 					"YIN":    "yang",
 				},
 				Exec: &MockExec{
@@ -68,17 +69,17 @@ func Test_Hook_Execute_V2_Protocol(t *testing.T) {
 			check: func(t *testing.T, response string, err error, mockExec ExecInterface) {
 				require.Equal(t, `{"message": "hello world"}`, response)
 				require.Equal(t, nil, err)
-				require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `BATMAN=robin hood`)
+				require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `BATMAN=robin`)
+				require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `WHOAMI=lumpy space princess`)
 				require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `YIN=yang`)
 			},
 		},
 		"successful execution with payload > 64kb": {
 			opts: HookExecOpts{
 				Hook: HookScript{Name: "happypath", Command: "echo {}"},
 				Env: map[string]string{
-					"batman": "robin",
-"BATMAN": "robin",
-"YIN":    "yang",
+					"BATMAN": "robin",
+					"YIN":    "yang",
 				},
 				Exec: &MockExec{
 					mockCommand: &MockCommand{
