shopify-money 0.14.8 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 186250af5f49f67f218ce8b30f47cd6c20ff762f9fdb2813ca9ffe1d12a7700d
-  data.tar.gz: 1e6ce179c120a2781dc0be48ee0377c4179e6dc07c8fd6306689947a0847daf4
+  metadata.gz: 56953634d128da5cb62dfb11b5c60868a162c4ff7edb9bb810110d17c02469f9
+  data.tar.gz: c8e6f0e99b480cfcf9be4183adfc633baf14f0f9ee0d777bea01b225c01cfc09
 SHA512:
-  metadata.gz: 3d2076a57d0b364a782ce10ffe8961e8ca57b535b8c12ba006887a92336902694ecf1a9b34538632bdad51ea3bbbf2e44aa15988344a406d16890168c72823a8
-  data.tar.gz: 8b539f90a996544a93fbb733dbde048460a4a5069e1efef6e44b63ae8e790dd6f90a407b7e4ae0fbbb2d850864b5015204ec8a4663b2d45673589611e16ce02b
+  metadata.gz: 4888a59e2f50348092b73c2b7255b215f994f9b47efbd379ad88d6f4624f4eca403840f584f4d5760a17c02d20f38f0fc49f286c0e19e3557e4976037d1f8049
+  data.tar.gz: ce77e720fae3724a0cc8303336ac12aee09a7a61862f0ba01f0dc7dce3c4e86f2fd7073ee7828d5cc313b77d9b285dfce07bd84e1d8e289e597e1cf25a349c7d

data/lib/money/currency.rb

@@ -50,6 +50,10 @@ class Money
       self.class == other.class && iso_code == other.iso_code
     end
 
+    def hash
+      [ self.class, iso_code ].hash
+    end
+
     def compatible?(other)
       other.is_a?(NullCurrency) || eql?(other)
     end

data/lib/money/money.rb

@@ -26,19 +26,10 @@ class Money
     end
     alias_method :from_amount, :new
 
-    def zero(currency = NULL_CURRENCY)
-      new(0, currency)
-    end
-    alias_method :empty, :zero
-
     def parse(*args, **kwargs)
       parser.parse(*args, **kwargs)
     end
 
-    def from_cents(cents, currency = nil)
-      new(cents.round.to_f / 100, currency)
-    end
-
     def from_subunits(subunits, currency_iso, format: :iso4217)
       currency = Helpers.value_to_currency(currency_iso)
 
@@ -106,11 +97,6 @@ class Money
     coder['currency'] = @currency.iso_code
   end
 
-  def cents
-    # Money.deprecate('`money.cents` is deprecated and will be removed in the next major release. Please use `money.subunits` instead. Keep in mind, subunits are currency aware.')
-    (value * 100).to_i
-  end
-
   def subunits(format: :iso4217)
     subunit_to_unit_value = if format == :iso4217
       @currency.subunit_to_unit
@@ -243,10 +229,6 @@ class Money
     end
   end
 
-  def to_liquid
-    cents
-  end
-
   def to_json(options = {})
     to_s
   end

data/lib/money/null_currency.rb

@@ -1,5 +1,36 @@
 # frozen_string_literal: true
 class Money
