fixup! fix!: enforce regex for agent names

deansheather · deansheather · commit d24701654616 · 2025-02-20T04:55:34.000Z
diff --git a/provisioner/terraform/resources_test.go b/provisioner/terraform/resources_test.go
@@ -984,6 +984,7 @@ func TestInvalidTerraformAddress(t *testing.T) {
 	require.Equal(t, state.Resources[0].ModulePath, "invalid terraform address")
 }
 
+//nolint:tparallel
 func TestAppSlugValidation(t *testing.T) {
 	t.Parallel()
 	ctx, logger := ctxAndLogger(t)
@@ -1061,6 +1062,7 @@ func TestAppSlugDuplicate(t *testing.T) {
 	require.ErrorContains(t, err, "duplicate app slug")
 }
 
+//nolint:tparallel
 func TestAgentNameInvalid(t *testing.T) {
 	t.Parallel()
 	ctx, logger := ctxAndLogger(t)

Original file line number	Diff line number	Diff line change
`@@ -984,6 +984,7 @@ func TestInvalidTerraformAddress(t *testing.T) {`
`984`	`984`	`require.Equal(t, state.Resources[0].ModulePath, "invalid terraform address")`
`985`	`985`	`}`
`986`	`986`
	`987`	`+//nolint:tparallel`
`987`	`988`	`func TestAppSlugValidation(t *testing.T) {`
`988`	`989`	`t.Parallel()`
`989`	`990`	`ctx, logger := ctxAndLogger(t)`
`@@ -1061,6 +1062,7 @@ func TestAppSlugDuplicate(t *testing.T) {`
`1061`	`1062`	`require.ErrorContains(t, err, "duplicate app slug")`
`1062`	`1063`	`}`
`1063`	`1064`
	`1065`	`+//nolint:tparallel`
`1064`	`1066`	`func TestAgentNameInvalid(t *testing.T) {`
`1065`	`1067`	`t.Parallel()`
`1066`	`1068`	`ctx, logger := ctxAndLogger(t)`