diff options
author | Thong Kuah <tkuah@gitlab.com> | 2019-06-24 01:36:32 +0300 |
---|---|---|
committer | Thong Kuah <tkuah@gitlab.com> | 2019-06-24 01:37:47 +0300 |
commit | 26b057293d18f35367e39882f3724372b616dc51 (patch) | |
tree | 6ec358f45ce0e2a03ddc5f839b88915020cc01ac /spec/workers | |
parent | f89a33c9d13db824bb4f4618512d82b6c4e6515a (diff) |
Remove deprecated call in BuildSuccessWorker
We now create_deployment on Ci::Build create.
No need to call create_deployment again
Diffstat (limited to 'spec/workers')
-rw-r--r-- | spec/workers/build_success_worker_spec.rb | 44 |
1 files changed, 0 insertions, 44 deletions
diff --git a/spec/workers/build_success_worker_spec.rb b/spec/workers/build_success_worker_spec.rb index ffe8796ded9..f4ff8cb15f8 100644 --- a/spec/workers/build_success_worker_spec.rb +++ b/spec/workers/build_success_worker_spec.rb @@ -6,51 +6,7 @@ describe BuildSuccessWorker do describe '#perform' do subject { described_class.new.perform(build.id) } - before do - allow_any_instance_of(Deployment).to receive(:create_ref) - end - context 'when build exists' do - context 'when deployment was not created with the build creation' do # An edge case during the transition period - let!(:build) { create(:ci_build, :deploy_to_production) } - - before do - allow(Deployments::FinishedWorker).to receive(:perform_async) - Deployment.delete_all - build.reload - end - - it 'creates a successful deployment' do - expect(build).not_to be_has_deployment - - subject - - build.reload - expect(build).to be_has_deployment - expect(build.deployment).to be_success - end - end - - context 'when deployment was created with the build creation' do # Counter part of the above edge case - let!(:build) { create(:ci_build, :deploy_to_production) } - - it 'does not create a new deployment' do - expect(build).to be_has_deployment - - expect { subject }.not_to change { Deployment.count } - end - end - - context 'when build is not associated with project' do - let!(:build) { create(:ci_build, project: nil) } - - it 'does not create deployment' do - subject - - expect(build.reload).not_to be_has_deployment - end - end - context 'when the build will stop an environment' do let!(:build) { create(:ci_build, :stop_review_app, environment: environment.name, project: environment.project) } let(:environment) { create(:environment, state: :available) } |