diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-02-14 10:38:17 +0100 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-02-14 11:27:14 +0100 |
commit | c0db4400f4d838e496090a6cd52a65bf2cd2051c (patch) | |
tree | 86b2e5997ebef757b49d568395fe447933b56d81 /spec/services/ci/retry_build_service_spec.rb | |
parent | 21401a39b8d7ef42f71595803a0a82f11f375e71 (diff) | |
download | gitlab-ce-c0db4400f4d838e496090a6cd52a65bf2cd2051c.tar.gz |
Preserve base service abstraction for retry services
Diffstat (limited to 'spec/services/ci/retry_build_service_spec.rb')
-rw-r--r-- | spec/services/ci/retry_build_service_spec.rb | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/spec/services/ci/retry_build_service_spec.rb b/spec/services/ci/retry_build_service_spec.rb index 0815469f2cb..9cd1da59c54 100644 --- a/spec/services/ci/retry_build_service_spec.rb +++ b/spec/services/ci/retry_build_service_spec.rb @@ -7,13 +7,13 @@ describe Ci::RetryBuildService, :services do let(:build) { create(:ci_build, pipeline: pipeline) } let(:service) do - described_class.new(build, user) + described_class.new(project, user) end - describe '#retry!' do - let(:new_build) { service.retry! } + describe '#execute' do + let(:new_build) { service.execute(build) } - context 'when user has ability to retry build' do + context 'when user has ability to execute build' do before do project.team << [user, :developer] end @@ -30,7 +30,7 @@ describe Ci::RetryBuildService, :services do expect(MergeRequests::AddTodoWhenBuildFailsService) .to receive_message_chain(:new, :close) - service.retry! + service.execute(build) end context 'when there are subsequent builds that are skipped' do @@ -39,16 +39,16 @@ describe Ci::RetryBuildService, :services do end it 'resumes pipeline processing in subsequent stages' do - service.retry! + service.execute(build) expect(subsequent_build.reload).to be_created end end end - context 'when user does not have ability to retry build' do + context 'when user does not have ability to execute build' do it 'raises an error' do - expect { service.retry! } + expect { service.execute(build) } .to raise_error Gitlab::Access::AccessDeniedError end end |