diff options
Diffstat (limited to 'spec/helpers/groups_helper_spec.rb')
-rw-r--r-- | spec/helpers/groups_helper_spec.rb | 72 |
1 files changed, 11 insertions, 61 deletions
diff --git a/spec/helpers/groups_helper_spec.rb b/spec/helpers/groups_helper_spec.rb index a38483a956d..8b4ac6a7cfd 100644 --- a/spec/helpers/groups_helper_spec.rb +++ b/spec/helpers/groups_helper_spec.rb @@ -287,39 +287,6 @@ RSpec.describe GroupsHelper do end end - describe '#parent_group_options' do - let_it_be(:current_user) { create(:user) } - let_it_be(:group) { create(:group, name: 'group') } - let_it_be(:group2) { create(:group, name: 'group2') } - - before do - group.add_owner(current_user) - group2.add_owner(current_user) - end - - it 'includes explicitly owned groups except self' do - expect(parent_group_options(group2)).to eq([{ id: group.id, text: group.human_name }].to_json) - end - - it 'excludes parent group' do - subgroup = create(:group, parent: group2) - - expect(parent_group_options(subgroup)).to eq([{ id: group.id, text: group.human_name }].to_json) - end - - it 'includes subgroups with inherited ownership' do - subgroup = create(:group, parent: group) - - expect(parent_group_options(group2)).to eq([{ id: group.id, text: group.human_name }, { id: subgroup.id, text: subgroup.human_name }].to_json) - end - - it 'excludes own subgroups' do - create(:group, parent: group2) - - expect(parent_group_options(group2)).to eq([{ id: group.id, text: group.human_name }].to_json) - end - end - describe '#can_disable_group_emails?' do let_it_be(:current_user) { create(:user) } let_it_be(:group) { create(:group, name: 'group') } @@ -502,32 +469,6 @@ RSpec.describe GroupsHelper do end end - describe '#subgroups_and_projects_list_app_data' do - let_it_be(:group) { create(:group) } - let_it_be(:user) { create(:user) } - - before do - allow(helper).to receive(:current_user).and_return(user) - - allow(helper).to receive(:can?).with(user, :create_subgroup, group) { true } - allow(helper).to receive(:can?).with(user, :create_projects, group) { true } - end - - it 'returns expected hash' do - expect(helper.subgroups_and_projects_list_app_data(group)).to match({ - show_schema_markup: 'true', - new_subgroup_path: including("groups/new?parent_id=#{group.id}#create-group-pane"), - new_project_path: including("/projects/new?namespace_id=#{group.id}"), - new_subgroup_illustration: including('illustrations/subgroup-create-new-sm'), - new_project_illustration: including('illustrations/project-create-new-sm'), - empty_subgroup_illustration: including('illustrations/empty-state/empty-subgroup-md'), - render_empty_state: 'true', - can_create_subgroups: 'true', - can_create_projects: 'true' - }) - end - end - describe '#group_overview_tabs_app_data' do let_it_be(:group) { create(:group) } let_it_be(:user) { create(:user) } @@ -548,8 +489,17 @@ RSpec.describe GroupsHelper do shared_projects_endpoint: including("/groups/#{group.path}/-/shared_projects.json"), archived_projects_endpoint: including("/groups/#{group.path}/-/children.json?archived=only"), current_group_visibility: group.visibility, - initial_sort: initial_sort - }.merge(helper.group_overview_tabs_app_data(group)) + initial_sort: initial_sort, + show_schema_markup: 'true', + new_subgroup_path: including("groups/new?parent_id=#{group.id}#create-group-pane"), + new_project_path: including("/projects/new?namespace_id=#{group.id}"), + new_subgroup_illustration: including('illustrations/subgroup-create-new-sm'), + new_project_illustration: including('illustrations/project-create-new-sm'), + empty_subgroup_illustration: including('illustrations/empty-state/empty-subgroup-md'), + render_empty_state: 'true', + can_create_subgroups: 'true', + can_create_projects: 'true' + } ) end end |