diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-02-15 10:54:33 +0100 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-02-15 10:54:33 +0100 |
commit | 25b3b2e5bc52000ac05efa9e87bff7bd949d8ed9 (patch) | |
tree | 1ac995d1b9619327f1c3f91104434512ef5cab57 /spec | |
parent | fc133129462f046a488bfc7d3e6bad7223c01207 (diff) | |
download | gitlab-ce-25b3b2e5bc52000ac05efa9e87bff7bd949d8ed9.tar.gz |
Make it possible to resolve all todos for pipeline
Diffstat (limited to 'spec')
-rw-r--r-- | spec/services/merge_requests/add_todo_when_build_fails_service_spec.rb | 25 |
1 files changed, 24 insertions, 1 deletions
diff --git a/spec/services/merge_requests/add_todo_when_build_fails_service_spec.rb b/spec/services/merge_requests/add_todo_when_build_fails_service_spec.rb index bb7830c7eea..d80fb8a1af1 100644 --- a/spec/services/merge_requests/add_todo_when_build_fails_service_spec.rb +++ b/spec/services/merge_requests/add_todo_when_build_fails_service_spec.rb @@ -17,7 +17,7 @@ describe MergeRequests::AddTodoWhenBuildFailsService do described_class.new(project, user, commit_message: 'Awesome message') end - let(:todo_service) { TodoService.new } + let(:todo_service) { spy('todo service') } let(:merge_request) do create(:merge_request, merge_user: user, @@ -107,4 +107,27 @@ describe MergeRequests::AddTodoWhenBuildFailsService do end end end + + describe '#close_all' do + context 'when using pipeline that belongs to merge request' do + it 'resolves todos about failed builds for pipeline' do + service.close_all(pipeline) + + expect(todo_service) + .to have_received(:merge_request_build_retried) + .with(merge_request) + end + end + + context 'when pipeline is not related to merge request' do + let(:pipeline) { create(:ci_empty_pipeline) } + + it 'does not resolve any todos about failed builds' do + service.close_all(pipeline) + + expect(todo_service) + .not_to have_received(:merge_request_build_retried) + end + end + end end |