summaryrefslogtreecommitdiff
path: root/spec/services/merge_requests/update_service_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/services/merge_requests/update_service_spec.rb')
-rw-r--r--spec/services/merge_requests/update_service_spec.rb37
1 files changed, 19 insertions, 18 deletions
diff --git a/spec/services/merge_requests/update_service_spec.rb b/spec/services/merge_requests/update_service_spec.rb
index 6d70e8ec16a..99703c7a8ec 100644
--- a/spec/services/merge_requests/update_service_spec.rb
+++ b/spec/services/merge_requests/update_service_spec.rb
@@ -98,16 +98,16 @@ describe MergeRequests::UpdateService, services: true do
end
end
- context 'task queue' do
- let!(:pending_task) { create(:task, :assigned, user: user, project: project, target: merge_request, author: user2) }
+ context 'todos' do
+ let!(:pending_todo) { create(:todo, :assigned, user: user, project: project, target: merge_request, author: user2) }
context 'when the title change' do
before do
update_merge_request({ title: 'New title' })
end
- it 'marks pending tasks as done' do
- expect(pending_task.reload).to be_done
+ it 'marks pending todos as done' do
+ expect(pending_todo.reload).to be_done
end
end
@@ -116,8 +116,8 @@ describe MergeRequests::UpdateService, services: true do
update_merge_request({ description: 'Also please fix' })
end
- it 'marks pending tasks as done' do
- expect(pending_task.reload).to be_done
+ it 'marks pending todos as done' do
+ expect(pending_todo.reload).to be_done
end
end
@@ -126,21 +126,22 @@ describe MergeRequests::UpdateService, services: true do
update_merge_request({ assignee: user2 })
end
- it 'marks previous assignee pending tasks as done' do
- expect(pending_task.reload).to be_done
+ it 'marks previous assignee pending todos as done' do
+ expect(pending_todo.reload).to be_done
end
- it 'creates a pending task for new assignee' do
+ it 'creates a pending todo for new assignee' do
attributes = {
project: project,
author: user,
user: user2,
- target: merge_request,
- action: Task::ASSIGNED,
+ target_id: merge_request.id,
+ target_type: merge_request.class.name,
+ action: Todo::ASSIGNED,
state: :pending
}
- expect(Task.where(attributes).count).to eq 1
+ expect(Todo.where(attributes).count).to eq 1
end
end
@@ -149,8 +150,8 @@ describe MergeRequests::UpdateService, services: true do
update_merge_request({ milestone: create(:milestone) })
end
- it 'marks pending tasks as done' do
- expect(pending_task.reload).to be_done
+ it 'marks pending todos as done' do
+ expect(pending_todo.reload).to be_done
end
end
@@ -159,8 +160,8 @@ describe MergeRequests::UpdateService, services: true do
update_merge_request({ label_ids: [label.id] })
end
- it 'marks pending tasks as done' do
- expect(pending_task.reload).to be_done
+ it 'marks pending todos as done' do
+ expect(pending_todo.reload).to be_done
end
end
@@ -169,8 +170,8 @@ describe MergeRequests::UpdateService, services: true do
update_merge_request({ target_branch: 'target' })
end
- it 'marks pending tasks as done' do
- expect(pending_task.reload).to be_done
+ it 'marks pending todos as done' do
+ expect(pending_todo.reload).to be_done
end
end
end