onfido 1.1.1 → 2.0.0

data/spec/integrations/document_spec.rb

@@ -1,7 +1,11 @@
+# frozen_string_literal: true
+
 require 'tempfile'
 
 describe Onfido::Document do
-  subject(:document) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:document) { onfido.document }
 
   describe '#create' do
     let(:params) do
@@ -32,8 +36,8 @@ describe Onfido::Document do
       let(:file) { 'https://onfido.com/images/logo.png' }
 
       it 'raises an ArgumentError' do
-        expect { document.create(**params) }.
-          to raise_error(ArgumentError, /must be a `File`-like object/)
+        expect { document.create(**params) }
+          .to raise_error(ArgumentError, /must be a `File`-like object/)
       end
     end
   end

data/spec/integrations/extraction_spec.rb

@@ -1,7 +1,11 @@
+# frozen_string_literal: true
+
 require 'tempfile'
 
 describe Onfido::Extraction do
-  subject(:extraction) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:extraction) { onfido.extraction }
 
   describe '#create' do
     let(:params) do
@@ -11,7 +15,7 @@ describe Onfido::Extraction do
     end
 
     it 'creates a new extraction' do
-      response = extraction.create(params)
+      response = extraction.create(**params)
 
       expect(response['document_id']).to eq('7568415-123123-123123')
     end

data/spec/integrations/live_photo_spec.rb

@@ -1,7 +1,11 @@
+# frozen_string_literal: true
+
 require 'tempfile'
 
 describe Onfido::LivePhoto do
-  subject(:live_photo) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:live_photo) { onfido.live_photo }
 
   describe '#create' do
     let(:params) { { applicant_id: '123456', file: file } }
@@ -25,8 +29,8 @@ describe Onfido::LivePhoto do
       let(:file) { 'https://onfido.com/images/photo.jpg' }
 
       it 'raises an ArgumentError' do
-        expect { live_photo.create(**params) }.
-          to raise_error(ArgumentError, /must be a `File`-like object/)
+        expect { live_photo.create(**params) }
+          .to raise_error(ArgumentError, /must be a `File`-like object/)
       end
     end
   end

data/spec/integrations/live_video_spec.rb

@@ -1,7 +1,12 @@
+# frozen_string_literal: true
+
 require 'tempfile'
 
 describe Onfido::LiveVideo do
-  subject(:live_video) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:live_video) { onfido.live_video }
+
   let(:live_video_id) { 'c9701e9b-83aa-442f-995b-20320ee8fb01' }
 
   describe '#find' do

data/spec/integrations/report_spec.rb

@@ -1,5 +1,10 @@
+# frozen_string_literal: true
+
 describe Onfido::Report do
-  subject(:report) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:report) { onfido.report }
+
   describe '#find' do
     it 'returns a report for an existing check' do
       report_id = '6951786-123123-422221'

data/spec/integrations/resource_spec.rb

@@ -0,0 +1,93 @@
+# frozen_string_literal: true
+
+class TestResource < Onfido::Resource
+  def get(path:)
+    super(path: path)
+  end
+end
+
+describe Onfido::Resource do
+  include_context 'fake onfido api'
+
+  subject(:resource) { TestResource.new(Onfido::Options.new(api_key: 'test', region: :eu)) }
+
+  context '4xx response' do
+    it 'raises a custom error' do
+      path = '4xx_response'
+
+      expect { resource.get(path: path) }
+        .to raise_error(Onfido::RequestError, 'Something went wrong')
+    end
+  end
+
+  context 'unexpected error format' do
+    it 'raises a custom error' do
+      path = 'unexpected_error_format'
+
+      expect { resource.get(path: path) }
+        .to raise_error(Onfido::RequestError, /response code was 400/)
+    end
+  end
+
+  context 'unparseable JSON 5xx' do
+    it 'raises a server error' do
+      path = 'unparseable_response'
+
+      expect { resource.get(path: path) }
+        .to raise_error(Onfido::ServerError, /response code was 504/)
+    end
+  end
+
+  context 'timeout' do
+    before do
+      allow(RestClient::Request)
+        .to receive(:execute)
+        .and_raise(RestClient::RequestTimeout)
+    end
+
+    it 'raises a ConnectionError' do
+      expect { resource.get(path: '') }
+        .to raise_error(Onfido::ConnectionError, /Could not connect/)
+    end
+  end
+
+  context 'broken connection' do
+    before do
+      allow(RestClient::Request)
+        .to receive(:execute)
+        .and_raise(RestClient::ServerBrokeConnection)
+    end
+
+    it 'raises a ConnectionError' do
+      expect { resource.get(path: '') }
+        .to raise_error(Onfido::ConnectionError, /connection to the server/)
+    end
+  end
+
+  context 'bad SSL certificate' do
+    before do
+      allow(RestClient::Request)
+        .to receive(:execute)
+        .and_raise(RestClient::SSLCertificateNotVerified.new(nil))
+    end
+
+    it 'raises a ConnectionError' do
+      expect { resource.get(path: '') }
+        .to raise_error(Onfido::ConnectionError, /SSL certificate/)
+    end
+  end
+
+  context 'with a timeout error response' do
+    before do
+      allow_any_instance_of(RestClient::ExceptionWithResponse)
+        .to receive(:response).and_return(double(body: '', code: '408'))
+      expect(RestClient::Request).to receive(:execute)
+        .and_raise(RestClient::ExceptionWithResponse)
+    end
+
+    it 'raises a ConnectionError' do
+      expect { resource.get(path: '') }
+        .to raise_error(Onfido::ConnectionError)
+    end
+  end
+end

