spec/lib/bulk_imports/network_error_spec.rb


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88

# frozen_string_literal: true

require 'spec_helper'

RSpec.describe BulkImports::NetworkError, :clean_gitlab_redis_cache do
  let(:tracker) { double(id: 1, stage: 2, entity: double(id: 3)) }

  describe '.new' do
    it 'requires either a message or a HTTP response' do
      expect { described_class.new }
        .to raise_error(ArgumentError, 'message or response required')
    end
  end

  describe '#retriable?' do
    it 'returns true for MAX_RETRIABLE_COUNT times when cause if one of RETRIABLE_EXCEPTIONS' do
      raise described_class::RETRIABLE_EXCEPTIONS.sample
    rescue StandardError => cause
      begin
        raise described_class, cause
      rescue StandardError => exception
        described_class::MAX_RETRIABLE_COUNT.times do
          expect(exception.retriable?(tracker)).to eq(true)
        end

        expect(exception.retriable?(tracker)).to eq(false)
      end
    end

    it 'returns true for MAX_RETRIABLE_COUNT times when response is one of RETRIABLE_CODES' do
      exception = described_class.new(response: double(code: 429))

      described_class::MAX_RETRIABLE_COUNT.times do
        expect(exception.retriable?(tracker)).to eq(true)
      end

      expect(exception.retriable?(tracker)).to eq(false)
    end

    it 'returns false for other exceptions' do
      raise StandardError
    rescue StandardError => cause
      begin
        raise described_class, cause
      rescue StandardError => exception
        expect(exception.retriable?(tracker)).to eq(false)
      end
    end

    context 'when entity is passed' do
      it 'increments entity cache key' do
        entity = create(:bulk_import_entity)
        exception = described_class.new('Error!')

        allow(exception).to receive(:cause).and_return(SocketError.new('Error!'))

        expect(Gitlab::Cache::Import::Caching)
          .to receive(:increment)
          .with("bulk_imports/#{entity.id}/network_error/SocketError")
          .and_call_original

        exception.retriable?(entity)
      end
    end
  end

  describe '#retry_delay' do
    it 'returns the default value when there is not a rate limit error' do
      exception = described_class.new('foo')

      expect(exception.retry_delay).to eq(described_class::DEFAULT_RETRY_DELAY_SECONDS.seconds)
    end

    context 'when the exception is a rate limit error' do
      it 'returns the "Retry-After"' do
        exception = described_class.new(response: double(code: 429, headers: { 'Retry-After' => 20 }))

        expect(exception.retry_delay).to eq(20.seconds)
      end

      it 'returns the default value when there is no "Retry-After" header' do
        exception = described_class.new(response: double(code: 429, headers: {}))

        expect(exception.retry_delay).to eq(described_class::DEFAULT_RETRY_DELAY_SECONDS.seconds)
      end
    end
  end
end