easybill-rest-client 2.0.2

data/spec/lib/easybill_rest_client/generic_api_spec.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+RSpec.describe EasybillRestClient::GenericApi do
+  class Resource
+    attr_reader :attributes
+
+    def initialize(attributes)
+      @attributes = attributes
+    end
+
+    def id
+      attributes[:id]
+    end
+
+    def method_missing(method)
+      respond_to_missing?(method) ? attributes[method] : super
+    end
+
+    def respond_to_missing?(method)
+      attributes.key?(method)
+    end
+  end
+
+  class ApiClass < EasybillRestClient::GenericApi
+    def resource_class
+      Resource
+    end
+
+    def resource_name
+      'resource'
+    end
+  end
+
+  let(:api_client) { instance_double(EasybillRestClient::ApiClient) }
+
+  subject(:api) { ApiClass.new(api_client) }
+
+  describe '#save' do
+    context 'entity does not exist' do
+      it 'creates it' do
+        expect(api_client).to receive(:request).with(:post, '/resource', foo: 'bar')
+
+        api.save(Resource.new(foo: 'bar'))
+      end
+    end
+
+    context 'entity does exist' do
+      it 'updates it' do
+        expect(api_client).to receive(:request).with(:put, '/resource/1', id: 1, foo: 'bar')
+
+        api.save(Resource.new(id: 1, foo: 'bar'))
+      end
+    end
+  end
+end

data/spec/lib/easybill_rest_client/pdf_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+RSpec.describe EasybillRestClient::Pdf do
+  describe '#filename' do
+    context 'filename ends with ".pdf"' do
+      subject(:pdf_asset) { described_class.new(filename: 'something.pdf', content: '') }
+
+      it 'returns the original filename' do
+        expect(pdf_asset.filename).to eq('something.pdf')
+      end
+    end
+
+    context 'filename does not end with ".pdf"' do
+      subject(:pdf_asset) { described_class.new(filename: 'something', content: '') }
+
+      it 'appends ".pdf" to the filename' do
+        expect(pdf_asset.filename).to eq('something.pdf')
+      end
+    end
+
+    context 'filename ends with ".PDF"' do
+      subject(:pdf_asset) { described_class.new(filename: 'SOMETHING.PDF', content: '') }
+
+      it 'returns the original filename' do
+        expect(pdf_asset.filename).to eq('SOMETHING.PDF')
+      end
+    end
+  end
+end

data/spec/lib/easybill_rest_client/request_logger_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+RSpec.describe EasybillRestClient::RequestLogger do
+  let(:logger) { instance_double(Logger) }
+
+  subject(:request_logger) { described_class.new(logger: logger, request_id: '123') }
+
+  %i[info warn error].each do |method|
+    it "logs #{method}s" do
+      expect(logger).to receive(method).with('[easybill-rest-client] RequestId=123 foo')
+      request_logger.public_send(method, 'foo')
+    end
+  end
+end

data/spec/lib/easybill_rest_client/request_spec.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+RSpec.describe EasybillRestClient::Request do
+  subject(:request) do
+    described_class.new(
+      api_key: 'api-key',
+      method: :get,
+      endpoint: '/',
+      params: {},
+      logger: Logger.new(nil),
+      tries: 2,
+      retry_cool_off_time: 0
+    )
+  end
+
+  context 'too many requests have been sent' do
+    let(:too_many_requests) { Net::HTTPTooManyRequests.new(nil, '429', nil) }
+    let(:ok) do
+      response = Net::HTTPOK.new(nil, '200', nil)
+      response.body = 'BODYBODYBODY'
+      response.instance_variable_set('@read', true)
+      response.content_type = 'application/json'
+      response
+    end
+
+    it 'retries api calls that return a "Too Many Requests" error' do
+      allow(Net::HTTP).to receive(:start).and_return(too_many_requests, ok)
+
+      expect(request.run.body).to eq('BODYBODYBODY')
+    end
+  end
+end

data/spec/lib/easybill_rest_client/retry_on_spec.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+RSpec.describe EasybillRestClient::RetryOn do
+  subject(:retry_handler) { described_class.new(logger, 2, 0) }
+
+  let(:logger) { instance_double(Logger) }
+  let(:open_timeout) { EasybillRestClient::Request::OPEN_TIMEOUT }
+
+  context 'timeout while opening connection' do
+    it 'retries the request' do
+      expect(logger).to receive(:warn)
+        .with("Unable to open connection after #{open_timeout}s, retrying...")
+
+      raise_error = true
+      retry_handler.retry_on(Net::OpenTimeout) do
+        if raise_error
+          raise_error = false
+          raise Net::OpenTimeout
+        end
+      end
+    end
+  end
+
+  context 'too many requests' do
+    it 'retries the request' do
+      expect(logger).to receive(:warn).with('Too many requests!')
+
+      raise_error = true
+      retry_handler.retry_on(EasybillRestClient::TooManyRequests) do
+        if raise_error
+          raise_error = false
+          raise EasybillRestClient::TooManyRequests
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'pry'
+require 'vcr'
+Dir[File.expand_path('../support/*.rb', __FILE__)].each { |rb| require rb }
+require 'easybill-rest-client'
+require 'webmock/rspec'
+
+RSpec.configure do |config|
+  config.include SetupClient
+
+  config.expect_with :rspec do |expectations|
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  config.mock_with :rspec do |mocks|
+    mocks.verify_partial_doubles = true
+  end
+end
+
+VCR.configure do |c|
+  c.default_cassette_options = { match_requests_on: %i[method uri body] }
+  c.configure_rspec_metadata!
+  c.cassette_library_dir = 'spec/fixtures/vcr'
+  c.hook_into :webmock
+  c.filter_sensitive_data('easybill-username') { ENV.fetch('EASYBILL_USERNAME') }
+  c.filter_sensitive_data('easybill-api-key') { ENV.fetch('EASYBILL_API_KEY') }
+  c.filter_sensitive_data('easybill-basic-auth-key') { ENV.fetch('EASYBILL_BASIC_AUTH_KEY') }
+end
+
+ENV['EASYBILL_USERNAME'] ||= 'easybill-username'
+ENV['EASYBILL_API_KEY'] ||= 'easybill-api-key'
+ENV['EASYBILL_BASIC_AUTH_KEY'] ||= 'easybill-basic-auth-key'

data/spec/support/setup_client.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require 'logger'
+
+module SetupClient
+  def client
+    @client ||= EasybillRestClient::Client.new(
+      api_key: ENV.fetch('EASYBILL_API_KEY'),
+      logger: Logger.new('log/test.log')
+    )
+  end
+end