data/spec/integrations/sdk_token_spec.rb

@@ -1,5 +1,9 @@
+# frozen_string_literal: true
+
 describe Onfido::SdkToken do
-  subject(:sdk_token) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:sdk_token) { onfido.sdk_token }
 
   describe '#create' do
     it 'creates a new SDK token for the applicant' do

data/spec/integrations/webhook_spec.rb

@@ -1,28 +1,32 @@
+# frozen_string_literal: true
+
 describe Onfido::Webhook do
-  subject(:webhook) { described_class.new }
+  include_context 'fake onfido api'
+
+  subject(:webhook) { onfido.webhook }
 
-  describe "#create" do
+  describe '#create' do
     let(:params) do
       {
-        url: "https://webhookendpoint.url",
+        url: 'https://webhookendpoint.url',
         enabled: true,
         events: [
-          "report.completed",
-          "check.completed"
+          'report.completed',
+          'check.completed'
         ]
       }
     end
 
-    it "creates the webhook" do
+    it 'creates the webhook' do
       response = webhook.create(**params)
 
       expect(response['id']).to_not be_nil
     end
 
-    it "responds with the right url" do
+    it 'responds with the right url' do
       response = webhook.create(**params)
 
-      expect(response["url"]).to eq params[:url]
+      expect(response['url']).to eq params[:url]
     end
   end
 
@@ -36,29 +40,29 @@ describe Onfido::Webhook do
     end
   end
 
-  describe "#destroy" do
-    it "removes the webhook" do
+  describe '#destroy' do
+    it 'removes the webhook' do
       webhook_id = 'fcb73186-0733-4f6f-9c57-d9d5ef979443'
       expect { webhook.destroy(webhook_id) }.not_to raise_error
     end
   end
 
-  describe "#all" do
-    it "returns all the registered webhooks" do
+  describe '#all' do
+    it 'returns all the registered webhooks' do
       response = webhook.all
 
-      expect(response["webhooks"].count).to eq 2
+      expect(response['webhooks'].count).to eq 2
     end
 
-    it "returns with id" do
+    it 'returns with id' do
       response = webhook.all
 
-      expect(response["webhooks"][0]["id"]).to_not be_nil
-      expect(response["webhooks"][1]["id"]).to_not be_nil
+      expect(response['webhooks'][0]['id']).to_not be_nil
+      expect(response['webhooks'][1]['id']).to_not be_nil
     end
   end
 
-  describe ".valid?" do
+  describe '.valid?' do
     subject(:valid?) do
       described_class.valid?(request_body, request_signature, token)
     end
@@ -71,34 +75,34 @@ describe Onfido::Webhook do
 
     it { is_expected.to be(true) }
 
-    context "with an invalid signature" do
+    context 'with an invalid signature' do
       let(:request_signature) do
         's21fd54ew2w1f5d15642132f3d7727ff9a32a7c87072ce514df1f6d3228bec'
       end
       it { is_expected.to be(false) }
     end
 
-    context "with a nil request signature" do
+    context 'with a nil request signature' do
       let(:request_signature) { nil }
       specify { expect { valid? }.to raise_error(ArgumentError) }
     end
 