+  # A placeholder currency for instances where no actual currency is available,
+  # as defined by ISO4217. You should rarely, if ever, need to use this
+  # directly. It's here mostly for backwards compatibility and for that reason
+  # behaves like a dollar, which is how this gem worked before the introduction
+  # of currency.
+  #
+  # Here follows a list of preferred alternatives over using Money with
+  # NullCurrency:
+  #
+  # For comparisons where you don't know the currency beforehand, you can use
+  # Numeric predicate methods like #positive?/#negative?/#zero?/#nonzero?.
+  # Comparison operators with Numeric (==, !=, <=, =>, <, >) work as well.
+  #
+  # @example
+  #   Money.new(1, 'CAD').positive? #=> true
+  #   Money.new(2, 'CAD') >= 0      #=> true
+  #
+  # Money with NullCurrency has behaviour that may surprise you, such as
+  # database validations or GraphQL enum not allowing the string representation
+  # of NullCurrency. Prefer using Money.new(0, currency) where possible, as
+  # this sidesteps these issues and provides additional currency check
+  # safeties.
+  #
+  # Unlike other currencies, it is allowed to calculate a Money object with
+  # NullCurrency with another currency. The resulting Money object will have
+  # the other currency.
+  #
+  # @example
+  #   Money.new(0, Money::NULL_CURRENCY) + Money.new(5, 'CAD')
+  #   #=> #<Money value:5.00 currency:CAD>
+  #
   class NullCurrency
 
     attr_reader :iso_code, :iso_numeric, :name, :smallest_denomination, :subunit_symbol,
@@ -9,7 +40,7 @@ class Money
       @symbol                = '$'
       @disambiguate_symbol   = nil
       @subunit_symbol        = nil
-      @iso_code              = 'XXX' # Valid ISO4217
+      @iso_code              = 'XXX'
       @iso_numeric           = '999'
       @name                  = 'No Currency'
       @smallest_denomination = 1

data/lib/money/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 class Money
-  VERSION = "0.14.8"
+  VERSION = "0.15.0"
 end

data/lib/rubocop/cop/money.rb

@@ -1,3 +1,4 @@
 # frozen_string_literal: true
 
 require 'rubocop/cop/money/missing_currency'
+require 'rubocop/cop/money/zero_money'

data/lib/rubocop/cop/money/missing_currency.rb

@@ -20,11 +20,11 @@ module RuboCop
         #
 
         def_node_matcher :money_new, <<~PATTERN
-          (send (const nil? :Money) {:new :from_amount :from_cents} $...)
+          (send (const {nil? cbase} :Money) {:new :from_amount :from_cents} $...)
         PATTERN
 
         def_node_matcher :to_money_without_currency?, <<~PATTERN
-          (send _ :to_money)
+          ({send csend} _ :to_money)
         PATTERN
 
         def_node_matcher :to_money_block?, <<~PATTERN
@@ -42,11 +42,9 @@ module RuboCop
             add_offense(node, message: 'to_money is missing currency argument')
           end
         end
+        alias on_csend on_send
 
         def autocorrect(node)
-          currency = cop_config['ReplacementCurrency']
-          return unless currency
-
           receiver, method, _ = *node
 
           lambda do |corrector|
@@ -55,20 +53,30 @@ module RuboCop
 
               corrector.replace(
                 node.loc.expression,
-                "#{receiver.source}.#{method}(#{amount&.source || 0}, '#{currency}')"
+                "#{receiver.source}.#{method}(#{amount&.source || 0}, #{replacement_currency})"
               )
             end
 
             if to_money_without_currency?(node)
-              corrector.insert_after(node.loc.expression, "('#{currency}')")
+              corrector.insert_after(node.loc.expression, "(#{replacement_currency})")
             elsif to_money_block?(node)
               corrector.replace(
                 node.loc.expression,
-                "#{receiver.source}.#{method} { |x| x.to_money('#{currency}') }"
+                "#{receiver.source}.#{method} { |x| x.to_money(#{replacement_currency}) }"
               )
             end
           end
         end
+
+        private
+
+        def replacement_currency
+          if cop_config['ReplacementCurrency']
+            "'#{cop_config['ReplacementCurrency']}'"
+          else
+            'Money::NULL_CURRENCY'
+          end
+        end
       end
     end
   end

