2016-03-08 19:01:33 -05:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe Groups::UpdateService, services: true do
|
2016-12-20 11:52:27 -05:00
|
|
|
let!(:user) { create(:user) }
|
|
|
|
let!(:private_group) { create(:group, :private) }
|
|
|
|
let!(:internal_group) { create(:group, :internal) }
|
|
|
|
let!(:public_group) { create(:group, :public) }
|
2016-03-08 19:01:33 -05:00
|
|
|
|
2016-03-20 18:09:33 -04:00
|
|
|
describe "#execute" do
|
2016-03-08 19:01:33 -05:00
|
|
|
context "project visibility_level validation" do
|
|
|
|
context "public group with public projects" do
|
2016-12-20 11:52:27 -05:00
|
|
|
let!(:service) { described_class.new(public_group, user, visibility_level: Gitlab::VisibilityLevel::INTERNAL) }
|
2016-03-08 19:01:33 -05:00
|
|
|
|
|
|
|
before do
|
|
|
|
public_group.add_user(user, Gitlab::Access::MASTER)
|
2017-03-27 17:14:01 -04:00
|
|
|
create(:empty_project, :public, group: public_group)
|
2016-03-08 19:01:33 -05:00
|
|
|
end
|
|
|
|
|
2016-03-20 18:26:58 -04:00
|
|
|
it "does not change permission level" do
|
|
|
|
service.execute
|
|
|
|
expect(public_group.errors.count).to eq(1)
|
2016-03-08 19:01:33 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-03-18 08:28:16 -04:00
|
|
|
context "internal group with internal project" do
|
2016-12-20 11:52:27 -05:00
|
|
|
let!(:service) { described_class.new(internal_group, user, visibility_level: Gitlab::VisibilityLevel::PRIVATE) }
|
2016-03-20 18:09:33 -04:00
|
|
|
|
|
|
|
before do
|
|
|
|
internal_group.add_user(user, Gitlab::Access::MASTER)
|
2017-03-27 17:14:01 -04:00
|
|
|
create(:empty_project, :internal, group: internal_group)
|
2016-03-20 18:09:33 -04:00
|
|
|
end
|
|
|
|
|
2016-03-20 18:26:58 -04:00
|
|
|
it "does not change permission level" do
|
|
|
|
service.execute
|
|
|
|
expect(internal_group.errors.count).to eq(1)
|
2016-03-08 19:01:33 -05:00
|
|
|
end
|
2016-03-20 18:09:33 -04:00
|
|
|
end
|
2016-03-08 19:01:33 -05:00
|
|
|
end
|
2017-03-28 07:09:44 -04:00
|
|
|
|
|
|
|
context "with parent_id user doesn't have permissions for" do
|
|
|
|
let(:service) { described_class.new(public_group, user, parent_id: private_group.id) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
service.execute
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not update parent_id' do
|
|
|
|
updated_group = public_group.reload
|
|
|
|
|
|
|
|
expect(updated_group.parent_id).to be_nil
|
|
|
|
end
|
|
|
|
end
|
2016-03-08 19:01:33 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
context "unauthorized visibility_level validation" do
|
2016-12-20 11:52:27 -05:00
|
|
|
let!(:service) { described_class.new(internal_group, user, visibility_level: 99) }
|
2016-03-20 18:09:33 -04:00
|
|
|
before do
|
|
|
|
internal_group.add_user(user, Gitlab::Access::MASTER)
|
|
|
|
end
|
2016-03-08 19:01:33 -05:00
|
|
|
|
|
|
|
it "does not change permission level" do
|
2016-03-20 18:26:58 -04:00
|
|
|
service.execute
|
2016-03-08 19:01:33 -05:00
|
|
|
expect(internal_group.errors.count).to eq(1)
|
|
|
|
end
|
|
|
|
end
|
2016-12-20 11:52:27 -05:00
|
|
|
|
|
|
|
context 'rename group' do
|
2017-02-06 23:08:51 -05:00
|
|
|
let!(:service) { described_class.new(internal_group, user, path: SecureRandom.hex) }
|
2016-12-20 11:52:27 -05:00
|
|
|
|
|
|
|
before do
|
|
|
|
internal_group.add_user(user, Gitlab::Access::MASTER)
|
2017-03-27 17:14:01 -04:00
|
|
|
create(:empty_project, :internal, group: internal_group)
|
2016-12-20 11:52:27 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(service.execute).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'error moving group' do
|
|
|
|
before do
|
|
|
|
allow(internal_group).to receive(:move_dir).and_raise(Gitlab::UpdatePathError)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not raise an error' do
|
|
|
|
expect { service.execute }.not_to raise_error
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(service.execute).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'has the right error' do
|
|
|
|
service.execute
|
|
|
|
|
|
|
|
expect(internal_group.errors.full_messages.first).to eq('Gitlab::UpdatePathError')
|
|
|
|
end
|
2016-12-21 07:29:27 -05:00
|
|
|
|
|
|
|
it "hasn't changed the path" do
|
|
|
|
expect { service.execute}.not_to change { internal_group.reload.path}
|
|
|
|
end
|
2016-12-20 11:52:27 -05:00
|
|
|
end
|
|
|
|
end
|
2016-03-08 19:01:33 -05:00
|
|
|
end
|