danconia 0.2.8 → 0.4.0

data/spec/danconia/money_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 module Danconia
   describe Money do
     context 'instantiation' do
@@ -39,16 +37,15 @@ module Danconia
       end
 
       it 'should exchange the other currency if it is different' do
-        expect(Money(1, 'ARS', exchange: fake_exchange(rate: 4)) + Money(1, 'USD')).to eq Money(5, 'ARS')
+        m1 = Money(1, 'ARS', exchange_opts: {exchange: fake_exchange(rate: 4)})
+        expect(m1 + Money(1, 'USD')).to eq Money(5, 'ARS')
       end
 
       it 'should return a new object with the same options' do
-        e = fake_exchange
-        m1 = Money(4, decimals: 3, exchange: e)
+        m1 = Money(4, decimals: 3)
         m2 = m1 * 2
         expect(m2).to_not eql m1
         expect(m2.decimals).to eq 3
-        expect(m2.exchange).to eq e
       end
 
       it 'round should return a money object with the same currency' do
@@ -72,7 +69,7 @@ module Danconia
       end
 
       it 'should exchange to the source currency if they differ' do
-        TestHelpers.with_rates 'USDARS' => 4 do |config|
+        TestHelpers.with_rates 'USDARS' => 4 do |_config|
           expect(Money(3, 'ARS') < Money(1, 'USD')).to be true
           expect(Money(4, 'ARS') < Money(1, 'USD')).to be false
         end
@@ -129,7 +126,8 @@ module Danconia
       end
 
       it 'should allow to pass the exchange to the instance' do
-        expect(Money(2, 'USD', exchange: fake_exchange(rate: 3)).exchange_to('ARS')).to eq Money(6, 'ARS')
+        m = Money(2, 'USD', exchange_opts: {exchange: fake_exchange(rate: 3)})
+        expect(m.exchange_to('ARS')).to eq Money(6, 'ARS')
       end
 
       it 'should allow to pass the exchange when converting' do
@@ -153,8 +151,8 @@ module Danconia
       end
 
       it 'should return a new object with the same opts' do
-        m1 = Money(1, 'USD', decimals: 0, exchange: fake_exchange(rate: 3))
-        m2 = m1.exchange_to('ARS')
+        m1 = Money(1, 'USD', decimals: 0)
+        m2 = m1.exchange_to('ARS', exchange: fake_exchange(rate: 3))
         expect(m2).to_not eql m1
         expect(m2.decimals).to eq 0
         expect(m1).to eq Money(1, 'USD')
@@ -164,6 +162,38 @@ module Danconia
         expect(Money(1, 'USD').exchange_to('')).to eq Money(1, 'USD')
         expect(Money(1, 'ARS').exchange_to('')).to eq Money(1, 'ARS')
       end
+
+      context 'opts' do
+        let(:exchange) do
+          Class.new(Exchange) do
+            def rates opts
+              case opts[:type]
+              when 'divisa' then {'USDARS' => 7}
+              when 'billete' then {'USDARS' => 8}
+              else {}
+              end
+            end
+          end.new
+        end
+
+        it 'allows to specify opts to pass to the exchange (filters for example)' do
+          expect(Money(1, 'USD').exchange_to('ARS', type: 'divisa', exchange: exchange)).to eq Money(7, 'ARS')
+          expect { Money(1, 'USD').exchange_to('ARS', exchange: exchange) }.to raise_error Errors::ExchangeRateNotFound
+        end
+
+        it 'should be preserved after operations' do
+          m1 = Money(1, 'USD').exchange_to('ARS', type: 'divisa', exchange: exchange)
+          m2 = m1 + Money(2, 'USD')
+
+          expect(m2).to eq Money(21, 'ARS')
+          expect(m1 < Money(2, 'USD')).to eq true
+        end
+
+        it 'should use the instance exchange_opts by default' do
+          m = Money(1, 'USD', exchange_opts: {exchange: exchange, type: 'billete'})
+          expect(m.exchange_to('ARS')).to eq Money(8, 'ARS')
+        end
+      end
     end
 
     context 'default_currency?' do
@@ -183,14 +213,8 @@ module Danconia
       end
     end
 
