summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2014-10-28 16:00:03 +0200
committerValery Sizov <valery@gitlab.com>2014-10-29 16:00:38 +0200
commit43fba7f0c518942d060e7a7dd9f7ec3065ac1f69 (patch)
tree5d4653185c0948f493796eabece206ed143834fa
parent912715c3e452723dd7dda3430f57e69b7f1605af (diff)
downloadgitlab-ce-43fba7f0c518942d060e7a7dd9f7ec3065ac1f69.tar.gz
Add failing test that should be green after group members api get fixed
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-rw-r--r--spec/requests/api/groups_spec.rb20
1 files changed, 17 insertions, 3 deletions
diff --git a/spec/requests/api/groups_spec.rb b/spec/requests/api/groups_spec.rb
index 42ccad71aaf..f56caeaf5ad 100644
--- a/spec/requests/api/groups_spec.rb
+++ b/spec/requests/api/groups_spec.rb
@@ -220,13 +220,27 @@ describe API::API, api: true do
context "when a member of the group" do
it "should return ok and add new member" do
- count_before=group_no_members.group_members.count
new_user = create(:user)
- post api("/groups/#{group_no_members.id}/members", owner), user_id: new_user.id, access_level: GroupMember::MASTER
+
+ expect {
+ post api("/groups/#{group_no_members.id}/members", owner),
+ user_id: new_user.id, access_level: GroupMember::MASTER
+ }.to change { group_no_members.members.count }.by(1)
+
response.status.should == 201
json_response['name'].should == new_user.name
json_response['access_level'].should == GroupMember::MASTER
- group_no_members.group_members.count.should == count_before + 1
+ end
+
+ it "should not allow guest to modify group members" do
+ new_user = create(:user)
+
+ expect {
+ post api("/groups/#{group_with_members.id}/members", guest),
+ user_id: new_user.id, access_level: GroupMember::MASTER
+ }.not_to change { group_with_members.members.count }
+
+ response.status.should == 403
end
it "should return error if member already exists" do