shopify-money 1.1.2 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68cd70a11908e256c3743d9cd5c66ad2d3fb93eac02fd7973a5e8c20c6ca3e6b
-  data.tar.gz: c5619ce2dc46b6271ae96c326c57b8d4fc7dfef9f3696db1f140c83b5df0f5bf
+  metadata.gz: de174294a88067d61cfcf9321bc62fc3de2b4c265e089d07ff8bedca30ce782a
+  data.tar.gz: 9cb65f619cf3ceb3054437c3ede664d9fa949859af7858e1f280d82f983cdace
 SHA512:
-  metadata.gz: 116fc4ca873f7373653509bc104d8343160877c3085ac2446dacbb918ecf2e823a915b9f2bb97ab48d7a39c4f2515518fa64ffece7a2c501b4a3958ea5455457
-  data.tar.gz: 3b702aa16acd05f2100589ac319931c268999e616ce28e741c921539000c39b48521807c6a81d1a0aaaf70c2f436a02a4f236109104db00bd4cdb5b9c8e9c01e
+  metadata.gz: cfd7348a2506689d46b70eebb613c4ebdb3ef33ad31ae361d130c4348691f90049a66a50920f96c9f39f532bfe3fc0901c939a6d2e0be6d90d6d045165209f37
+  data.tar.gz: 729391b98fbbfc671726cedd35d6ee13afed28f6b88b98d825eb6cae23cc968de72437fdacf51bf5f68dbaf3aabf4f84d296709764031ce83128a4010012a1b7

data/.github/workflows/tests.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        ruby: ['2.6', '2.7', '3.0', '3.1']
+        ruby: ['2.6', '2.7', '3.0', '3.1', '3.2']
 
     name: Ruby ${{ matrix.ruby }}
     steps:

data/README.md

@@ -156,12 +156,17 @@ require:
 
 Money/MissingCurrency:
   Enabled: true
-```
+  # If your application is currently handling only one currency,
+  # it can autocorrect this by specifying a default currency.
+  ReplacementCurrency: CAD
 
-If your application is currently handling only one currency, it can autocorrect this by specifying a currency under the `Enabled` line:
+Money/ZeroMoney:
+  Enabled: true
+  # Same here:
+  # ReplacementCurrency: CAD
 
-```yaml
-ReplacementCurrency: 'CAD'
+Money/UnsafeToMoney:
+  Enabled: true
 ```
 
 ## Contributing to money

data/dev.yml

@@ -3,7 +3,7 @@
 ---
 name: money
 up:
-- ruby: 2.6.6
+- ruby: 3.2.0
 - bundler
 commands:
   test: bundle exec rspec

data/lib/money/version.rb

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

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

@@ -0,0 +1,35 @@
+
+module RuboCop
+  module Cop
+    module Money
+      # Prevents the use of `to_money` because it has inconsistent behaviour.
+      # Use `Money.new` instead.
+      #
+      # @example
+      #  # bad
+      #  "2.000".to_money("USD")     #<Money value:2000.00 currency:USD>
+      #
+      #  # good
+      #  Money.new("2.000", "USD")   #<Money value:2.00 currency:USD>
+      class UnsafeToMoney < Cop
+        MSG = '`to_money` has inconsistent behaviour. Use `Money.new` instead.'.freeze
+
+        def on_send(node)
+          return unless node.method?(:to_money)
+          return if node.receiver.is_a?(AST::NumericNode)
+
+          add_offense(node, location: :selector)
+        end
+
+        def autocorrect(node)
+          lambda do |corrector|
+            receiver = node.receiver.source
+            args = node.arguments.map(&:source)
+            args.prepend(receiver)
+            corrector.replace(node.loc.expression, "Money.new(#{args.join(', ')})")
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/money.rb

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

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

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require_relative '../../../rubocop_helper'
+require 'rubocop/cop/money/unsafe_to_money'
+
+RSpec.describe RuboCop::Cop::Money::UnsafeToMoney do
+  subject(:cop) { described_class.new(config) }
+
+  let(:config) { RuboCop::Config.new }
+
+  context 'with default configuration' do
+    it 'does not register an offense for literal integer' do
+      expect_no_offenses(<<~RUBY)
+        1.to_money
+      RUBY
+    end
+
+    it 'does not register an offense for literal float' do
+      expect_no_offenses(<<~RUBY)
+        1.000.to_money
+      RUBY
+    end
+
+    it 'registers an offense and corrects for Money.new without a currency argument' do
+      expect_offense(<<~RUBY)
+        '2.000'.to_money
+                ^^^^^^^^ #{described_class::MSG}
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new('2.000')
+      RUBY
+    end
+
+    it 'registers an offense and corrects for Money.new with a currency argument' do
+      expect_offense(<<~RUBY)
+        '2.000'.to_money('USD')
+                ^^^^^^^^ #{described_class::MSG}
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new('2.000', 'USD')
+      RUBY
+    end
+
+    it 'registers an offense and corrects for Money.new with a complex receiver' do
+      expect_offense(<<~RUBY)
+        obj.money.to_money('USD')
+                  ^^^^^^^^ #{described_class::MSG}
+      RUBY
+
+      expect_correction(<<~RUBY)
+        Money.new(obj.money, 'USD')
+      RUBY
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: shopify-money
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.2.1
 platform: ruby
 authors:
 - Shopify Inc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-01-24 00:00:00.000000000 Z
+date: 2023-06-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -140,6 +140,7 @@ files:
 - lib/money_column/railtie.rb
 - lib/rubocop/cop/money.rb
 - lib/rubocop/cop/money/missing_currency.rb
+- lib/rubocop/cop/money/unsafe_to_money.rb
 - lib/rubocop/cop/money/zero_money.rb
 - lib/shopify-money.rb
 - money.gemspec
@@ -159,6 +160,7 @@ files:
 - spec/rails/job_argument_serializer_spec.rb
 - spec/rails_spec_helper.rb
 - spec/rubocop/cop/money/missing_currency_spec.rb
+- spec/rubocop/cop/money/unsafe_to_money_spec.rb
 - spec/rubocop/cop/money/zero_money_spec.rb
 - spec/rubocop_helper.rb
 - spec/schema.rb
@@ -183,7 +185,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
+rubygems_version: 3.4.14
 signing_key: 
 specification_version: 4
 summary: Shopify's money gem
@@ -204,6 +206,7 @@ test_files:
 - spec/rails/job_argument_serializer_spec.rb
 - spec/rails_spec_helper.rb
 - spec/rubocop/cop/money/missing_currency_spec.rb
+- spec/rubocop/cop/money/unsafe_to_money_spec.rb
 - spec/rubocop/cop/money/zero_money_spec.rb
 - spec/rubocop_helper.rb
 - spec/schema.rb