-    context 'to_json' do
-      it 'should delegate to the amount' do
-        expect(Money(1).to_json).to eq '"1.0"'
-      end
-    end
-
     def fake_exchange args = {}
-      double 'Danconia::Exchanges::Exchange', args.reverse_merge(rate: nil)
+      double 'Danconia::Exchange', args.reverse_merge(rate: nil)
     end
   end
 end

data/spec/danconia/serializable_spec.rb

@@ -0,0 +1,16 @@
+module Danconia
+  describe Serializable do
+    context 'marshalling' do
+      it 'money objects support dump and load' do
+        expect(Marshal.load(Marshal.dump(Money(5, 'USD')))).to eq Money(5, 'USD')
+        expect(Marshal.load(Marshal.dump(Money(3.2, 'ARS')))).to eq Money(3.2, 'ARS')
+      end
+    end
+
+    context 'to_json' do
+      it 'should delegate to the amount' do
+        expect(Money(1).to_json).to eq %({"amount":"1.0","currency":"USD"})
+      end
+    end
+  end
+end

data/spec/danconia/stores/active_record_spec.rb

@@ -1,31 +1,97 @@
-require 'spec_helper'
-
 module Danconia
   module Stores
     describe ActiveRecord, active_record: true do
+      before do
+        ::ActiveRecord::Schema.define do
+          create_table :exchange_rates do |t|
+            t.string :pair, limit: 6
+            t.decimal :rate, precision: 12, scale: 6
+            t.string :rate_type
+            t.date :date
+          end
+        end
+      end
+
       context 'save_rates' do
         it 'should create or update the rates' do
           ExchangeRate.create! pair: 'USDEUR', rate: 2
-          expect { subject.save_rates 'USDEUR' => 3, 'USDARS' => 4 }.to change { ExchangeRate.count }.by 1
-          expect(subject.rates).to eq('USDEUR' => 3, 'USDARS' => 4)
+
+          expect do
+            subject.save_rates [{pair: 'USDEUR', rate: 3}, {pair: 'USDARS', rate: 4}]
+          end.to change { ExchangeRate.count }.by 1
+
+          expect(subject.rates).to match [
+            include(pair: 'USDEUR', rate: 3),
+            include(pair: 'USDARS', rate: 4)
+          ]
+        end
+
+        it 'allows to specify other keys to use as unique' do
+          store = ActiveRecord.new(unique_keys: %i[pair rate_type])
+          store.save_rates [
+            {pair: 'USDARS', rate: 3, rate_type: 'billetes'},
+            {pair: 'USDARS', rate: 4, rate_type: 'divisas'}
+          ]
+          store.save_rates [
+            {pair: 'USDARS', rate: 33, rate_type: 'billetes'}
+          ]
+          expect(subject.rates).to match [
+            include(pair: 'USDARS', rate: 33, rate_type: 'billetes'),
+            include(pair: 'USDARS', rate: 4, rate_type: 'divisas')
+          ]
+        end
+
+        it 'ignores fields not present in the database table' do
+          subject.save_rates [{pair: 'USDEUR', rate: 3, non_existant: 'ignoreme'}]
         end
       end
 
-      context '#direct_rate' do
-        it 'should find the rate for the pair' do
+      context 'rates' do
+        it 'returns an array like the one it received' do
           ExchangeRate.create! pair: 'USDEUR', rate: 2
-          expect(subject.direct_rate('USD', 'EUR')).to eq 2
-          expect(subject.direct_rate('USD', 'ARS')).to eq nil
+          ExchangeRate.create! pair: 'USDARS', rate: 40
+
+          expect(subject.rates).to match [
+            include(pair: 'USDEUR', rate: 2),
+            include(pair: 'USDARS', rate: 40)
+          ]
+        end
+
+        it 'allows to pass filters' do
+          store = ActiveRecord.new(unique_keys: %i[pair date])
+          store.save_rates [
+            {pair: 'USDEUR', rate: 10, date: Date.new(2020, 1, 1)},
+            {pair: 'USDEUR', rate: 20, date: Date.new(2020, 1, 2)},
+            {pair: 'USDEUR', rate: 30, date: Date.new(2020, 1, 3)}
+          ]
+
+          expect(store.rates.size).to eq 3
+          expect(store.rates(date: Date.new(2020, 1, 2))).to match [include(rate: 20)]
         end
       end
 