data/lib/rubocop/cop/money/zero_money.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Money
+      class ZeroMoney < Cop
+        # `Money.zero` and it's alias `empty`, with or without currency
+        # argument is removed in favour of the more explicit Money.new
+        # syntax. Supplying it with a real currency is preferred for
+        # additional currency safety checks.
+        #
+        # If no currency was supplied, it defaults to
+        # Money::NULL_CURRENCY which was the default setting of
+        # Money.default_currency and should effectively be the same. The cop
+        # can be configured with a ReplacementCurrency in case that is more
+        # appropriate for your application.
+        #
+        # @example
+        #
+        #   # bad
+        #   Money.zero
+        #
+        #   # good when configured with `ReplacementCurrency: CAD`
+        #   Money.new(0, 'CAD')
+        #
+
+        MSG = 'Money.zero is removed, use `Money.new(0, %<currency>s)`.'
+
+        def_node_matcher :money_zero, <<~PATTERN
+          (send (const {nil? cbase} :Money) {:zero :empty} $...)
+        PATTERN
+
+        def on_send(node)
+          money_zero(node) do |currency_arg|
+            add_offense(node, message: format(MSG, currency: replacement_currency(currency_arg)))
+          end
+        end
+
+        def autocorrect(node)
+          receiver, _ = *node
+
+          lambda do |corrector|
+            money_zero(node) do |currency_arg|
+              replacement_currency = replacement_currency(currency_arg)
+
+              corrector.replace(
+                node.loc.expression,
+                "#{receiver.source}.new(0, #{replacement_currency})"
+              )
+            end
+          end
+        end
+
+        private
+
+        def replacement_currency(currency_arg)
+          return currency_arg.first.source unless currency_arg.empty?
+          return "'#{cop_config['ReplacementCurrency']}'" if cop_config['ReplacementCurrency']
+
+          'Money::NULL_CURRENCY'
+        end
+      end
+    end
+  end
+end

data/spec/allocator_spec.rb

@@ -43,7 +43,7 @@ RSpec.describe "Allocator" do
 
     specify "#allocate will convert rationals with high precision" do
       ratios = [Rational(1, 1), Rational(0)]
-      expect(new_allocator("858993456.12").allocate(ratios)).to eq([Money.new("858993456.12"), Money.empty])
+      expect(new_allocator("858993456.12").allocate(ratios)).to eq([Money.new("858993456.12"), Money.new(0, Money::NULL_CURRENCY)])
       ratios = [Rational(1, 6), Rational(5, 6)]
       expect(new_allocator("3.00").allocate(ratios)).to eq([Money.new("0.50"), Money.new("2.50")])
     end
@@ -131,8 +131,8 @@ RSpec.describe "Allocator" do
 
     specify "#allocate_max_amounts supports all-zero maxima" do
       expect(
-        new_allocator(3).allocate_max_amounts([Money.empty, Money.empty, Money.empty]),
-      ).to eq([Money.empty, Money.empty, Money.empty])
+        new_allocator(3).allocate_max_amounts([Money.new(0, Money::NULL_CURRENCY), Money.new(0, Money::NULL_CURRENCY), Money.new(0, Money::NULL_CURRENCY)]),
+      ).to eq([Money.new(0, Money::NULL_CURRENCY), Money.new(0, Money::NULL_CURRENCY), Money.new(0, Money::NULL_CURRENCY)])
     end
 
     specify "#allocate_max_amounts allocates the right amount without rounding error" do

data/spec/core_extensions_spec.rb

@@ -17,7 +17,7 @@ RSpec.describe Integer do
   it_should_behave_like "an object supporting to_money"
 
   it "parses 0 to Money.zero" do
-    expect(0.to_money).to eq(Money.zero)
+    expect(0.to_money).to eq(Money.new(0, Money::NULL_CURRENCY))
   end
 end
 
@@ -30,7 +30,7 @@ RSpec.describe Float do
   it_should_behave_like "an object supporting to_money"
 
   it "parses 0.0 to Money.zero" do
-    expect(0.0.to_money).to eq(Money.zero)
+    expect(0.0.to_money).to eq(Money.new(0, Money::NULL_CURRENCY))
   end
 end
 
@@ -43,8 +43,8 @@ RSpec.describe String do
   it_should_behave_like "an object supporting to_money"
 
   it "parses an empty string to Money.zero" do
