money-open-exchange-rates 1.4.1 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8d1aa20a7e00ad9b92ff6f516af74c9f26a0b4a4cfacdbab246833806976cb7
-  data.tar.gz: 61646aab5bbcfb00e362df9f6fe8e0bc80b9ef553a99ac6bfd26fa1c7d8f5c93
+  metadata.gz: 86f324ed978cde7c5b05b50a8fe3d1ce4740333874457c8776c9871b3fa9ee31
+  data.tar.gz: ef5d0c06086d520f7a8816a69007b98d4741fb0063148ee0c66b9ec29a658b6d
 SHA512:
-  metadata.gz: 61eea8294a9b0b0a311db01d9b792c3831adc1d1711af486f9c43bac434bb5ced1dde20527624f16054e56cda867306bbf43f63dda9a17ef536da302cc77c250
-  data.tar.gz: 118f24d214bd24f7264aa575e87de2f04e115ebca9d83ce70c24abdd1cc0d93b2995f192ec760314717bbbed846ab75273ff851812a9fb6fd16a114043f63d76
+  metadata.gz: 5361f382ab1462ae3c6b1a18e543c1dbe4d8d31f68278c460092d8fd93b65a9fd9070f290915a498b1b9bb10b2c6c6c6fb3bff356a90f0f0b75b12d296914990
+  data.tar.gz: d85876e62c4f13ebe24c71a273cb6667e7a331707cd7f48e90393b1d620b51593ff03cc77bc24cac78075fd21983290d99cba5eb62bfdb3c2235d1b790669f8d

data/History.md

@@ -1,4 +1,17 @@
 
+1.4.2 / 2023-07-25
+==================
+
+  * gitlab: Ruby 3.2 in, 2.6 out
+  * Merge pull request #70 from @swiknaba / master
+    * CI: nicer name, latest versions of Github actions
+    * CI: Ruby 3.2 in, 2.6 out
+  * Bump year
+  * Update jruby version
+  * Update rubocop target ruby version
+  * Update Rubies on CIs
+  * Handle AppIdInactive error
+
 1.4.1 / 2023-05-07
 ==================
 

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License
 
