test fix

Emyrk · Emyrk · commit 88e4db92705b · 2024-12-11T11:37:50.000-06:00
diff --git a/enterprise/coderd/scim.go b/enterprise/coderd/scim.go
@@ -517,6 +517,7 @@ func immutabilityViolation[T comparable](old, new T) bool {
 	return old != new
 }
 
+//nolint:revive // active is not a control flag
 func scimUserStatus(user database.User, active bool) database.UserStatus {
 	if !active {
 		return database.UserStatusSuspended
diff --git a/enterprise/coderd/scim_test.go b/enterprise/coderd/scim_test.go
@@ -720,7 +720,7 @@ func TestScim(t *testing.T) {
 
 			sUser.UserName += "changed"
 
-			res, err = client.Request(ctx, http.MethodPut, "/scim/v2/Users", sUser, setScimAuth(scimAPIKey))
+			res, err = client.Request(ctx, http.MethodPut, "/scim/v2/Users/"+sUser.ID, sUser, setScimAuth(scimAPIKey))
 			require.NoError(t, err)
 			defer res.Body.Close()
 			assert.Equal(t, http.StatusBadRequest, res.StatusCode)

Original file line number	Diff line number	Diff line change
`@@ -517,6 +517,7 @@ func immutabilityViolation[T comparable](old, new T) bool {`
`517`	`517`	`return old != new`
`518`	`518`	`}`
`519`	`519`
	`520`	`+//nolint:revive // active is not a control flag`
`520`	`521`	`func scimUserStatus(user database.User, active bool) database.UserStatus {`
`521`	`522`	`if !active {`
`522`	`523`	`return database.UserStatusSuspended`