-    expect(''.to_money).to eq(Money.zero)
-    expect(' '.to_money).to eq(Money.zero)
+    expect(''.to_money).to eq(Money.new(0, Money::NULL_CURRENCY))
+    expect(' '.to_money).to eq(Money.new(0, Money::NULL_CURRENCY))
   end
 end
 
@@ -57,6 +57,6 @@ RSpec.describe BigDecimal do
   it_should_behave_like "an object supporting to_money"
 
   it "parses a zero BigDecimal to Money.zero" do
-    expect(BigDecimal("-0.000").to_money).to eq(Money.zero)
+    expect(BigDecimal("-0.000").to_money).to eq(Money.new(0, Money::NULL_CURRENCY))
   end
 end

data/spec/currency_spec.rb

@@ -77,6 +77,16 @@ RSpec.describe "Currency" do
     end
   end
 
+  describe "#hash" do
+    specify "equal currencies from different loaders have the same hash" do
+      currency_1 = Money::Currency.find('USD')
+      currency_2 = YAML.load(Money::Currency.find('USD').to_yaml)
+
+      expect(currency_1.eql?(currency_2)).to eq(true)
+      expect(currency_1.hash).to eq(currency_2.hash)
+    end
+  end
+
   describe "==" do
     it "returns true when both objects have the same currency" do
       expect(currency == Money.new(1, 'USD').currency).to eq(true)

data/spec/money_column_spec.rb

@@ -136,7 +136,7 @@ RSpec.describe 'MoneyColumn' do
 
   it 'does not overwrite a currency column with a default currency when saving zero' do
     expect(record.currency.to_s).to eq('EUR')
-    record.update(price: Money.zero)
+    record.update(price: Money.new(0, Money::NULL_CURRENCY))
     expect(record.currency.to_s).to eq('EUR')
   end
 

data/spec/money_parser_spec.rb

@@ -8,12 +8,12 @@ RSpec.describe MoneyParser do
 
   describe "parsing of amounts with period decimal separator" do
     it "parses an empty string to $0" do
-      expect(@parser.parse("")).to eq(Money.zero)
+      expect(@parser.parse("")).to eq(Money.new(0, Money::NULL_CURRENCY))
     end
 
     it "parses an invalid string when not strict" do
       expect(Money).to receive(:deprecate).twice
-      expect(@parser.parse("no money")).to eq(Money.zero)
+      expect(@parser.parse("no money")).to eq(Money.new(0, Money::NULL_CURRENCY))
       expect(@parser.parse("1..")).to eq(Money.new(1))
     end
 

data/spec/money_spec.rb

@@ -9,10 +9,6 @@ RSpec.describe "Money" do
   let (:non_fractional_money) { Money.new(1, 'JPY') }
   let (:zero_money) { Money.new(0) }
 
-  it "is contructable with empty class method" do
-    expect(Money.empty).to eq(Money.new)
-  end
-
   context "default currency not set" do
     before(:each) do
       @default_currency = Money.default_currency
@@ -28,15 +24,11 @@ RSpec.describe "Money" do
   end
 
   it ".zero has no currency" do
-    expect(Money.zero.currency).to be_a(Money::NullCurrency)
+    expect(Money.new(0, Money::NULL_CURRENCY).currency).to be_a(Money::NullCurrency)
   end
 
   it ".zero is a 0$ value" do
-    expect(Money.zero).to eq(Money.new(0))
-  end
-
-  it ".zero accepts an optional currency" do
-    expect(Money.zero('USD')).to eq(Money.new(0, 'USD'))
+    expect(Money.new(0, Money::NULL_CURRENCY)).to eq(Money.new(0))
   end
 
   it "returns itself with to_money" do
@@ -247,11 +239,6 @@ RSpec.describe "Money" do
     expect((1.50 * Money.new(1.00))).to eq(Money.new(1.50))
   end
 
