Commit 97cf31c4 authored by Marin Jankovski's avatar Marin Jankovski

Update user access level when adding to group.

parent 2c0c9e26
...@@ -26,7 +26,8 @@ class Group < Namespace ...@@ -26,7 +26,8 @@ class Group < Namespace
def add_users(user_ids, group_access) def add_users(user_ids, group_access)
user_ids.compact.each do |user_id| user_ids.compact.each do |user_id|
self.users_groups.create(user_id: user_id, group_access: group_access) user = self.users_groups.find_or_initialize_by_user_id(user_id)
user.update_attributes(group_access: group_access)
end end
end end
......
...@@ -42,4 +42,16 @@ describe Group do ...@@ -42,4 +42,16 @@ describe Group do
it { group.users_groups.masters.map(&:user).should include(user) } it { group.users_groups.masters.map(&:user).should include(user) }
end end
describe :add_users do
let(:user) { create(:user) }
before { group.add_users([user.id], UsersGroup::GUEST) }
it "should update the group permission" do
group.users_groups.guests.map(&:user).should include(user)
group.add_users([user.id], UsersGroup::DEVELOPER)
group.users_groups.developers.map(&:user).should include(user)
group.users_groups.guests.map(&:user).should_not include(user)
end
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment