omni_exchange 1.1.1 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23f8e6cc867244781c587d8ba217e373dabd00c5bf78710b673d8e6f4b330cb0
-  data.tar.gz: fffb0e30a3bb6f953ff6a38afcf821023f0e9f898c91448c29e2a6697153bf5d
+  metadata.gz: 3ab4966ed766dfb9d1762d0b476ce31c5b418e90a92cb7036ced1e0cd251522f
+  data.tar.gz: b8de8d1a0e0ab10bfe53bc6bce3ad0d3ce8d8387dc22182bba9a8f78e95dafb1
 SHA512:
-  metadata.gz: a4fe1ff6099ab9d3662f3a4881303e6c3c7d380cfc49c559c09ca39f1a5a918e381fb017c45cb9552e4b7dd35f8c7f52beaa2762898eb84e43e737aa59f7999e
-  data.tar.gz: 67d7ae42c4ff478a18669cbbed4ac9c99ac81947402e671e8fec1f072d7e3eeacaf46f18b4570e23b675a7dd34c1ff44b34c051c947aa6a2766e50fc3531755e
+  metadata.gz: 67730c3e4c1664dcd04ead3a6a4cb03b25dfc152712dcd7bdf4c706d9ff5b72c36820bfbc4ac87cf1c7e5c8e0586519bbeda5424fdc6db939e4e105676c0b5b2
+  data.tar.gz: 6c9ff326e891090d87b41277ee2d0402ecf85b1c7f72d826184a8a8cef7519dbdca07bc0b1e8f62d829229877673d70ca86c67458771689bb9de86282076128c

data/.gitignore

@@ -17,3 +17,6 @@ sandbox.rb
 omni_exchange-0.1.0.gem
 omni_exchange-0.2.0.gem
 omni_exchange-1.1.0.gem
+omni_exchange-1.1.1.gem
+omni_exchange-1.2.0.gem
+omni_exchange-1.3.0.gem

data/.rubocop_todo.yml

@@ -1,15 +1,20 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2022-06-30 01:49:03 UTC using RuboCop version 0.93.1.
+# on 2022-07-12 04:26:20 UTC using RuboCop version 0.93.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
+# Offense count: 1
+Lint/UselessAssignment:
+  Exclude:
+    - 'lib/sandbox.rb'
+
 # Offense count: 2
 # Configuration parameters: IgnoredMethods.
 Metrics/AbcSize:
-  Max: 28
+  Max: 33
 
 # Offense count: 1
 # Configuration parameters: CountComments, CountAsOne, ExcludedMethods.
@@ -39,7 +44,7 @@ Style/ExponentialNotation:
   Exclude:
     - 'spec/omni_exchange_spec.rb'
 
-# Offense count: 3
+# Offense count: 4
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, ConsistentQuotesInMultiline.
 # SupportedStyles: single_quotes, double_quotes
@@ -47,9 +52,17 @@ Style/StringLiterals:
   Exclude:
     - 'lib/sandbox.rb'
 
-# Offense count: 16
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: .
+# SupportedStyles: percent, brackets
+Style/SymbolArray:
+  EnforcedStyle: percent
+  MinSize: 3
+
+# Offense count: 19
 # Cop supports --auto-correct.
 # Configuration parameters: AutoCorrect, AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
 Layout/LineLength:
-  Max: 226
+  Max: 220

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    omni_exchange (1.1.1)
+    omni_exchange (1.3.0)
       faraday (= 0.17.4)
       money (~> 6.13.1)
 

data/README.md

@@ -57,7 +57,7 @@ To convert currency and/or get an exchange rate, all you have to do is call `Omn
 What you get back is a hash containing:
 1. converted_amount: (BigDecimal) the amount of money exchanged from the base currency to the target currency
 2. exchange_rate: (BigDecimal) the rate used to calculate the converted_amount
-3. provider_class: (Class) the name of the provider class that supplied the exchange_rate (ie. OmniExchange::OpenExhangeRates)
+3. provider: (Symbol) the provider that supplied the exchange_rate (ie. :open_exchange_rates, :xe)
 
 [For the sake of precise calculation](https://www.bigbinary.com/blog/handling-money-in-ruby), converted_amount and exchange_rate are BigDecimal. Simply call `.to_f` to the results if you'd like to see a number that is easier to read.
 
@@ -67,7 +67,7 @@ Here is an example. Lets say I want to convert $1.00 US Dollar to Japanese Yen,
 ```ruby
 USD_to_JPY = OmniExchange.get_fx_data(amount: 100, base_currency: 'USD', target_currency: 'JPY', providers: [:open_exchange_rates, :xe])
 
-puts USD_to_JPY # => { :converted_amount=>0.13566633333e3, :exchange_rate=>0.13566633333e1, :provider_class=>OmniExchange::OpenExchangeRates }
+puts USD_to_JPY # => { :converted_amount=>0.13566633333e3, :exchange_rate=>0.13566633333e1, :provider=>:open_exchange_rates }
 
 puts USD_to_JPY[:converted_amount] # => 0.13566633333e3
 puts USD_to_JPY[:converted_amount].to_f # => 135.66633333

data/bin/console

@@ -6,6 +6,7 @@ require 'omni_exchange'
 require 'dotenv/load'
 require 'faraday'
 require 'money'
+require 'net/http'
 require_relative '../lib/omni_exchange/provider'
 require_relative '../lib/omni_exchange/configuration'
 require_relative '../lib/omni_exchange/error'

data/lib/omni_exchange/error.rb

@@ -8,4 +8,8 @@ module OmniExchange
   # A custom error for failure to get data from a provider
   class HttpError < StandardError
   end