-  it "is multipliable by a cents amount" do
-    expect((Money.new(1.00) * 0.50)).to eq(Money.new(0.50))
-    expect((0.50 * Money.new(1.00))).to eq(Money.new(0.50))
-  end
-
   it "is multipliable by a rational" do
     expect((Money.new(3.3) * Rational(1, 12))).to eq(Money.new(0.28))
     expect((Rational(1, 12) * Money.new(3.3))).to eq(Money.new(0.28))
@@ -306,10 +293,6 @@ RSpec.describe "Money" do
     expect { Money.new(55.00) / 55 }.to raise_error(RuntimeError)
   end
 
-  it "returns cents in to_liquid" do
-    expect(Money.new(1.00).to_liquid).to eq(100)
-  end
-
   it "returns cents in to_json" do
     expect(Money.new(1.00).to_json).to eq("1.00")
   end
@@ -330,19 +313,7 @@ RSpec.describe "Money" do
     expect(Money.new(1.50).to_f.to_s).to eq("1.5")
   end
 
-  it "is creatable from an integer value in cents" do
-    expect(Money.from_cents(1950)).to eq(Money.new(19.50))
-  end
-
-  it "is creatable from an integer value of 0 in cents" do
-    expect(Money.from_cents(0)).to eq(Money.new)
-  end
-
-  it "is creatable from a float cents amount" do
-    expect(Money.from_cents(1950.5)).to eq(Money.new(19.51))
-  end
-
-  describe '#from_subunits' do 
+  describe '#from_subunits' do
     it "creates Money object from an integer value in cents and currency" do
       expect(Money.from_subunits(1950, 'CAD')).to eq(Money.new(19.50))
     end
@@ -619,10 +590,6 @@ RSpec.describe "Money" do
       expect(money.value).to eq(BigDecimal("1.00"))
     end
 
-    it "returns cents as 100 cents" do
-      expect(money.cents).to eq(100)
-    end
-
     it "returns cents as 100 cents" do
       expect(money.subunits).to eq(100)
     end
@@ -783,20 +750,6 @@ RSpec.describe "Money" do
   end
 
   describe "from_amount quacks like RubyMoney" do
-    it "accepts numeric values" do
-      expect(Money.from_amount(1)).to eq Money.from_cents(1_00)
-      expect(Money.from_amount(1.0)).to eq Money.from_cents(1_00)
-      expect(Money.from_amount(BigDecimal("1"))).to eq Money.from_cents(1_00)
-    end
-
-    it "accepts string values" do
-      expect(Money.from_amount("1")).to eq Money.from_cents(1_00)
-    end
-
-    it "accepts nil values" do
-      expect(Money.from_amount(nil)).to eq Money.from_cents(0)
-    end
-
     it "accepts an optional currency parameter" do
       expect { Money.from_amount(1, "CAD") }.to_not raise_error
     end
@@ -816,10 +769,14 @@ RSpec.describe "Money" do
       money = YAML.dump(Money.new(750, 'usd'))
       expect(money).to eq("--- !ruby/object:Money\nvalue: '750.0'\ncurrency: USD\n")
     end
+
+    it "does not change BigDecimal value to Integer while rounding for currencies without subunits" do
+      money = Money.new(100, 'JPY').to_yaml
+      expect(money).to eq("--- !ruby/object:Money\nvalue: '100.0'\ncurrency: JPY\n")
+    end
   end
 
   describe "YAML deserialization" do
-
     it "accepts values with currencies" do
       money = YAML.load("--- !ruby/object:Money\nvalue: '750.0'\ncurrency: USD\n")
       expect(money).to eq(Money.new(750, 'usd'))
@@ -963,10 +920,4 @@ RSpec.describe "Money" do
       expect(money.currency.iso_code).to eq('EUR')
     end
   end
-
-  describe '#to_yaml' do
-    it 'returns a yaml representation' do
-      expect(Money.new(100, 'JPY').to_yaml).to eq("--- !ruby/object:Money\nvalue: '100.0'\ncurrency: JPY\n")
-    end
-  end
 end

