Fix logger and add raise option (#11)

* Fix logger
* Add option to raise on error requests

Author: Unai Abrisketa

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    beyond_api (0.2.1.pre)
+    beyond_api (0.3.0.pre)
       faraday (~> 0.15)
 
 GEM

lib/beyond_api.rb

@@ -10,6 +10,14 @@
 module BeyondApi
   class Error < StandardError; end
 
+  def self.logger
+    @@logger ||= defined?(Rails) ? Rails.logger : Logger.new(STDOUT)
+  end
+
+  def self.logger=(logger)
+    @@logger = logger
+  end
+
   class << self
     attr_accessor :configuration
   end
@@ -22,7 +30,7 @@ def self.setup
 
   class Configuration
     attr_accessor :client_id, :client_secret, :open_timeout, :timeout,  :remove_response_links,
-                  :remove_response_key_underscores, :object_struct_responses
+                  :remove_response_key_underscores, :object_struct_responses, :raise_error_requests
 
     def initialize
       @client_id = nil
@@ -32,6 +40,7 @@ def initialize
       @remove_response_links = false
       @remove_response_key_underscores = false
       @object_struct_responses = false
+      @raise_error_requests = false
     end
   end
 end

lib/beyond_api/utils.rb

@@ -10,8 +10,9 @@ def handle_response(response, status, respond_with_true: false)
         response = sanitize_response(response)
         BeyondApi.configuration.object_struct_responses ? to_object_struct(response) : response
       else
-        logger.error "[Beyond API] #{response}" if logger
-        BeyondApi::Error.new(response)
+        error = BeyondApi::Error.new(response)
+        BeyondApi.logger.error "[Beyond API] #{response}"
+        BeyondApi.configuration.raise_error_requests ? raise(error) : error
       end
     end
 

lib/beyond_api/version.rb

@@ -1,3 +1,3 @@
 module BeyondApi
-  VERSION = "0.2.1.pre".freeze
+  VERSION = "0.3.0.pre".freeze
 end

lib/generators/templates/beyond_api_initializer.rb

@@ -26,4 +26,10 @@
   # With OpenStructs => response.embeded.products.first.id
   # Without OpenStructs => response["embeded"]["products"].first["id"]
   # config.object_struct_responses = false
+
+  # Configure if the gem should raise on error requests. Setting it to true is
+  # useful for working with exceptions. Setting it to false  will return a
+  # BeyondApi::Error object with detailed information of the error.
+  # Default is false.
+  # config.raise_error_requests = false
 end