-      before do
-        ::ActiveRecord::Schema.define do
-          create_table :exchange_rates do |t|
-            t.string :pair, limit: 6
-            t.decimal :rate, precision: 12, scale: 6
-            t.index :pair, unique: true
-          end
+      context 'special date field' do
+        let(:store) do
+          store = ActiveRecord.new(unique_keys: %i[pair date], date_field: :date)
+          store.save_rates [
+            {pair: 'USDEUR', rate: 10, date: Date.new(2000, 1, 1)},
+            {pair: 'USDEUR', rate: 20, date: Date.new(2000, 1, 2)},
+            {pair: 'USDEUR', rate: 30, date: Date.new(2000, 1, 4)}
+          ]
+          store
+        end
+
+        it 'calling #rates with a particular date when there are rates for that date' do
+          expect(store.rates(date: Date.new(2000, 1, 2))).to match [include(rate: 20)]
+        end
+
+        it 'calling #rates with a particular date when there are not rates for that date should return the previous' do
+          expect(store.rates(date: Date.new(2000, 1, 3))).to match [include(rate: 20)]
+          expect(store.rates(date: Date.new(1999, 12, 31))).to eq []
+        end
+
+        it 'calling #rates without a particular date, uses today' do
+          expect(store.rates).to match [include(rate: 30)]
         end
       end
     end

data/spec/danconia/stores/in_memory_spec.rb

@@ -0,0 +1,18 @@
+module Danconia
+  module Stores
+    describe InMemory do
+      context 'rates' do
+        it 'filters rates by type' do
+          subject.save_rates [
+            {pair: 'USDARS', rate: 3, rate_type: 'divisas'},
+            {pair: 'USDARS', rate: 4, rate_type: 'billetes'}
+          ]
+
+          expect(subject.rates(rate_type: 'divisas')).to match [include(rate: 3)]
+          expect(subject.rates(rate_type: 'billetes')).to match [include(rate: 4)]
+          expect(subject.rates).to match [include(rate: 3), include(rate: 4)]
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,8 +1,9 @@
 require 'danconia'
 require 'danconia/test_helpers'
+require 'danconia/integrations/active_record'
 require 'webmock/rspec'
 
-Dir["#{__dir__}/support/*.rb"].each { |f| require f }
+Dir["#{__dir__}/support/*.rb"].sort.each { |f| require f }
 
 RSpec.configure do |config|
   config.filter_run_when_matching :focus

metadata

@@ -1,29 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: danconia
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.4.0
 platform: ruby
 authors:
 - Emmanuel Nicolau
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-31 00:00:00.000000000 Z
+date: 1980-01-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: activerecord
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '4.0'
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -108,14 +122,45 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.80.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.80.0
 description: Multi-currency money library backed by BigDecimal
 email: emmanicolau@gmail.com
-executables: []
+executables:
+- console
 extensions: []
 extra_rdoc_files: []
 files:
+- ".envrc"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - ".ruby-version"
 - ".travis.yml"
 - Gemfile
@@ -124,7 +169,9 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- bin/console
 - danconia.gemspec
+- examples/bna.rb
 - examples/currency_layer.rb
 - examples/fixed_rates.rb
 - examples/single_currency.rb
@@ -133,28 +180,38 @@ files:
 - lib/danconia/currency.rb
 - lib/danconia/errors/api_error.rb
 - lib/danconia/errors/exchange_rate_not_found.rb
+- lib/danconia/exchange.rb
+- lib/danconia/exchanges/bna.rb
 - lib/danconia/exchanges/currency_layer.rb
-- lib/danconia/exchanges/exchange.rb
 - lib/danconia/exchanges/fixed_rates.rb
 - lib/danconia/integrations/active_record.rb
 - lib/danconia/kernel.rb
 - lib/danconia/money.rb
+- lib/danconia/pair.rb
+- lib/danconia/serializable.rb
 - lib/danconia/stores/active_record.rb
 - lib/danconia/stores/in_memory.rb
 - lib/danconia/test_helpers.rb
 - lib/danconia/version.rb
