diff options
author | Michael Kozono <mkozono@gmail.com> | 2017-09-01 16:49:09 -0700 |
---|---|---|
committer | Michael Kozono <mkozono@gmail.com> | 2017-09-06 12:07:20 -0700 |
commit | 1cc7f4a45d9e9fdf1943eb92d3cd2071ba497337 (patch) | |
tree | cdaebcf4f86d442a00a45d7a6bab4584a7b48dbd /spec/models/namespace_spec.rb | |
parent | 8437a24ff46220e2813ce3d82a52062dd259276b (diff) | |
download | gitlab-ce-1cc7f4a45d9e9fdf1943eb92d3cd2071ba497337.tar.gz |
Enable share_with_group_lock on subgroup
…when needed
Diffstat (limited to 'spec/models/namespace_spec.rb')
-rw-r--r-- | spec/models/namespace_spec.rb | 112 |
1 files changed, 112 insertions, 0 deletions
diff --git a/spec/models/namespace_spec.rb b/spec/models/namespace_spec.rb index 69286eff984..2bfe04fecc7 100644 --- a/spec/models/namespace_spec.rb +++ b/spec/models/namespace_spec.rb @@ -406,4 +406,116 @@ describe Namespace do it { expect(group.all_projects.to_a).to eq([project2, project1]) } end + + describe '#share_with_group_lock with subgroups' do + context 'when creating a subgroup' do + let(:subgroup) { create(:group, parent: root_group )} + + context 'under a parent with share lock enabled' do + let(:root_group) { create(:group, share_with_group_lock: true) } + + it 'enables share lock on the subgroup' do + expect(subgroup.share_with_group_lock).to be_truthy + end + end + + context 'under a parent with share lock disabled' do + let(:root_group) { create(:group) } + + it 'does not enable share lock on the subgroup' do + expect(subgroup.share_with_group_lock).to be_falsey + end + end + end + + context 'when enabling the parent group share lock' do + let(:root_group) { create(:group) } + let!(:subgroup) { create(:group, parent: root_group )} + + it 'the subgroup share lock becomes enabled' do + root_group.update(share_with_group_lock: true) + + expect(subgroup.reload.share_with_group_lock).to be_truthy + end + end + + context 'when disabling the parent group share lock (which was already enabled)' do + let(:root_group) { create(:group, share_with_group_lock: true) } + + context 'and the subgroup share lock is enabled' do + let(:subgroup) { create(:group, parent: root_group, share_with_group_lock: true )} + + it 'the subgroup share lock does not change' do + root_group.update(share_with_group_lock: false) + + expect(subgroup.reload.share_with_group_lock).to be_truthy + end + end + + context 'but the subgroup share lock is disabled' do + let(:subgroup) { create(:group, parent: root_group )} + + it 'the subgroup share lock does not change' do + root_group.update(share_with_group_lock: false) + + expect(subgroup.reload.share_with_group_lock?).to be_falsey + end + end + end + + # Note: Group transfers are not yet implemented + context 'when a group is transferred into a root group' do + context 'when the root group share lock is enabled' do + let(:root_group) { create(:group, share_with_group_lock: true) } + + context 'when the subgroup share lock is enabled' do + let(:subgroup) { create(:group, share_with_group_lock: true )} + + it 'the subgroup share lock does not change' do + subgroup.parent = root_group + subgroup.save! + + expect(subgroup.share_with_group_lock).to be_truthy + end + end + + context 'when the subgroup share lock is disabled' do + let(:subgroup) { create(:group)} + + it 'the subgroup share lock becomes enabled' do + subgroup.parent = root_group + subgroup.save! + + expect(subgroup.share_with_group_lock).to be_truthy + end + end + end + + context 'when the root group share lock is disabled' do + let(:root_group) { create(:group) } + + context 'when the subgroup share lock is enabled' do + let(:subgroup) { create(:group, share_with_group_lock: true )} + + it 'the subgroup share lock does not change' do + subgroup.parent = root_group + subgroup.save! + + expect(subgroup.share_with_group_lock).to be_truthy + end + end + + context 'when the subgroup share lock is disabled' do + let(:subgroup) { create(:group)} + + it 'the subgroup share lock does not change' do + subgroup.parent = root_group + subgroup.save! + + expect(subgroup.share_with_group_lock).to be_falsey + end + end + end + end + end end |