data/spec/rubocop/cop/money/missing_currency_spec.rb

@@ -8,12 +8,16 @@ RSpec.describe RuboCop::Cop::Money::MissingCurrency do
 
   let(:config) { RuboCop::Config.new }
 
-  describe '#on_send' do
-    it 'registers an offense for Money.new without a currency argument' do
+  context 'with default configuration' do
+    it 'registers an offense and corrects for Money.new without a currency argument' do
       expect_offense(<<~RUBY)
         Money.new(1)
         ^^^^^^^^^^^^ Money is missing currency argument
       RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(1, Money::NULL_CURRENCY)
+      RUBY
     end
 
     it 'does not register an offense for Money.new with currency argument' do
@@ -22,18 +26,26 @@ RSpec.describe RuboCop::Cop::Money::MissingCurrency do
       RUBY
     end
 
-    it 'registers an offense for Money.new without a currency argument' do
+    it 'registers an offense and corrects for Money.new without a currency argument' do
       expect_offense(<<~RUBY)
         Money.new
         ^^^^^^^^^ Money is missing currency argument
       RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(0, Money::NULL_CURRENCY)
+      RUBY
     end
 
-    it 'registers an offense for Money.from_amount without a currency argument' do
+    it 'registers an offense and corrects for Money.from_amount without a currency argument' do
       expect_offense(<<~RUBY)
         Money.from_amount(1)
         ^^^^^^^^^^^^^^^^^^^^ Money is missing currency argument
       RUBY
+
+      expect_correction(<<~RUBY)
+        Money.from_amount(1, Money::NULL_CURRENCY)
+      RUBY
     end
 
     it 'does not register an offense for Money.from_amount with currency argument' do
@@ -42,11 +54,15 @@ RSpec.describe RuboCop::Cop::Money::MissingCurrency do
       RUBY
     end
 
-    it 'registers an offense for Money.from_cents without a currency argument' do
+    it 'registers an offense and corrects for Money.from_cents without a currency argument' do
       expect_offense(<<~RUBY)
         Money.from_cents(1)
         ^^^^^^^^^^^^^^^^^^^ Money is missing currency argument
       RUBY
+
+      expect_correction(<<~RUBY)
+        Money.from_cents(1, Money::NULL_CURRENCY)
+      RUBY
     end
 
     it 'does not register an offense for Money.from_cents with currency argument' do
@@ -55,11 +71,26 @@ RSpec.describe RuboCop::Cop::Money::MissingCurrency do
       RUBY
     end
 
-    it 'registers an offense for to_money without a currency argument' do
+    it 'registers an offense and corrects for to_money without a currency argument' do
       expect_offense(<<~RUBY)
         '1'.to_money
         ^^^^^^^^^^^^ to_money is missing currency argument
       RUBY
+
+      expect_correction(<<~RUBY)
+        '1'.to_money(Money::NULL_CURRENCY)
+      RUBY
+    end
+
+    it 'registers an offense and corrects for safe navigation to_money without a currency argument' do
+      expect_offense(<<~RUBY)
+        item&.to_money
+        ^^^^^^^^^^^^^^ to_money is missing currency argument
+      RUBY
+
+      expect_correction(<<~RUBY)
+        item&.to_money(Money::NULL_CURRENCY)
+      RUBY
     end
 
     it 'does not register an offense for to_money with currency argument' do
@@ -68,15 +99,19 @@ RSpec.describe RuboCop::Cop::Money::MissingCurrency do
       RUBY
     end
 
-    it 'registers an offense for to_money block pass form' do
+    it 'registers an offense and corrects for to_money block pass form' do
       expect_offense(<<~RUBY)
         ['1'].map(&:to_money)
         ^^^^^^^^^^^^^^^^^^^^^ to_money is missing currency argument
       RUBY
