Merge pull request #67 from stephencookdev/feature/EG-3600-region-sup…

…port

Region support

lib/onfido/configuration.rb

@@ -1,6 +1,10 @@
 module Onfido
   module Configuration
-    attr_accessor :api_key, :open_timeout, :read_timeout, :api_version
+    REGION_HOSTS = {
+      us: "api.us.onfido.com"
+    }.freeze
+
+    attr_accessor :api_key, :region, :open_timeout, :read_timeout, :api_version
 
     def self.extended(base)
       base.reset
@@ -12,6 +16,7 @@ def configure
 
     def reset
       self.api_key = nil
+      self.region = nil
       self.open_timeout = 30
       self.read_timeout = 80
       self.api_version = 'v2'
@@ -30,22 +35,13 @@ def logger
       RestClient.log ||= NullLogger.new
     end
 
-    def region
-      return unless api_key
-
-      first_bit = api_key.split("_")[0]
-
-      return if %w(live test).include?(first_bit)
-
-      first_bit
-    end
-
     def endpoint
-      if region
-        "https://api.#{region}.onfido.com/#{api_version}/"
-      else
-        "https://api.onfido.com/#{api_version}/"
+      region_host = region ? REGION_HOSTS[region.downcase.to_sym] : "api.onfido.com"
+      unless region_host
+        raise "The region \"#{region.downcase}\" is not currently supported"
       end
+
+      "https://#{region_host}/#{api_version}/"
     end
   end
 end

lib/onfido/version.rb

@@ -1,3 +1,3 @@
 module Onfido
-  VERSION = '0.12.0'.freeze
+  VERSION = '0.13.0'.freeze
 end

spec/onfido_spec.rb

@@ -35,17 +35,18 @@
       end
     end
 
-    describe 'using a US API token' do
+    describe 'using the US region' do
       it 'should change endpoint' do
-        onfido.api_key = "us_live_asdfghjkl1234567890qwertyuiop"
+        onfido.region = 'us'
         expect(onfido.endpoint).to eq('https://api.us.onfido.com/v2/')
       end
     end
 
-    describe 'using a EU API token' do
+    describe 'using an unsupported region' do
       it 'should change endpoint' do
-        onfido.api_key = "eu_live_asdfghjkl1234567890qwertyuiop"
-        expect(onfido.endpoint).to eq('https://api.eu.onfido.com/v2/')
+        onfido.region = 'de'
+        expect { onfido.endpoint }.
+          to raise_error('The region "de" is not currently supported')
       end
     end