diff options
Diffstat (limited to 'spec/models/project_services/buildkite_service_spec.rb')
-rw-r--r-- | spec/models/project_services/buildkite_service_spec.rb | 151 |
1 files changed, 0 insertions, 151 deletions
diff --git a/spec/models/project_services/buildkite_service_spec.rb b/spec/models/project_services/buildkite_service_spec.rb deleted file mode 100644 index f6bf1551bf0..00000000000 --- a/spec/models/project_services/buildkite_service_spec.rb +++ /dev/null @@ -1,151 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe BuildkiteService, :use_clean_rails_memory_store_caching do - include ReactiveCachingHelpers - include StubRequests - - let(:project) { create(:project) } - - subject(:service) do - described_class.create!( - project: project, - properties: { - service_hook: true, - project_url: 'https://buildkite.com/organization-name/example-pipeline', - token: 'secret-sauce-webhook-token:secret-sauce-status-token' - } - ) - end - - describe 'Associations' do - it { is_expected.to belong_to :project } - it { is_expected.to have_one :service_hook } - end - - describe 'Validations' do - context 'when service is active' do - before do - subject.active = true - end - - it { is_expected.to validate_presence_of(:project_url) } - it { is_expected.to validate_presence_of(:token) } - it_behaves_like 'issue tracker service URL attribute', :project_url - end - - context 'when service is inactive' do - before do - subject.active = false - end - - it { is_expected.not_to validate_presence_of(:project_url) } - it { is_expected.not_to validate_presence_of(:token) } - end - end - - describe '.supported_events' do - it 'supports push, merge_request, and tag_push events' do - expect(service.supported_events).to eq %w(push merge_request tag_push) - end - end - - describe 'commits methods' do - before do - allow(project).to receive(:default_branch).and_return('default-brancho') - end - - it 'always activates SSL verification after saved' do - service.create_service_hook(enable_ssl_verification: false) - - service.enable_ssl_verification = false - service.active = true - - expect { service.save! } - .to change { service.service_hook.enable_ssl_verification }.from(false).to(true) - end - - describe '#webhook_url' do - it 'returns the webhook url' do - expect(service.webhook_url).to eq( - 'https://webhook.buildkite.com/deliver/secret-sauce-webhook-token' - ) - end - end - - describe '#commit_status_path' do - it 'returns the correct status page' do - expect(service.commit_status_path('2ab7834c')).to eq( - 'https://gitlab.buildkite.com/status/secret-sauce-status-token.json?commit=2ab7834c' - ) - end - end - - describe '#build_page' do - it 'returns the correct build page' do - expect(service.build_page('2ab7834c', nil)).to eq( - 'https://buildkite.com/organization-name/example-pipeline/builds?commit=2ab7834c' - ) - end - end - - describe '#commit_status' do - it 'returns the contents of the reactive cache' do - stub_reactive_cache(service, { commit_status: 'foo' }, 'sha', 'ref') - - expect(service.commit_status('sha', 'ref')).to eq('foo') - end - end - - describe '#calculate_reactive_cache' do - describe '#commit_status' do - let(:buildkite_full_url) do - 'https://gitlab.buildkite.com/status/secret-sauce-status-token.json?commit=123' - end - - subject { service.calculate_reactive_cache('123', 'unused')[:commit_status] } - - it 'sets commit status to :error when status is 500' do - stub_request(status: 500) - - is_expected.to eq(:error) - end - - it 'sets commit status to :error when status is 404' do - stub_request(status: 404) - - is_expected.to eq(:error) - end - - it 'passes through build status untouched when status is 200' do - stub_request(body: %q({"status":"Great Success"})) - - is_expected.to eq('Great Success') - end - - Gitlab::HTTP::HTTP_ERRORS.each do |http_error| - it "sets commit status to :error with a #{http_error.name} error" do - WebMock.stub_request(:get, buildkite_full_url) - .to_raise(http_error) - - expect(Gitlab::ErrorTracking) - .to receive(:log_exception) - .with(instance_of(http_error), project_id: project.id) - - is_expected.to eq(:error) - end - end - end - end - end - - def stub_request(status: 200, body: nil) - body ||= %q({"status":"success"}) - - stub_full_request(buildkite_full_url) - .to_return(status: status, - headers: { 'Content-Type' => 'application/json' }, - body: body) - end -end |