From 5c68f6185d541d4ce378748e0bed5aa55ae75561 Mon Sep 17 00:00:00 2001 From: Kyle Carberry Date: Thu, 26 Jan 2023 17:47:21 +0000 Subject: [PATCH] fix: loop variable captured by func literal in parameters test I noticed this when adding a new parameter type. There's a test case for an empty string that returned false for validation, but appears like it could be true. If there is no value for a string, then the default is used. In this case there is no default, but that's technically fine I believe. --- coderd/workspacebuilds_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/coderd/workspacebuilds_test.go b/coderd/workspacebuilds_test.go index 5caef361d52b7..7bb8e0a761262 100644 --- a/coderd/workspacebuilds_test.go +++ b/coderd/workspacebuilds_test.go @@ -900,7 +900,7 @@ func TestWorkspaceBuildValidateRichParameters(t *testing.T) { {numberParameterName, "10", true, numberRichParameters}, {numberParameterName, "11", false, numberRichParameters}, - {stringParameterName, "", false, stringRichParameters}, + {stringParameterName, "", true, stringRichParameters}, {stringParameterName, "foobar", true, stringRichParameters}, {stringParameterName, "abcd", true, regexRichParameters}, @@ -912,6 +912,7 @@ func TestWorkspaceBuildValidateRichParameters(t *testing.T) { } for _, tc := range tests { + tc := tc t.Run(tc.parameterName+"-"+tc.value, func(t *testing.T) { t.Parallel()