summaryrefslogtreecommitdiff
path: root/spec/services/projects/group_links/destroy_service_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/services/projects/group_links/destroy_service_spec.rb')
-rw-r--r--spec/services/projects/group_links/destroy_service_spec.rb18
1 files changed, 14 insertions, 4 deletions
diff --git a/spec/services/projects/group_links/destroy_service_spec.rb b/spec/services/projects/group_links/destroy_service_spec.rb
index 0fd1fcfe1a5..0a8c9580e70 100644
--- a/spec/services/projects/group_links/destroy_service_spec.rb
+++ b/spec/services/projects/group_links/destroy_service_spec.rb
@@ -3,15 +3,25 @@
require 'spec_helper'
describe Projects::GroupLinks::DestroyService, '#execute' do
- let(:project) { create(:project, :private) }
- let!(:group_link) { create(:project_group_link, project: project) }
- let(:user) { create :user }
- let(:subject) { described_class.new(project, user) }
+ let_it_be(:user) { create :user }
+ let_it_be(:project) { create(:project, :private) }
+ let_it_be(:group) { create(:group) }
+ let!(:group_link) { create(:project_group_link, project: project, group: group) }
+
+ subject { described_class.new(project, user) }
it 'removes group from project' do
expect { subject.execute(group_link) }.to change { project.project_group_links.count }.from(1).to(0)
end
+ it 'updates authorization' do
+ group.add_maintainer(user)
+
+ expect { subject.execute(group_link) }.to(
+ change { Ability.allowed?(user, :read_project, project) }
+ .from(true).to(false))
+ end
+
it 'returns false if group_link is blank' do
expect { subject.execute(nil) }.not_to change { project.project_group_links.count }
end