summaryrefslogtreecommitdiff
path: root/spec/helpers/issuables_description_templates_helper_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/helpers/issuables_description_templates_helper_spec.rb')
-rw-r--r--spec/helpers/issuables_description_templates_helper_spec.rb93
1 files changed, 93 insertions, 0 deletions
diff --git a/spec/helpers/issuables_description_templates_helper_spec.rb b/spec/helpers/issuables_description_templates_helper_spec.rb
new file mode 100644
index 00000000000..42643b755f8
--- /dev/null
+++ b/spec/helpers/issuables_description_templates_helper_spec.rb
@@ -0,0 +1,93 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe IssuablesDescriptionTemplatesHelper, :clean_gitlab_redis_cache do
+ include_context 'project issuable templates context'
+
+ describe '#issuable_templates' do
+ let_it_be(:inherited_from) { nil }
+ let_it_be(:user) { create(:user) }
+ let_it_be(:parent_group, reload: true) { create(:group) }
+ let_it_be(:project, reload: true) { create(:project, :custom_repo, files: issuable_template_files) }
+ let_it_be(:group_member) { create(:group_member, :developer, group: parent_group, user: user) }
+ let_it_be(:project_member) { create(:project_member, :developer, user: user, project: project) }
+
+ it 'returns empty hash when template type does not exist' do
+ expect(helper.issuable_templates(build(:project), 'non-existent-template-type')).to eq([])
+ end
+
+ context 'with cached issuable templates' do
+ before do
+ allow(Gitlab::Template::IssueTemplate).to receive(:template_names).and_return({})
+ allow(Gitlab::Template::MergeRequestTemplate).to receive(:template_names).and_return({})
+
+ helper.issuable_templates(project, 'issues')
+ helper.issuable_templates(project, 'merge_request')
+ end
+
+ it 'does not call TemplateFinder' do
+ expect(Gitlab::Template::IssueTemplate).not_to receive(:template_names)
+ expect(Gitlab::Template::MergeRequestTemplate).not_to receive(:template_names)
+ helper.issuable_templates(project, 'issues')
+ helper.issuable_templates(project, 'merge_request')
+ end
+ end
+
+ context 'when project has no parent group' do
+ it_behaves_like 'project issuable templates'
+ end
+
+ context 'when project has parent group' do
+ before do
+ project.update!(group: parent_group)
+ end
+
+ context 'when project parent group does not have a file template project' do
+ it_behaves_like 'project issuable templates'
+ end
+
+ context 'when project parent group has a file template project' do
+ let_it_be(:file_template_project) { create(:project, :custom_repo, group: parent_group, files: issuable_template_files) }
+ let_it_be(:group, reload: true) { create(:group, parent: parent_group) }
+ let_it_be(:project, reload: true) { create(:project, :custom_repo, group: group, files: issuable_template_files) }
+
+ before do
+ project.update!(group: group)
+ parent_group.update_columns(file_template_project_id: file_template_project.id)
+ end
+
+ it_behaves_like 'project issuable templates'
+ end
+ end
+ end
+
+ describe '#issuable_templates_names' do
+ let(:project) { double(Project, id: 21) }
+
+ let(:templates) do
+ [
+ { name: "another_issue_template", id: "another_issue_template", project_id: project.id },
+ { name: "custom_issue_template", id: "custom_issue_template", project_id: project.id }
+ ]
+ end
+
+ it 'returns project templates only' do
+ allow(helper).to receive(:ref_project).and_return(project)
+ allow(helper).to receive(:issuable_templates).and_return(templates)
+
+ expect(helper.issuable_templates_names(Issue.new)).to eq(%w[another_issue_template custom_issue_template])
+ end
+
+ context 'when there are not templates in the project' do
+ let(:templates) { {} }
+
+ it 'returns empty array' do
+ allow(helper).to receive(:ref_project).and_return(project)
+ allow(helper).to receive(:issuable_templates).and_return(templates)
+
+ expect(helper.issuable_templates_names(Issue.new)).to eq([])
+ end
+ end
+ end
+end