-Copyright (c) 2011-2020 Laurent Arnoud <laurent@spkdev.net>
+Copyright (c) 2011-2023 Laurent Arnoud <laurent@spkdev.net>
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -249,7 +249,7 @@ See [GitHub](https://github.com/spk/money-open-exchange-rates/graphs/contributor
 
 The MIT License
 
-Copyright © 2011-2020 Laurent Arnoud <laurent@spkdev.net>
+Copyright © 2011-2023 Laurent Arnoud <laurent@spkdev.net>
 
 ---
 [![Build](https://img.shields.io/gitlab/pipeline/spkdev/money-open-exchange-rates/master)](https://gitlab.com/spkdev/money-open-exchange-rates/-/commits/master)

data/lib/money/bank/open_exchange_rates_bank.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
+require 'json'
+require 'money'
 require 'net/http'
+require 'time'
 require 'uri'
-require 'money'
-require 'json'
 require File.expand_path('../../open_exchange_rates_bank/version', __dir__)
 
 # Money gem class
@@ -20,6 +21,14 @@ class Money
     # Access restricted (e.g. usage/request limit exceeded for account)
     class AccessRestricted < StandardError; end
 
+    # app_id_inactive
+    class AppIdInactive < StandardError; end
+
+    ERROR_MAP = {
+      access_restricted: AccessRestricted,
+      app_id_inactive: AppIdInactive
+    }.freeze
+
     # OpenExchangeRatesBank base class
     class OpenExchangeRatesBank < Money::Bank::VariableExchange
       VERSION = ::OpenExchangeRatesBank::VERSION
@@ -374,8 +383,8 @@ class Money
       # @return [Hash] key is country code (ISO 3166-1 alpha-3) value Float
       def exchange_rates
         doc = JSON.parse(read_from_cache || read_from_url)
-        if doc['error'] && doc['message'] == 'access_restricted'
-          raise AccessRestricted
+        if doc['error'] && ERROR_MAP.key?(doc['message'].to_sym)
+          raise ERROR_MAP[doc['message'].to_sym]
         end
 
         self.rates_timestamp = doc[TIMESTAMP_KEY]

data/lib/open_exchange_rates_bank/version.rb

@@ -2,5 +2,5 @@
 
 # Module for version constant
 module OpenExchangeRatesBank
-  VERSION = '1.4.1'
+  VERSION = '1.4.2'
 end

data/test/data/app_id_inactive.json

@@ -0,0 +1 @@
+{"error": true, "status": 401, "message": "app_id_inactive", "description": "This App ID has been deactivated. Please use another or contact support@openexchangerates.org for assistance."}

data/test/integration/Gemfile

@@ -1,4 +1,6 @@
 # frozen_string_literal: true
 
+source 'https://rubygems.org'
+
 gem 'money', '~> 6.7'
 gem 'money-open-exchange-rates', path: '../../'

data/test/integration/Gemfile.lock

@@ -1,26 +1,24 @@
 PATH
-  remote: ../../
+  remote: ../..
   specs:
-    money-open-exchange-rates (0.6.1)
-      monetize (~> 1.4)
-      money (~> 6.7)
+    money-open-exchange-rates (1.4.1)
+      money (~> 6.12)
 
 GEM
+  remote: https://rubygems.org/
   specs:
-    i18n (0.7.0)
-    monetize (1.4.0)
-      money (~> 6.7)
-    money (6.7.1)
-      i18n (>= 0.6.4, <= 0.7.0)
-      sixarm_ruby_unaccent (>= 1.1.1, < 2)
-    sixarm_ruby_unaccent (1.1.1)
+    concurrent-ruby (1.2.2)
+    i18n (1.13.0)
+      concurrent-ruby (~> 1.0)
+    money (6.16.0)
+      i18n (>= 0.6.4, <= 2)
 
 PLATFORMS
-  ruby
+  x86_64-linux
 
 DEPENDENCIES
   money (~> 6.7)
   money-open-exchange-rates!
 
 BUNDLED WITH
-   1.13.1
+   2.4.12

data/test/open_exchange_rates_bank_test.rb

@@ -28,6 +28,9 @@ describe Money::Bank::OpenExchangeRatesBank do
   let(:oer_access_restricted_error_path) do
     data_file('access_restricted_error.json')
   end
+  let(:oer_app_id_inactive_error_path) do
+    data_file('app_id_inactive.json')
+  end
 
   describe 'exchange' do
     before do
@@ -106,6 +109,13 @@ describe Money::Bank::OpenExchangeRatesBank do
       _(proc { subject.update_rates }).must_raise Money::Bank::AccessRestricted
     end
 
+    it 'should raise AppIdInactive error when restricted by oer' do
+      subject.cache = nil
+      filepath = oer_app_id_inactive_error_path
+      subject.stubs(:api_response).returns File.read(filepath)
+      _(proc { subject.update_rates }).must_raise Money::Bank::AppIdInactive
+    end
+
     it 'should update itself with exchange rates from OpenExchangeRates' do
       subject.oer_rates.keys.each do |currency|
         next unless Money::Currency.find(currency)
@@ -295,13 +305,11 @@ describe Money::Bank::OpenExchangeRatesBank do
     end
 
     it 'should allow update after save' do
-      begin
-        subject.refresh_rates
-        # rubocop:disable Style/RescueStandardError
-      rescue
-        # rubocop:enable Style/RescueStandardError
-        assert false, 'Should allow updating after saving'
-      end
+      subject.refresh_rates
+      # rubocop:disable Style/RescueStandardError
+    rescue
+      # rubocop:enable Style/RescueStandardError
+      assert false, 'Should allow updating after saving'
     end
 
     it 'should not break an existing file if save fails to read' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: money-open-exchange-rates
 version: !ruby/object:Gem::Version
-  version: 1.4.1
+  version: 1.4.2
 platform: ruby
 authors:
 - Laurent Arnoud
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-07 00:00:00.000000000 Z
+date: 2023-07-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: money
@@ -144,6 +144,7 @@ files:
 - lib/open_exchange_rates_bank/version.rb
 - test/data/2015-01-01.json
 - test/data/access_restricted_error.json
+- test/data/app_id_inactive.json
 - test/data/latest.json
 - test/integration/Gemfile
 - test/integration/Gemfile.lock
@@ -162,7 +163,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="