+
+  # A custom error for exceeding the monthly request limit for xe.com
+  class XeMonthlyLimit < StandardError
+  end
 end

data/lib/omni_exchange/providers/open_exchange_rates.rb

@@ -27,11 +27,15 @@ module OmniExchange
           req.options.timeout = config[:read_timeout] || OmniExchange::Configuration::DEFAULT_READ_TIMEOUT
           req.options.open_timeout = config[:connect_timeout] || OmniExchange::Configuration::DEFAULT_CONNECTION_TIMEOUT
         end
-      rescue Faraday::Error, Faraday::ConnectionFailed => e
+      rescue *EXCEPTIONS => e
         raise e.class, 'Open Exchange Rates has timed out.'
       end
 
-      exchange_rate = JSON.parse(response.body, symbolize_names: true)[:rates][target_currency.to_sym].to_d
+      begin
+        exchange_rate = JSON.parse(response.body, symbolize_names: true)[:rates][target_currency.to_sym].to_d
+      rescue JSON::ParserError => e
+        raise e.class, 'JSON::ParserError in OmniExchange::OpenExchangeRates'
+      end
 
       currency_unit = get_currency_unit(base_currency).to_d
 

data/lib/omni_exchange/providers/xe.rb

@@ -32,11 +32,19 @@ module OmniExchange
           req.options.timeout = config[:read_timeout] || OmniExchange::Configuration::DEFAULT_READ_TIMEOUT
           req.options.open_timeout = config[:connect_timeout] || OmniExchange::Configuration::DEFAULT_CONNECTION_TIMEOUT
         end
-      rescue Faraday::Error, Faraday::ConnectionFailed => e
+      rescue *EXCEPTIONS => e
         raise e.class, 'xe.com has timed out.'
       end
 
-      JSON.parse(response.body, symbolize_names: true)[:to][0][:mid].to_d
+      begin
+        body = JSON.parse(response.body, symbolize_names: true)
+      rescue JSON::ParserError => e
+        raise e.class, 'JSON::ParserError in OmniExchange::Xe'
+      end
+
+      raise OmniExchange::XeMonthlyLimit, 'Xe.com monthly limit has been exceeded' if body[:code] == 3
+
+      body[:to][0][:mid].to_d
     end
 
     # when this file is required at the top of lib/omni_exchange.rb, this method call is run and allows

data/lib/omni_exchange/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OmniExchange
-  VERSION = '1.1.1'
+  VERSION = '1.4.0'
 end

data/lib/omni_exchange.rb

@@ -12,6 +12,7 @@ require 'faraday'
 require 'money'
 require 'json'
 require 'bigdecimal/util'
+require 'net/http'
 
 # rubocop:disable Lint/Syntax
 module OmniExchange
@@ -32,6 +33,9 @@ module OmniExchange
     yield(configuration)
   end
 
+  # if a provider raises one of these exceptions, OmniExchange will gracefully attempt to use another provider
+  EXCEPTIONS = [Faraday::Error, Faraday::ConnectionFailed, Faraday::TimeoutError, Faraday::SSLError, Net::OpenTimeout, Net::WriteTimeout, Net::ReadTimeout, OpenSSL::SSL::SSLError]
+
   module_function
 
   # returns foreign exchange data including the amount of money in one country's currency when exchanged from an
@@ -44,15 +48,12 @@ module OmniExchange
   # @param providers: [Array] an array of symbols of the providers that will be used to get exchange rates API
   #   data. The symbols must be found in the @providers hash in the Provider class (lib/omni_exchange/provider.rb).
   #   ie. xe:, :open_exchange_rates
-  # @return [Hash] If all of the providers in the providers hash fail to retrieve data, or if one of the currencies
-  #   is not valid, an exception is raised.
+  # @return [Hash] If none of the providers in the providers hash are able to retrieve data, or if one of the
+  #   currencies is not valid, an exception is raised.
   #   * :converted_amount [BigDecimal] the amount of money exchanged from the base currency to the target
   #       currency as a BigDecimal for precice calculation. ie. 1, 10, 100
   #   * :exchange_rate [BigDecimal] the rate used to calculate the converted_amount as a BigDecimal. ie. 0.95211e1
-  #   * :provider_class [Class] the provider class that supplied the exchange_rate data. ie. OmniExchange::Xe
-  #the amount of the base currency exchanged to the target currency using an exchange rate
-  #   provided by one of the data providers in the providers hash. The final amount is returned as a BigDecimal
-  #   for precise calculation. If all of the providers in the providers hash fail to retrieve data, an exception is raised.
+  #   * :provider [Symbol] the provider that supplied the exchange_rate data. ie. :xe, :open_exchange_rates
   def get_fx_data(amount:, base_currency:, target_currency:, providers:)
     # if one of the currencies is not valid (ie. 'fake_crypto'), an exception is raised.
     begin
@@ -73,8 +74,8 @@ module OmniExchange
 
       exchanged_amount = rate.to_d * amount.to_d
 
-      return { converted_amount: exchanged_amount, exchange_rate: rate, provider_class: klass }
-    rescue Faraday::Error, Faraday::ConnectionFailed => e
+      return { converted_amount: exchanged_amount, exchange_rate: rate, provider: OmniExchange::Provider.all.key(klass) }
+    rescue *EXCEPTIONS, OmniExchange::XeMonthlyLimit, JSON::ParserError => e
       error_messages << e.inspect
     end
     

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omni_exchange
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.4.0
 platform: ruby
 authors:
 - Yun Chung
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-07-04 00:00:00.000000000 Z
+date: 2022-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday