2019-09-30 08:06:01 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2013-02-11 08:32:29 -05:00
|
|
|
require "spec_helper"
|
|
|
|
|
2020-06-24 05:08:32 -04:00
|
|
|
RSpec.describe IssuesHelper do
|
2017-08-02 15:55:11 -04:00
|
|
|
let(:project) { create(:project) }
|
2013-02-19 08:21:59 -05:00
|
|
|
let(:issue) { create :issue, project: project }
|
|
|
|
let(:ext_project) { create :redmine_project }
|
2013-02-11 08:32:29 -05:00
|
|
|
|
2016-06-19 19:06:57 -04:00
|
|
|
describe '#award_user_list' do
|
2016-10-06 14:00:51 -04:00
|
|
|
it "returns a comma-separated list of the first X users" do
|
|
|
|
user = build_stubbed(:user, name: 'Joe')
|
|
|
|
awards = Array.new(3, build_stubbed(:award_emoji, user: user))
|
2016-06-19 19:06:57 -04:00
|
|
|
|
2017-02-22 17:54:59 -05:00
|
|
|
expect(award_user_list(awards, nil, limit: 3))
|
|
|
|
.to eq('Joe, Joe, and Joe')
|
2016-06-19 19:06:57 -04:00
|
|
|
end
|
|
|
|
|
2016-07-11 14:53:07 -04:00
|
|
|
it "displays the current user's name as 'You'" do
|
2016-10-06 14:00:51 -04:00
|
|
|
user = build_stubbed(:user, name: 'Joe')
|
|
|
|
award = build_stubbed(:award_emoji, user: user)
|
2016-06-19 19:06:57 -04:00
|
|
|
|
2016-10-06 14:00:51 -04:00
|
|
|
expect(award_user_list([award], user)).to eq('You')
|
|
|
|
expect(award_user_list([award], nil)).to eq 'Joe'
|
2016-06-21 18:22:03 -04:00
|
|
|
end
|
|
|
|
|
2016-10-06 14:00:51 -04:00
|
|
|
it "truncates lists" do
|
|
|
|
user = build_stubbed(:user, name: 'Jane')
|
|
|
|
awards = Array.new(5, build_stubbed(:award_emoji, user: user))
|
|
|
|
|
2017-02-22 17:54:59 -05:00
|
|
|
expect(award_user_list(awards, nil, limit: 3))
|
|
|
|
.to eq('Jane, Jane, Jane, and 2 more.')
|
2016-06-21 18:22:03 -04:00
|
|
|
end
|
|
|
|
|
2016-10-06 14:00:51 -04:00
|
|
|
it "displays the current user in front of other users" do
|
|
|
|
current_user = build_stubbed(:user)
|
|
|
|
my_award = build_stubbed(:award_emoji, user: current_user)
|
|
|
|
award = build_stubbed(:award_emoji, user: build_stubbed(:user, name: 'Jane'))
|
|
|
|
awards = Array.new(5, award).push(my_award)
|
|
|
|
|
2017-06-21 09:48:12 -04:00
|
|
|
expect(award_user_list(awards, current_user, limit: 2))
|
|
|
|
.to eq("You, Jane, and 4 more.")
|
2016-06-19 19:06:57 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-12-30 09:44:21 -05:00
|
|
|
describe '#award_state_class' do
|
2016-04-16 15:09:08 -04:00
|
|
|
let!(:upvote) { create(:award_emoji) }
|
2018-04-06 14:19:37 -04:00
|
|
|
let(:awardable) { upvote.awardable }
|
|
|
|
let(:user) { upvote.user }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(helper).to receive(:can?) do |*args|
|
|
|
|
Ability.allowed?(*args)
|
|
|
|
end
|
|
|
|
end
|
2015-11-19 07:41:05 -05:00
|
|
|
|
2016-12-30 09:44:21 -05:00
|
|
|
it "returns disabled string for unauthenticated user" do
|
2018-04-06 14:19:37 -04:00
|
|
|
expect(helper.award_state_class(awardable, AwardEmoji.all, nil)).to eq("disabled")
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns disabled for a user that does not have access to the awardable" do
|
|
|
|
expect(helper.award_state_class(awardable, AwardEmoji.all, build(:user))).to eq("disabled")
|
2015-11-19 07:41:05 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
it "returns active string for author" do
|
2018-04-06 14:19:37 -04:00
|
|
|
expect(helper.award_state_class(awardable, AwardEmoji.all, upvote.user)).to eq("active")
|
|
|
|
end
|
|
|
|
|
|
|
|
it "is blank for a user that has access to the awardable" do
|
|
|
|
user = build(:user)
|
|
|
|
expect(helper).to receive(:can?).with(user, :award_emoji, awardable).and_return(true)
|
|
|
|
|
|
|
|
expect(helper.award_state_class(awardable, AwardEmoji.all, user)).to be_blank
|
2015-11-19 07:41:05 -05:00
|
|
|
end
|
|
|
|
end
|
2015-12-25 05:08:53 -05:00
|
|
|
|
2016-04-07 16:04:09 -04:00
|
|
|
describe "awards_sort" do
|
2015-12-25 05:08:53 -05:00
|
|
|
it "sorts a hash so thumbsup and thumbsdown are always on top" do
|
2015-12-25 08:46:01 -05:00
|
|
|
data = { "thumbsdown" => "some value", "lifter" => "some value", "thumbsup" => "some value" }
|
2017-02-22 12:46:57 -05:00
|
|
|
expect(awards_sort(data).keys).to eq(%w(thumbsup thumbsdown lifter))
|
2015-12-25 05:08:53 -05:00
|
|
|
end
|
|
|
|
end
|
2016-04-06 10:02:22 -04:00
|
|
|
|
2017-03-08 10:39:20 -05:00
|
|
|
describe "#link_to_discussions_to_resolve" do
|
|
|
|
describe "passing only a merge request" do
|
|
|
|
let(:merge_request) { create(:merge_request) }
|
|
|
|
|
|
|
|
it "links just the merge request" do
|
2017-06-29 13:06:35 -04:00
|
|
|
expected_path = project_merge_request_path(merge_request.project, merge_request)
|
2017-03-08 10:39:20 -05:00
|
|
|
|
|
|
|
expect(link_to_discussions_to_resolve(merge_request, nil)).to include(expected_path)
|
|
|
|
end
|
|
|
|
|
2019-09-13 09:26:31 -04:00
|
|
|
it "contains the reference to the merge request" do
|
2017-03-08 10:39:20 -05:00
|
|
|
expect(link_to_discussions_to_resolve(merge_request, nil)).to include(merge_request.to_reference)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "when passing a discussion" do
|
2019-01-16 07:09:29 -05:00
|
|
|
let(:diff_note) { create(:diff_note_on_merge_request) }
|
2017-03-08 10:39:20 -05:00
|
|
|
let(:merge_request) { diff_note.noteable }
|
2017-03-17 15:25:52 -04:00
|
|
|
let(:discussion) { diff_note.to_discussion }
|
2017-03-08 10:39:20 -05:00
|
|
|
|
|
|
|
it "links to the merge request with first note if a single discussion was passed" do
|
|
|
|
expected_path = Gitlab::UrlBuilder.build(diff_note)
|
|
|
|
|
|
|
|
expect(link_to_discussions_to_resolve(merge_request, discussion)).to include(expected_path)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "contains both the reference to the merge request and a mention of the discussion" do
|
|
|
|
expect(link_to_discussions_to_resolve(merge_request, discussion)).to include("#{merge_request.to_reference} (discussion #{diff_note.id})")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2018-04-10 13:49:26 -04:00
|
|
|
|
|
|
|
describe '#show_new_issue_link?' do
|
|
|
|
before do
|
|
|
|
allow(helper).to receive(:current_user)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'is false when no project there is no project' do
|
|
|
|
expect(helper.show_new_issue_link?(nil)).to be_falsey
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'is true when there is a project and no logged in user' do
|
|
|
|
expect(helper.show_new_issue_link?(build(:project))).to be_truthy
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'is true when the current user does not have access to the project' do
|
|
|
|
project = build(:project)
|
|
|
|
allow(helper).to receive(:current_user).and_return(project.owner)
|
|
|
|
|
|
|
|
expect(helper).to receive(:can?).with(project.owner, :create_issue, project).and_return(true)
|
|
|
|
expect(helper.show_new_issue_link?(project)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
2019-09-13 09:26:31 -04:00
|
|
|
|
|
|
|
describe '#issue_closed_link' do
|
|
|
|
let(:new_issue) { create(:issue, project: project) }
|
|
|
|
let(:guest) { create(:user) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(helper).to receive(:can?) do |*args|
|
|
|
|
Ability.allowed?(*args)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'successfully displays link to issue and with css class' do |action|
|
|
|
|
it 'returns link' do
|
2020-03-02 10:08:01 -05:00
|
|
|
link = "<a class=\"#{css_class}\" href=\"/#{new_issue.project.full_path}/-/issues/#{new_issue.iid}\">(#{action})</a>"
|
2019-09-13 09:26:31 -04:00
|
|
|
|
|
|
|
expect(helper.issue_closed_link(issue, user, css_class: css_class)).to match(link)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'does not display link' do
|
|
|
|
it 'returns nil' do
|
|
|
|
expect(helper.issue_closed_link(issue, user)).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with linked issue' do
|
|
|
|
context 'with moved issue' do
|
|
|
|
before do
|
2020-07-27 08:09:50 -04:00
|
|
|
issue.update!(moved_to: new_issue)
|
2019-09-13 09:26:31 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'when user has permission to see new issue' do
|
|
|
|
let(:user) { project.owner }
|
|
|
|
let(:css_class) { 'text-white text-underline' }
|
|
|
|
|
|
|
|
it_behaves_like 'successfully displays link to issue and with css class', 'moved'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when user has no permission to see new issue' do
|
|
|
|
let(:user) { guest }
|
|
|
|
|
|
|
|
it_behaves_like 'does not display link'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with duplicated issue' do
|
|
|
|
before do
|
2020-07-27 08:09:50 -04:00
|
|
|
issue.update!(duplicated_to: new_issue)
|
2019-09-13 09:26:31 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'when user has permission to see new issue' do
|
|
|
|
let(:user) { project.owner }
|
|
|
|
let(:css_class) { 'text-white text-underline' }
|
|
|
|
|
|
|
|
it_behaves_like 'successfully displays link to issue and with css class', 'duplicated'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when user has no permission to see new issue' do
|
|
|
|
let(:user) { guest }
|
|
|
|
|
|
|
|
it_behaves_like 'does not display link'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'without linked issue' do
|
|
|
|
let(:user) { project.owner }
|
|
|
|
|
|
|
|
before do
|
2020-07-27 08:09:50 -04:00
|
|
|
issue.update!(moved_to: nil, duplicated_to: nil)
|
2019-09-13 09:26:31 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'does not display link'
|
|
|
|
end
|
|
|
|
end
|
2020-07-15 17:09:26 -04:00
|
|
|
|
|
|
|
describe '#show_moved_service_desk_issue_warning?' do
|
|
|
|
let(:project1) { create(:project, service_desk_enabled: true) }
|
|
|
|
let(:project2) { create(:project, service_desk_enabled: true) }
|
|
|
|
let!(:old_issue) { create(:issue, author: User.support_bot, project: project1) }
|
|
|
|
let!(:new_issue) { create(:issue, author: User.support_bot, project: project2) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(Gitlab::IncomingEmail).to receive(:enabled?) { true }
|
|
|
|
allow(Gitlab::IncomingEmail).to receive(:supports_wildcard?) { true }
|
|
|
|
|
2020-07-27 08:09:50 -04:00
|
|
|
old_issue.update!(moved_to: new_issue)
|
2020-07-15 17:09:26 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'is true when moved issue project has service desk disabled' do
|
|
|
|
project2.update!(service_desk_enabled: false)
|
|
|
|
|
|
|
|
expect(helper.show_moved_service_desk_issue_warning?(new_issue)).to be(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'is false when moved issue project has service desk enabled' do
|
|
|
|
expect(helper.show_moved_service_desk_issue_warning?(new_issue)).to be(false)
|
|
|
|
end
|
|
|
|
end
|
2020-09-21 05:09:29 -04:00
|
|
|
|
|
|
|
describe '#use_startup_call' do
|
|
|
|
it "returns false when a query param is present" do
|
|
|
|
allow(controller.request).to receive(:query_parameters).and_return({ foo: 'bar' })
|
|
|
|
|
|
|
|
expect(helper.use_startup_call?).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns false when user has stored sort preference" do
|
|
|
|
controller.instance_variable_set(:@sort, 'updated_asc')
|
|
|
|
|
|
|
|
expect(helper.use_startup_call?).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true when request.query_parameters is empty with default sorting preference' do
|
|
|
|
controller.instance_variable_set(:@sort, 'created_date')
|
|
|
|
allow(controller.request).to receive(:query_parameters).and_return({})
|
|
|
|
|
|
|
|
expect(helper.use_startup_call?).to eq(true)
|
|
|
|
end
|
|
|
|
end
|
2013-02-11 08:32:29 -05:00
|
|
|
end
|