fix test

mtojek · mtojek · commit 6bc1d2ddf56a · 2024-07-29T15:54:41.000+02:00
diff --git a/coderd/users.go b/coderd/users.go
@@ -1310,9 +1310,9 @@ func (api *API) CreateUser(ctx context.Context, store database.Store, req Create
 		for _, u := range append(owners, userAdmins...) {
 			if _, err := api.NotificationsEnqueuer.Enqueue(ctx, u.ID, notifications.TemplateUserAccountCreated,
 				map[string]string{
-					"user_account_name": user.Name,
+					"user_account_name": user.Username,
 				}, "api-users-create",
-				u.ID,
+				user.ID,
 			); err != nil {
 				api.Logger.Warn(ctx, "unable to notify about created user", slog.F("created_user", user.Name), slog.Error(err))
 			}
diff --git a/coderd/users_test.go b/coderd/users_test.go
@@ -628,7 +628,7 @@ func TestNotifyCreatedUser(t *testing.T) {
 		require.Len(t, notifyEnq.Sent, 1)
 		require.Equal(t, notifications.TemplateUserAccountCreated, notifyEnq.Sent[0].TemplateID)
 		require.Equal(t, firstUser.UserID, notifyEnq.Sent[0].UserID)
-		require.Contains(t, user.ID, notifyEnq.Sent[0].Targets)
+		require.Contains(t, notifyEnq.Sent[0].Targets, user.ID)
 		require.Equal(t, user.Username, notifyEnq.Sent[0].Labels["user_account_name"])
 	})
 }

Original file line number	Diff line number	Diff line change
`@@ -628,7 +628,7 @@ func TestNotifyCreatedUser(t *testing.T) {`
`628`	`628`	`require.Len(t, notifyEnq.Sent, 1)`
`629`	`629`	`require.Equal(t, notifications.TemplateUserAccountCreated, notifyEnq.Sent[0].TemplateID)`
`630`	`630`	`require.Equal(t, firstUser.UserID, notifyEnq.Sent[0].UserID)`
`631`		`- require.Contains(t, user.ID, notifyEnq.Sent[0].Targets)`
	`631`	`+ require.Contains(t, notifyEnq.Sent[0].Targets, user.ID)`
`632`	`632`	`require.Equal(t, user.Username, notifyEnq.Sent[0].Labels["user_account_name"])`
`633`	`633`	`})`
`634`	`634`	`}`