+
+      expect_correction(<<~RUBY)
+        ['1'].map { |x| x.to_money(Money::NULL_CURRENCY) }
+      RUBY
     end
   end
 
-  describe '#autocorrect' do
+  context 'with ReplacementCurrency configuration' do
     let(:config) do
       RuboCop::Config.new(
         'Money/MissingCurrency' => {

data/spec/rubocop/cop/money/zero_money_spec.rb

@@ -0,0 +1,78 @@
+# frozen_string_literal: true
+
+require_relative '../../../rubocop_helper'
+require 'rubocop/cop/money/zero_money'
+
+RSpec.describe RuboCop::Cop::Money::ZeroMoney do
+  subject(:cop) { described_class.new(config) }
+
+  let(:config) { RuboCop::Config.new }
+
+  context 'with default configuration' do
+    it 'registers an offense and corrects Money.zero without currency' do
+      expect_offense(<<~RUBY)
+        Money.zero
+        ^^^^^^^^^^ Money.zero is removed, use `Money.new(0, Money::NULL_CURRENCY)`.
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(0, Money::NULL_CURRENCY)
+      RUBY
+    end
+
+    it 'registers an offense and corrects Money.zero with currency' do
+      expect_offense(<<~RUBY)
+        Money.zero('CAD')
+        ^^^^^^^^^^^^^^^^^ Money.zero is removed, use `Money.new(0, 'CAD')`.
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(0, 'CAD')
+      RUBY
+    end
+
+    it 'does not register an offense when using Money.new with a currency' do
+      expect_no_offenses(<<~RUBY)
+        Money.new(0, 'CAD')
+      RUBY
+    end
+  end
+
+  context 'with ReplacementCurrency configuration' do
+    let(:config) do
+      RuboCop::Config.new(
+        'Money/ZeroMoney' => {
+          'ReplacementCurrency' => 'CAD'
+        }
+      )
+    end
+
+    it 'registers an offense and corrects Money.zero without currency' do
+      expect_offense(<<~RUBY)
+        Money.zero
+        ^^^^^^^^^^ Money.zero is removed, use `Money.new(0, 'CAD')`.
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(0, 'CAD')
+      RUBY
+    end
+
+    it 'registers an offense and corrects Money.zero with currency' do
+      expect_offense(<<~RUBY)
+        Money.zero('EUR')
+        ^^^^^^^^^^^^^^^^^ Money.zero is removed, use `Money.new(0, 'EUR')`.
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(0, 'EUR')
+      RUBY
+    end
+
+    it 'does not register an offense when using Money.new with a currency' do
+      expect_no_offenses(<<~RUBY)
+        Money.new(0, 'EUR')
+      RUBY
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: shopify-money
 version: !ruby/object:Gem::Version
-  version: 0.14.8
+  version: 0.15.0
 platform: ruby
 authors:
 - Shopify Inc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-06 00:00:00.000000000 Z
+date: 2021-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -135,6 +135,7 @@ files:
 - lib/money_column/railtie.rb
 - lib/rubocop/cop/money.rb
 - lib/rubocop/cop/money/missing_currency.rb
+- lib/rubocop/cop/money/zero_money.rb
 - lib/shopify-money.rb
 - money.gemspec
 - spec/accounting_money_parser_spec.rb
@@ -149,6 +150,7 @@ files:
 - spec/money_spec.rb
 - spec/null_currency_spec.rb
 - spec/rubocop/cop/money/missing_currency_spec.rb
+- spec/rubocop/cop/money/zero_money_spec.rb
 - spec/rubocop_helper.rb
 - spec/schema.rb
 - spec/spec_helper.rb
@@ -189,6 +191,7 @@ test_files:
 - spec/money_spec.rb
 - spec/null_currency_spec.rb
 - spec/rubocop/cop/money/missing_currency_spec.rb
+- spec/rubocop/cop/money/zero_money_spec.rb
 - spec/rubocop_helper.rb
 - spec/schema.rb
 - spec/spec_helper.rb