diff options
author | Marin Jankovski <marin@gitlab.com> | 2019-04-17 20:23:31 +0000 |
---|---|---|
committer | Marin Jankovski <marin@gitlab.com> | 2019-04-17 20:23:31 +0000 |
commit | 807bb47f950f04f8d65764794bf72095d55480ab (patch) | |
tree | 80b9e0773cd7c3b391f9d544c4382a55efa6d14c /spec/services/ci/prepare_build_service_spec.rb | |
parent | 2d82633fdd95e0b7b1ea79bc12514ab4f6e71a88 (diff) | |
parent | a364be88a4f4b2a7efafda7d04881b7adde941c7 (diff) | |
download | gitlab-ce-807bb47f950f04f8d65764794bf72095d55480ab.tar.gz |
Merge branch '11-10-stable-prepare-rc8' into '11-10-stable'
Prepare 11.10.0-rc8 release
See merge request gitlab-org/gitlab-ce!27449
Diffstat (limited to 'spec/services/ci/prepare_build_service_spec.rb')
-rw-r--r-- | spec/services/ci/prepare_build_service_spec.rb | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/spec/services/ci/prepare_build_service_spec.rb b/spec/services/ci/prepare_build_service_spec.rb index 1797f8f964f..2d027f13e52 100644 --- a/spec/services/ci/prepare_build_service_spec.rb +++ b/spec/services/ci/prepare_build_service_spec.rb @@ -38,7 +38,21 @@ describe Ci::PrepareBuildService do end it 'drops the build' do - expect(build).to receive(:drop!).with(:unmet_prerequisites).once + expect(build).to receive(:drop).with(:unmet_prerequisites).once + + subject + end + end + + context 'prerequisites raise an error' do + before do + allow(prerequisite).to receive(:complete!).and_raise Kubeclient::HttpError.new(401, 'unauthorized', nil) + end + + it 'drops the build and notifies Sentry' do + expect(build).to receive(:drop).with(:unmet_prerequisites).once + expect(Gitlab::Sentry).to receive(:track_acceptable_exception) + .with(instance_of(Kubeclient::HttpError), hash_including(extra: { build_id: build.id })) subject end |