2019-12-21 20:55:43 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2020-05-24 23:13:21 +05:30
|
|
|
require 'spec_helper'
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2020-07-28 23:09:34 +05:30
|
|
|
RSpec.describe TodosHelper do
|
2020-05-24 23:13:21 +05:30
|
|
|
let_it_be(:user) { create(:user) }
|
|
|
|
let_it_be(:author) { create(:user) }
|
2022-08-13 15:12:31 +05:30
|
|
|
let_it_be(:project) { create(:project) }
|
|
|
|
let_it_be(:issue) { create(:issue, title: 'Issue 1', project: project) }
|
2020-05-24 23:13:21 +05:30
|
|
|
let_it_be(:design) { create(:design, issue: issue) }
|
|
|
|
let_it_be(:note) do
|
|
|
|
create(:note,
|
|
|
|
project: issue.project,
|
|
|
|
note: 'I am note, hear me roar')
|
|
|
|
end
|
2021-01-29 00:20:46 +05:30
|
|
|
|
2020-05-24 23:13:21 +05:30
|
|
|
let_it_be(:design_todo) do
|
|
|
|
create(:todo, :mentioned,
|
|
|
|
user: user,
|
2022-08-13 15:12:31 +05:30
|
|
|
project: project,
|
2020-05-24 23:13:21 +05:30
|
|
|
target: design,
|
|
|
|
author: author,
|
|
|
|
note: note)
|
|
|
|
end
|
2021-01-29 00:20:46 +05:30
|
|
|
|
2020-06-23 00:09:42 +05:30
|
|
|
let_it_be(:alert_todo) do
|
|
|
|
alert = create(:alert_management_alert, iid: 1001)
|
|
|
|
create(:todo, target: alert)
|
|
|
|
end
|
2020-05-24 23:13:21 +05:30
|
|
|
|
2022-08-13 15:12:31 +05:30
|
|
|
let_it_be(:task_todo) do
|
|
|
|
task = create(:work_item, :task, project: project)
|
|
|
|
create(:todo, target: task, target_type: task.class.name, project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
let_it_be(:issue_todo) do
|
|
|
|
create(:todo, target: issue)
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
describe '#todos_count_format' do
|
|
|
|
it 'shows fuzzy count for 100 or more items' do
|
|
|
|
expect(helper.todos_count_format(100)).to eq '99+'
|
|
|
|
expect(helper.todos_count_format(1000)).to eq '99+'
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'shows exact count for 99 or fewer items' do
|
|
|
|
expect(helper.todos_count_format(99)).to eq '99'
|
|
|
|
expect(helper.todos_count_format(50)).to eq '50'
|
|
|
|
expect(helper.todos_count_format(1)).to eq '1'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-05-24 23:13:21 +05:30
|
|
|
describe '#todo_target_link' do
|
|
|
|
context 'when given a design' do
|
|
|
|
let(:todo) { design_todo }
|
|
|
|
|
|
|
|
it 'produces a good link' do
|
|
|
|
path = helper.todo_target_path(todo)
|
|
|
|
link = helper.todo_target_link(todo)
|
|
|
|
expected = "<a href=\"#{path}\">design #{design.to_reference}</a>"
|
|
|
|
|
|
|
|
expect(link).to eq(expected)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-06-23 00:09:42 +05:30
|
|
|
describe '#todo_target_title' do
|
|
|
|
context 'when the target does not exist' do
|
|
|
|
let(:todo) { double('Todo', target: nil) }
|
|
|
|
|
|
|
|
it 'returns an empty string' do
|
|
|
|
title = helper.todo_target_title(todo)
|
|
|
|
expect(title).to eq("")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when given a design todo' do
|
|
|
|
let(:todo) { design_todo }
|
|
|
|
|
|
|
|
it 'returns an empty string' do
|
|
|
|
title = helper.todo_target_title(todo)
|
|
|
|
expect(title).to eq("")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when given a non-design todo' do
|
|
|
|
let(:todo) do
|
|
|
|
build_stubbed(:todo, :assigned,
|
|
|
|
user: user,
|
|
|
|
project: issue.project,
|
|
|
|
target: issue,
|
|
|
|
author: author)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns the title' do
|
|
|
|
title = helper.todo_target_title(todo)
|
|
|
|
expect(title).to eq("\"Issue 1\"")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-05-24 23:13:21 +05:30
|
|
|
describe '#todo_target_path' do
|
|
|
|
context 'when given a design' do
|
|
|
|
let(:todo) { design_todo }
|
|
|
|
|
|
|
|
it 'responds with an appropriate path' do
|
|
|
|
path = helper.todo_target_path(todo)
|
|
|
|
issue_path = Gitlab::Routing.url_helpers
|
|
|
|
.project_issue_path(issue.project, issue)
|
|
|
|
|
|
|
|
expect(path).to eq("#{issue_path}/designs/#{design.filename}##{dom_id(design_todo.note)}")
|
|
|
|
end
|
|
|
|
end
|
2020-06-23 00:09:42 +05:30
|
|
|
|
|
|
|
context 'when given an alert' do
|
|
|
|
let(:todo) { alert_todo }
|
|
|
|
|
|
|
|
it 'responds with an appropriate path' do
|
|
|
|
path = helper.todo_target_path(todo)
|
|
|
|
|
|
|
|
expect(path).to eq(
|
|
|
|
"/#{todo.project.full_path}/-/alert_management/#{todo.target.iid}/details"
|
|
|
|
)
|
|
|
|
end
|
|
|
|
end
|
2022-08-13 15:12:31 +05:30
|
|
|
|
|
|
|
context 'when given a task' do
|
|
|
|
let(:todo) { task_todo }
|
|
|
|
|
|
|
|
it 'responds with an appropriate path' do
|
|
|
|
path = helper.todo_target_path(todo)
|
|
|
|
|
|
|
|
expect(path).to eq("/#{todo.project.full_path}/-/work_items/#{todo.target.id}")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when given an issue with a note anchor' do
|
|
|
|
let(:todo) { create(:todo, project: issue.project, target: issue, note: note) }
|
|
|
|
|
|
|
|
it 'responds with an appropriate path' do
|
|
|
|
path = helper.todo_target_path(todo)
|
|
|
|
|
|
|
|
expect(path).to eq("/#{issue.project.full_path}/-/issues/#{issue.iid}##{dom_id(note)}")
|
|
|
|
end
|
|
|
|
end
|
2020-05-24 23:13:21 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
describe '#todo_target_type_name' do
|
2022-08-13 15:12:31 +05:30
|
|
|
subject { helper.todo_target_type_name(todo) }
|
|
|
|
|
2020-05-24 23:13:21 +05:30
|
|
|
context 'when given a design todo' do
|
|
|
|
let(:todo) { design_todo }
|
|
|
|
|
2022-08-13 15:12:31 +05:30
|
|
|
it { is_expected.to eq('design') }
|
2020-05-24 23:13:21 +05:30
|
|
|
end
|
2020-06-23 00:09:42 +05:30
|
|
|
|
|
|
|
context 'when given an alert todo' do
|
|
|
|
let(:todo) { alert_todo }
|
|
|
|
|
2022-08-13 15:12:31 +05:30
|
|
|
it { is_expected.to eq('alert') }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when given a task todo' do
|
|
|
|
let(:todo) { task_todo }
|
2020-06-23 00:09:42 +05:30
|
|
|
|
2022-08-13 15:12:31 +05:30
|
|
|
it { is_expected.to eq('task') }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when given an issue todo' do
|
|
|
|
let(:todo) { issue_todo }
|
|
|
|
|
|
|
|
it { is_expected.to eq('issue') }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when given a merge request todo' do
|
|
|
|
let(:todo) do
|
|
|
|
merge_request = create(:merge_request, source_project: project)
|
|
|
|
create(:todo, target: merge_request)
|
2020-06-23 00:09:42 +05:30
|
|
|
end
|
2022-08-13 15:12:31 +05:30
|
|
|
|
|
|
|
it { is_expected.to eq('merge request') }
|
2020-06-23 00:09:42 +05:30
|
|
|
end
|
2020-05-24 23:13:21 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
describe '#todo_types_options' do
|
|
|
|
it 'includes a match for a design todo' do
|
|
|
|
options = helper.todo_types_options
|
|
|
|
design_option = options.find { |o| o[:id] == design_todo.target_type }
|
|
|
|
|
|
|
|
expect(design_option).to include(text: 'Design')
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
2020-07-28 23:09:34 +05:30
|
|
|
|
|
|
|
describe '#todo_target_state_pill' do
|
|
|
|
subject { helper.todo_target_state_pill(todo) }
|
|
|
|
|
|
|
|
shared_examples 'a rendered state pill' do |attr|
|
|
|
|
it 'returns expected html' do
|
|
|
|
aggregate_failures do
|
2022-07-23 23:45:48 +05:30
|
|
|
expect(subject).to have_css(attr[:css])
|
2020-07-28 23:09:34 +05:30
|
|
|
expect(subject).to have_content(attr[:state].capitalize)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'no state pill' do
|
|
|
|
specify { expect(subject).to eq(nil) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'merge request todo' do
|
|
|
|
let(:todo) { create(:todo, target: create(:merge_request)) }
|
|
|
|
|
|
|
|
it_behaves_like 'no state pill'
|
|
|
|
|
2022-07-23 23:45:48 +05:30
|
|
|
context 'closed MR' do
|
|
|
|
before do
|
|
|
|
todo.target.update!(state: 'closed')
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a rendered state pill', css: '.gl-bg-red-500', state: 'closed'
|
|
|
|
end
|
|
|
|
|
2020-07-28 23:09:34 +05:30
|
|
|
context 'merged MR' do
|
|
|
|
before do
|
|
|
|
todo.target.update!(state: 'merged')
|
|
|
|
end
|
|
|
|
|
2022-07-23 23:45:48 +05:30
|
|
|
it_behaves_like 'a rendered state pill', css: '.gl-bg-blue-500', state: 'merged'
|
2020-07-28 23:09:34 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'issue todo' do
|
|
|
|
let(:todo) { create(:todo, target: issue) }
|
|
|
|
|
|
|
|
it_behaves_like 'no state pill'
|
|
|
|
|
|
|
|
context 'closed issue' do
|
|
|
|
before do
|
|
|
|
todo.target.update!(state: 'closed')
|
|
|
|
end
|
|
|
|
|
2022-07-23 23:45:48 +05:30
|
|
|
it_behaves_like 'a rendered state pill', css: '.gl-bg-blue-500', state: 'closed'
|
2020-07-28 23:09:34 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'alert todo' do
|
|
|
|
let(:todo) { alert_todo }
|
|
|
|
|
|
|
|
it_behaves_like 'no state pill'
|
|
|
|
|
|
|
|
context 'resolved alert' do
|
|
|
|
before do
|
|
|
|
todo.target.resolve!
|
|
|
|
end
|
|
|
|
|
2022-07-23 23:45:48 +05:30
|
|
|
it_behaves_like 'a rendered state pill', css: '.gl-bg-blue-500', state: 'resolved'
|
2020-07-28 23:09:34 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2022-10-11 01:57:18 +05:30
|
|
|
describe '#no_todos_messages' do
|
|
|
|
context 'when getting todos messsages' do
|
|
|
|
it 'return these sentences' do
|
|
|
|
expected_sentences = [
|
|
|
|
s_('Todos|Good job! Looks like you don\'t have anything left on your To-Do List'),
|
|
|
|
s_('Todos|Isn\'t an empty To-Do List beautiful?'),
|
|
|
|
s_('Todos|Give yourself a pat on the back!'),
|
|
|
|
s_('Todos|Nothing left to do. High five!'),
|
|
|
|
s_('Todos|Henceforth, you shall be known as "To-Do Destroyer"')
|
|
|
|
]
|
|
|
|
expect(helper.no_todos_messages).to eq(expected_sentences)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-07-28 23:09:34 +05:30
|
|
|
describe '#todo_author_display?' do
|
|
|
|
using RSpec::Parameterized::TableSyntax
|
|
|
|
|
|
|
|
subject { helper.todo_author_display?(alert_todo) }
|
|
|
|
|
|
|
|
where(:action, :result) do
|
|
|
|
Todo::BUILD_FAILED | false
|
|
|
|
Todo::UNMERGEABLE | false
|
|
|
|
Todo::ASSIGNED | true
|
|
|
|
end
|
|
|
|
|
|
|
|
with_them do
|
|
|
|
before do
|
|
|
|
alert_todo.action = action
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to eq(result) }
|
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|