use const for test

Emyrk · Emyrk · commit 0656d80f0724 · 2024-02-28T11:02:22.000-06:00
diff --git a/enterprise/coderd/provisionerdaemons.go b/enterprise/coderd/provisionerdaemons.go
@@ -117,6 +117,7 @@ func (p *provisionerDaemonAuth) authorize(r *http.Request, tags map[string]strin
 	provAuth := httpmw.ProvisionerDaemonAuthenticated(r)
 	if provAuth {
 		// If using PSK auth, the daemon is, by definition, scoped to the organization.
+		tags = provisionersdk.MutateTags(uuid.Nil, tags)
 		return tags, true
 	}
 	return nil, false
diff --git a/enterprise/coderd/provisionerdaemons_test.go b/enterprise/coderd/provisionerdaemons_test.go
@@ -350,6 +350,7 @@ func TestProvisionerDaemonServe(t *testing.T) {
 
 	t.Run("PSK_daily_cost", func(t *testing.T) {
 		t.Parallel()
+		const provPSK = `provisionersftw`
 		client, user := coderdenttest.New(t, &coderdenttest.Options{
 			UserWorkspaceQuota: 10,
 			LicenseOptions: &coderdenttest.LicenseOptions{
@@ -358,7 +359,7 @@ func TestProvisionerDaemonServe(t *testing.T) {
 					codersdk.FeatureTemplateRBAC:               1,
 				},
 			},
-			ProvisionerDaemonPSK: "provisionersftw",
+			ProvisionerDaemonPSK: provPSK,
 		})
 		logger := slogtest.Make(t, nil).Leveled(slog.LevelDebug)
 		ctx, cancel := context.WithTimeout(context.Background(), testutil.WaitLong)
@@ -397,10 +398,10 @@ func TestProvisionerDaemonServe(t *testing.T) {
 				Tags: map[string]string{
 					provisionersdk.TagScope: provisionersdk.ScopeOrganization,
 				},
-				PreSharedKey: "provisionersftw",
+				PreSharedKey: provPSK,
 			})
 		}, &provisionerd.Options{
-			Logger:    logger.Named("provisionerd"),
+			Logger:    logger.Named(provPSK),
 			Connector: connector,
 		})
 		defer pd.Close()

Original file line number	Diff line number	Diff line change
`@@ -117,6 +117,7 @@ func (p provisionerDaemonAuth) authorize(r http.Request, tags map[string]strin`
`117`	`117`	`provAuth := httpmw.ProvisionerDaemonAuthenticated(r)`
`118`	`118`	`if provAuth {`
`119`	`119`	`// If using PSK auth, the daemon is, by definition, scoped to the organization.`
	`120`	`+ tags = provisionersdk.MutateTags(uuid.Nil, tags)`
`120`	`121`	`return tags, true`
`121`	`122`	`}`
`122`	`123`	`return nil, false`