-    context "with a token other than the one used to sign the request" do
-      let(:token) { "quite_secret_token" }
+    context 'with a token other than the one used to sign the request' do
+      let(:token) { 'quite_secret_token' }
       it { is_expected.to be(false) }
     end
 
-    context "with a nil token" do
+    context 'with a nil token' do
       let(:token) { nil }
       specify { expect { valid? }.to raise_error(ArgumentError) }
     end
 
-    context "with a modified request body" do
+    context 'with a modified request body' do
       let(:request_body) { '{"bar":"baz"}' }
       it { is_expected.to be(false) }
     end
 
-    context "with a nil request body" do
+    context 'with a nil request body' do
       let(:request_body) { nil }
       specify { expect { valid? }.to raise_error(ArgumentError) }
     end

data/spec/onfido/api_spec.rb

@@ -1,33 +1,22 @@
-describe Onfido::API do
-  subject(:api) { described_class.new }
-
-  describe 'given a single-word resource' do
-    specify { expect(api.address).to be_a(Onfido::Address) }
-  end
+# frozen_string_literal: true
 
-  describe 'given a multi-word resource' do
-    specify { expect(api.live_photo).to be_a(Onfido::LivePhoto) }
-  end
-
-  describe 'given an unknown resource' do
-    specify { expect { api.blood_test }.to raise_error(NameError) }
+describe Onfido::API do
+  before do
+    allow(Onfido::Options).to receive(:new).and_call_original
   end
 
-  describe 'given no API key' do
-    it 'uses nil for the resource API key' do
-      expect(Onfido::Address).to receive(:new).with(nil)
-      api.address
-    end
+  let(:options) do
+    {
+      api_key: 'test',
+      region: :eu,
+      open_timeout: 1,
+      read_timeout: 2
+    }
   end
 
-  describe 'given an API key' do
-    let(:api_key) { 'some_key' }
-
-    subject(:api) { described_class.new(api_key: api_key) }
+  it 'passes through options' do
+    described_class.new(**options)
 
-    it 'uses that key to create the resource' do
-      expect(Onfido::Address).to receive(:new).with(api_key)
-      api.address
-    end
+    expect(Onfido::Options).to have_received(:new).with(**options)
   end
 end

data/spec/onfido/connection_error_spec.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 describe Onfido::ConnectionError do
   subject(:error) do
     described_class.new(
-      "Invalid response object from API",
+      'Invalid response object from API',
       response_code: response_code,
       response_body: response_body
     )
@@ -10,7 +12,7 @@ describe Onfido::ConnectionError do
   let(:response_code) { nil }
   let(:response_body) { nil }
 
-  context "without a response_body" do
+  context 'without a response_body' do
     its(:json_body) { is_expected.to be_nil }
     its(:type) { is_expected.to be_nil }
     its(:fields) { is_expected.to be_nil }

data/spec/onfido/options_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+describe Onfido::Options do
+  subject(:options) do
+    described_class.new(
+      api_key: 'test',
+      region: :us,
+      open_timeout: 1,
+      read_timeout: 2
+    )
+  end
+
+  it 'checks region is valid' do
+    expect { described_class.new(api_key: 'test', region: :aa) }.to raise_error 'Unknown region aa'
+  end
+
+  context 'when creating rest client resource' do
+    let(:rest_client) { options.rest_client }
+
+    it 'configures with headers' do
+      expect(rest_client.options[:headers]).to eq(
+        'Accept' => 'application/json',
+        'Authorization' => 'Token token=test',
+        'User-Agent' => "onfido-ruby/#{Onfido::VERSION}"
+      )
+    end
+
+    it 'configures with region' do
+      expect(rest_client.url).to eq 'https://api.us.onfido.com/v3.1/'
+    end
+
+    it 'configures with timeouts' do
+      expect(rest_client.options).to include(
+        open_timeout: 1,
+        read_timeout: 2
+      )
+    end
+  end
+end

data/spec/onfido/request_error_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 describe Onfido::RequestError do
   subject(:error) do
     described_class.new(
@@ -20,8 +22,8 @@ describe Onfido::RequestError do
   end
 
   it 'returns the right message' do
-    expect { raise error }.
-      to raise_error('Authorization error: please re-check your credentials')
+    expect { raise error }
+      .to raise_error('Authorization error: please re-check your credentials')
   end
 
   its(:type) { is_expected.to eq('authorization_error') }