+- shell.nix
+- spec/danconia/exchanges/bna_spec.rb
 - spec/danconia/exchanges/currency_layer_spec.rb
-- spec/danconia/exchanges/fixed_rates_spec.rb
+- spec/danconia/exchanges/exchange_spec.rb
+- spec/danconia/exchanges/fixtures/bna/home.html
+- spec/danconia/exchanges/fixtures/currency_layer/failure.json
+- spec/danconia/exchanges/fixtures/currency_layer/success.json
 - spec/danconia/integrations/active_record_spec.rb
 - spec/danconia/money_spec.rb
+- spec/danconia/serializable_spec.rb
 - spec/danconia/stores/active_record_spec.rb
+- spec/danconia/stores/in_memory_spec.rb
 - spec/spec_helper.rb
 - spec/support/database.rb
 homepage: https://github.com/eeng/danconia
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -169,16 +226,21 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.3
-signing_key: 
+rubygems_version: 3.2.26
+signing_key:
 specification_version: 4
 summary: Multi-currency money library backed by BigDecimal
 test_files:
+- spec/danconia/exchanges/bna_spec.rb
 - spec/danconia/exchanges/currency_layer_spec.rb
-- spec/danconia/exchanges/fixed_rates_spec.rb
+- spec/danconia/exchanges/exchange_spec.rb
+- spec/danconia/exchanges/fixtures/bna/home.html
+- spec/danconia/exchanges/fixtures/currency_layer/failure.json
+- spec/danconia/exchanges/fixtures/currency_layer/success.json
 - spec/danconia/integrations/active_record_spec.rb
 - spec/danconia/money_spec.rb
+- spec/danconia/serializable_spec.rb
 - spec/danconia/stores/active_record_spec.rb
+- spec/danconia/stores/in_memory_spec.rb
 - spec/spec_helper.rb
 - spec/support/database.rb

data/lib/danconia/exchanges/exchange.rb

@@ -1,33 +0,0 @@
-module Danconia
-  module Exchanges
-    class Exchange
-      attr_reader :store
-
-      def initialize store: Stores::InMemory.new
-        @store = store
-      end
-
-      def rate from, to
-        if from == to
-          1.0
-        elsif from == 'USD' and direct_rate = @store.direct_rate(from, to)
-          direct_rate
-        elsif to == 'USD' and inverse_rate = @store.direct_rate(to, from)
-          (1.0 / inverse_rate).round 6
-        elsif from != 'USD' and to != 'USD' and from_in_usd = rate(from, 'USD') and to_per_usd = rate('USD', to)
-          (from_in_usd * to_per_usd).round 6
-        else
-          raise Errors::ExchangeRateNotFound.new(from, to)
-        end
-      end
-
-      def rates
-        @store.rates
-      end
-
-      def update_rates!
-        @store.save_rates fetch_rates
-      end
-    end
-  end
-end

data/spec/danconia/exchanges/fixed_rates_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-
-module Danconia
-  module Exchanges
-    describe FixedRates do
-      context 'rate' do
-        it 'returns the exchange rate value for the supplied currencies' do
-          exchange = FixedRates.new rates: {'USDEUR' => 3, 'USDARS' => 4}
-          expect(exchange.rate 'USD', 'EUR').to eq 3
-          expect(exchange.rate 'USD', 'ARS').to eq 4
-        end
-
-        it 'returns nil if not found' do
-          expect { subject.rate 'USD', 'EUR' }.to raise_error Errors::ExchangeRateNotFound
-        end
-
-        it 'if the direct conversion is not found, tries to find the inverse' do
-          exchange = FixedRates.new rates: {'USDEUR' => 3}
-          expect(exchange.rate 'EUR', 'USD').to eq (1.0 / 3).round 6
-        end
-
-        it 'if not direct nor inverse conversion is found and both are different than USD, tries to convert through USD' do
-          exchange = FixedRates.new rates: {'USDEUR' => 3, 'USDARS' => 6}
-          expect(exchange.rate 'EUR', 'ARS').to be_within(0.00001).of 2
-          expect(exchange.rate 'ARS', 'EUR').to be_within(0.00001).of 0.5
-        end
-      end
-    end
-  end
-end