Skip to content

Commit

Permalink
Stabilize the flaky TestResolveSensitiveParameters test (#3268)
Browse files Browse the repository at this point in the history
chore(test): Stabilize TestResolveSensitiveParameters

Signed-off-by: Kim Christensen <kimworking@gmail.com>
  • Loading branch information
kichristensen authored Dec 5, 2024
1 parent 6c56f35 commit 48de62e
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions pkg/runtime/runtime_manifest_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -329,11 +329,13 @@ install:

require.Len(t, args, 3)
assert.Equal(t, "deliciou$dubonnet", args[0])
assert.Equal(t, "{\"secret\":\"this_is_secret\"}", args[1])
assert.Equal(t, "{\"secret\":\"this_is_secret\"}", args[1])
assert.Equal(t, "regular param value", args[2])

// There should now be one sensitive value tracked under the manifest
assert.Equal(t, []string{"deliciou$dubonnet", "{\"secret\":\"this_is_secret\"}"}, rm.GetSensitiveValues())
assert.Len(t, rm.GetSensitiveValues(), 2)
assert.Contains(t, rm.GetSensitiveValues(), "deliciou$dubonnet")
assert.Contains(t, rm.GetSensitiveValues(), "{\"secret\":\"this_is_secret\"}")
}

func TestResolveCredential(t *testing.T) {
Expand Down

0 comments on commit 48de62e

Please sign in to comment.