Skip to content

Commit d2f878b

Browse files
zimegmwbrooks
andauthored
test: continue preference to uppercase keys for environment
Co-authored-by: Michael Brooks <michael@michaelbrooks.ca>
1 parent 5eb7472 commit d2f878b

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

internal/hooks/hook_executor_v2_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ func Test_Hook_Execute_V2_Protocol(t *testing.T) {
7777
Hook: HookScript{Name: "happypath", Command: "echo {}"},
7878
Env: map[string]string{
7979
"batman": "robin",
80-
"yin": "yang",
80+
"BATMAN": "robin",
81+
"YIN": "yang",
8182
},
8283
Exec: &MockExec{
8384
mockCommand: &MockCommand{
@@ -89,8 +90,8 @@ func Test_Hook_Execute_V2_Protocol(t *testing.T) {
8990
check: func(t *testing.T, response string, err error, mockExec ExecInterface) {
9091
require.Equal(t, sixtyFourKBString, response)
9192
require.Equal(t, nil, err)
92-
require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `batman=robin`)
93-
require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `yin=yang`)
93+
require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `BATMAN=robin`)
94+
require.Contains(t, mockExec.(*MockExec).mockCommand.Env, `YIN=yang`)
9495
},
9596
},
9697
"successful execution with payload > 512kb": {

0 commit comments

Comments
 (0)