Commit 7848c4da authored by Fabio Papa's avatar Fabio Papa

Clean up the show_spec examples previously added

parent 42167779
...@@ -56,27 +56,28 @@ describe 'Group show page' do ...@@ -56,27 +56,28 @@ describe 'Group show page' do
end end
context 'subgroup support' do context 'subgroup support' do
let(:restricted_group) { create(:group,
subgroup_creation_level: ::Gitlab::Access::OWNER_SUBGROUP_ACCESS) }
let(:relaxed_group) { create(:group,
subgroup_creation_level: ::Gitlab::Access::MAINTAINER_SUBGROUP_ACCESS) }
let(:owner) { create(:user) } let(:owner) { create(:user) }
let(:maintainer) { create(:user) } let(:maintainer) { create(:user) }
before do
group.add_owner(owner)
group.add_maintainer(maintainer)
end
context 'for owners' do context 'for owners' do
let(:path) { group_path(restricted_group) }
before do before do
restricted_group.add_owner(owner)
sign_in(owner) sign_in(owner)
end end
context 'when subgroups are supported', :js, :nested_groups do context 'when subgroups are supported', :js, :nested_groups do
before do before do
allow(Group).to receive(:supports_nested_objects?) { true } allow(Group).to receive(:supports_nested_objects?) { true }
visit path
end end
it 'allows creating subgroups' do it 'allows creating subgroups' do
visit path
expect(page) expect(page)
.to have_css("li[data-text='New subgroup']", visible: false) .to have_css("li[data-text='New subgroup']", visible: false)
end end
...@@ -85,11 +86,10 @@ describe 'Group show page' do ...@@ -85,11 +86,10 @@ describe 'Group show page' do
context 'when subgroups are not supported' do context 'when subgroups are not supported' do
before do before do
allow(Group).to receive(:supports_nested_objects?) { false } allow(Group).to receive(:supports_nested_objects?) { false }
visit path
end end
it 'does not allow creating subgroups' do it 'does not allow creating subgroups' do
visit path
expect(page) expect(page)
.not_to have_selector("li[data-text='New subgroup']", visible: false) .not_to have_selector("li[data-text='New subgroup']", visible: false)
end end
...@@ -107,30 +107,30 @@ describe 'Group show page' do ...@@ -107,30 +107,30 @@ describe 'Group show page' do
end end
context 'when subgroup_creation_level is set to maintainers' do context 'when subgroup_creation_level is set to maintainers' do
let(:group) do before do
create(:group, relaxed_group.add_maintainer(maintainer)
subgroup_creation_level: ::Gitlab::Access::MAINTAINER_SUBGROUP_ACCESS) path = group_path(relaxed_group)
visit path
end end
it 'allows creating subgroups' do it 'allows creating subgroups' do
visit path
expect(page) expect(page)
.to have_css("li[data-text='New subgroup']", visible: false) .to have_css("li[data-text='New subgroup']", visible: false)
end end
end end
context 'when subgroup_creation_level is set to owners' do context 'when subgroup_creation_level is set to owners' do
let(:group) do before do
create(:group, restricted_group.add_maintainer(maintainer)
subgroup_creation_level: ::Gitlab::Access::OWNER_SUBGROUP_ACCESS)
end end
it 'does not allow creating subgroups' do it 'does not allow creating subgroups' do
path = group_path(restricted_group)
visit path visit path
expect(page) expect(page)
.not_to have_css("li[data-text='New subgroup']", visible: false) .not_to have_selector("li[data-text='New subgroup']",
visible: false)
end end
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