diff --git a/ee/spec/services/groups/create_service_spec.rb b/ee/spec/services/groups/create_service_spec.rb index 12ae3ffbe9adbb689ef1415d2e0d2a7b247de192..fb38313a734e13ec26cc2fe97b1dc8897ab67d77 100644 --- a/ee/spec/services/groups/create_service_spec.rb +++ b/ee/spec/services/groups/create_service_spec.rb @@ -2,7 +2,7 @@ describe Groups::CreateService, '#execute' do let!(:user) { create :user } - let!(:opts) do + let!(:group_params) do { name: 'GitLab', path: 'group_path', @@ -12,7 +12,7 @@ context 'audit events' do include_examples 'audit event logging' do - let(:operation) { create_group(user, opts) } + let(:operation) { create_group(user, group_params) } let(:fail_condition!) do allow(Gitlab::VisibilityLevel).to receive(:allowed_for?).and_return(false) end @@ -33,6 +33,30 @@ end end + context 'repository_size_limit assignment as Bytes' do + let(:admin_user) { create(:user, admin: true) } + + context 'when param present' do + let(:opts) { { repository_size_limit: '100' } } + + it 'assign repository_size_limit as Bytes' do + group = create_group(admin_user, group_params.merge(opts)) + + expect(group.repository_size_limit).to eql(100 * 1024 * 1024) + end + end + + context 'when param not present' do + let(:opts) { { repository_size_limit: '' } } + + it 'assign nil value' do + group = create_group(admin_user, group_params.merge(opts)) + + expect(group.repository_size_limit).to be_nil + end + end + end + def create_group(user, opts) described_class.new(user, opts).execute end diff --git a/ee/spec/services/groups/update_service_spec.rb b/ee/spec/services/groups/update_service_spec.rb index d47afddd2130618494e88415f493b57a7c921ccf..53e56da5779942d2a03249170e2e5237f2e0f849 100644 --- a/ee/spec/services/groups/update_service_spec.rb +++ b/ee/spec/services/groups/update_service_spec.rb @@ -130,6 +130,30 @@ def update_file_template_project_id(id) end end + context 'repository_size_limit assignment as Bytes' do + let(:group) { create(:group, :public, repository_size_limit: 0) } + + context 'when param present' do + let(:opts) { { repository_size_limit: '100' } } + + it 'converts from MB to Bytes' do + update_group(group, user, opts) + + expect(group.reload.repository_size_limit).to eql(100 * 1024 * 1024) + end + end + + context 'when param not present' do + let(:opts) { { repository_size_limit: '' } } + + it 'assign nil value' do + update_group(group, user, opts) + + expect(group.reload.repository_size_limit).to be_nil + end + end + end + def update_group(group, user, opts) Groups::UpdateService.new(group, user, opts).execute end diff --git a/spec/services/groups/create_service_spec.rb b/spec/services/groups/create_service_spec.rb index 188780fe5aaf52fc3c8f7ea4e1d49a025453fe84..fe6a8691ae0cb1a6f686874d47c4db8a81744c7e 100644 --- a/spec/services/groups/create_service_spec.rb +++ b/spec/services/groups/create_service_spec.rb @@ -88,31 +88,6 @@ end end - context 'repository_size_limit assignment as Bytes' do - let(:admin_user) { create(:user, admin: true) } - let(:service) { described_class.new(admin_user, group_params.merge(opts)) } - - context 'when param present' do - let(:opts) { { repository_size_limit: '100' } } - - it 'assign repository_size_limit as Bytes' do - group = service.execute - - expect(group.repository_size_limit).to eql(100 * 1024 * 1024) - end - end - - context 'when param not present' do - let(:opts) { { repository_size_limit: '' } } - - it 'assign nil value' do - group = service.execute - - expect(group.repository_size_limit).to be_nil - end - end - end - describe 'creating a mattermost team' do let!(:params) { group_params.merge(create_chat_team: "true") } let!(:service) { described_class.new(user, params) } diff --git a/spec/services/groups/update_service_spec.rb b/spec/services/groups/update_service_spec.rb index 03a2a05e39ab316469ffd10ebad02f7471504c47..d87a7dd234d594035587e7adda713289ec3592b5 100644 --- a/spec/services/groups/update_service_spec.rb +++ b/spec/services/groups/update_service_spec.rb @@ -82,31 +82,6 @@ end end - context 'repository_size_limit assignment as Bytes' do - let(:group) { create(:group, :public, repository_size_limit: 0) } - let(:service) { described_class.new(group, user, opts) } - - context 'when param present' do - let(:opts) { { repository_size_limit: '100' } } - - it 'converts from MB to Bytes' do - service.execute - - expect(group.reload.repository_size_limit).to eql(100 * 1024 * 1024) - end - end - - context 'when param not present' do - let(:opts) { { repository_size_limit: '' } } - - it 'assign nil value' do - service.execute - - expect(group.reload.repository_size_limit).to be_nil - end - end - end - context "unauthorized visibility_level validation" do let!(:service) { described_class.new(internal_group, user, visibility_level: 99) } before do