Commit1258902

authored

chore: update error message for duplicate organization members (#17594)

1 parenta78f0fc commit1258902Copy full SHA for 1258902

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,8 @@ func (api API) postOrganizationMember(rw http.ResponseWriter, r http.Request)`
`62`	`62`	`}`
`63`	`63`	`ifdatabase.IsUniqueViolation(err,database.UniqueOrganizationMembersPkey) {`
`64`	`64`	`httpapi.Write(ctx,rw,http.StatusBadRequest, codersdk.Response{`
`65`		`-Message:"Organization member already exists in this organization",`
	`65`	`+Message:"User is already an organization member",`
	`66`	`+Detail:fmt.Sprintf("%s is already a member of %s",user.Username,organization.DisplayName),`
`66`	`67`	`})`
`67`	`68`	`return`
`68`	`69`	`}`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ func TestAddMember(t *testing.T) {`
`26`	`26`	`// Add user to org, even though they already exist`
`27`	`27`	`// nolint:gocritic // must be an owner to see the user`
`28`	`28`	`_,err:=owner.PostOrganizationMember(ctx,first.OrganizationID,user.Username)`
`29`		`-require.ErrorContains(t,err,"alreadyexists")`
	`29`	`+require.ErrorContains(t,err,"alreadyan organization member")`
`30`	`30`	`})`
`31`	`31`	`}`
`32`	`32`

Comments

(0)