IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-29 04:03:57 -06:00
Code Issues Packages Projects Releases Wiki Activity

Improve add user to group spec for uppercase usernames

Browse Source 
Oops forgot to check for this. See previous commit for more details.
This commit is contained in:
Blake Erickson 2018-10-18 13:32:36 -06:00
parent 93485facaf
commit f1ba981ae9
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
spec/requests/groups_controller_spec.rb
View File

@ -861,7 +861,7 @@ describe GroupsController do
it "adds by username" do it "adds by username" do
expect do expect do
put "/groups/#{group.id}/members.json", put "/groups/#{group.id}/members.json",
params: { usernames: [user1.username, user2.username].join(",") } params: { usernames: [user1.username, user2.username.upcase].join(",") }
end.to change { group.users.count }.by(2) end.to change { group.users.count }.by(2)
expect(response.status).to eq(200) expect(response.status).to eq(200)
@ -1075,7 +1075,7 @@ describe GroupsController do
it "removes by username" do it "removes by username" do
expect do expect do
delete "/groups/#{group1.id}/members.json", delete "/groups/#{group1.id}/members.json",
params: { usernames: [user1.username, user2.username].join(",") } params: { usernames: [user1.username, user2.username.upcase].join(",") }
end.to change { group1.users.count }.by(-2) end.to change { group1.users.count }.by(-2)
expect(response.status).to eq(200) expect(response.status).to eq(200)
end end