rubocop-magic_numbers 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e50541096764d84680fd5253e6233a4ed8ef80250a6cb416a0431c3bd194ce66
-  data.tar.gz: 0e85a4951ca5c75ba6d097950e14bc42591eb27167a04f2d723f2093b7ffc066
+  metadata.gz: 8c524463c1bd7ab5a180f5063ee27deceac29fdf0bfe9d682b03ce1f6f8a992d
+  data.tar.gz: ca98482dc543d684dad741da511de7f873ae83dd2ae714fb5115b112855b412c
 SHA512:
-  metadata.gz: 6cbbc1083814d6ec1ebdcb632e6fd34d283fbea606509640382fb79e5fbdaad27a8c90daf8db7588781be54251378a05489aebdbc26c6862f82686e27ebd8f31
-  data.tar.gz: 2d707528dbe8d168ddd09838b6ca5906232a70951921744c76ae6f30fd432b3f7d7c8870ebefce5b8f8becb4e506a977cb837260d84ad4dd89fadf28270e8996
+  metadata.gz: 421a00bdfe1b50f2ac56d4759577dd8f77967a3a24a2111bb901563ec09fee149dc2e64cd45661ea8761ac0c8f9594ecd2e7f82aaddd78f3136651601dee7632
+  data.tar.gz: 177fca60d9af08b2d409bc1e5018f9a9d4f7a3995fc4d04fe2e9efeafa0a9ccd2863ece0b90ce8a9ddcf078f1cb8bd1a251f8ff99ff870377f21d1903ee1a7b5

data/README.md

@@ -123,8 +123,12 @@ MagicNumbers/NoArgument:
     - 1
 MagicNumbers/NoAssignment:
   ForbiddenNumerics: All/Float/Integer # default All
+  PermittedValues: # defaults to []
+    - 0
+    - 1
+    - 3.14
 
-MagicNumbers/Default:
+MagicNumbers/NoDefault:
   ForbiddenNumerics: All/Float/Integer # default All
 
 MagicNumbers/NoReturn:
@@ -132,6 +136,25 @@ MagicNumbers/NoReturn:
   ForbiddenNumerics: All/Float/Integer # default All
 ```
 
+`MagicNumbers/NoAssignment` can be configured with `PermittedValues` when some
+assigned values do not need the same explanation as other numeric values. This
+allows configured values in local variable, instance variable, property, and
+multiple assignments. Unpermitted magic numbers are still detected:
+
+```ruby
+# Allowed when MagicNumbers/NoAssignment permits 0 and 1
+count = 0
+@total = 1
+first, second = 0, 1
+
+# Still detected as an offense
+retry_limit = 3
+first, second = 2
+first, second = 0, 2
+first, second = 2, 0
+first, second = 2, 3
+```
+
 For more information on configuring `rubocop`, please refer to the [official documentation](https://docs.rubocop.org/rubocop/configuration.html).
 
 ## Rails usage

data/lib/rubocop/cop/magic_numbers/no_assignment.rb

@@ -17,14 +17,17 @@ module RuboCop
           (send
             ({send self} ...)
             $_
-            (%<illegal_scalar_pattern>s _)
+            $(%<illegal_scalar_pattern>s _)
           )
         PATTERN
 
         MAGIC_NUMBER_MULTI_ASSIGN_PATTERN = <<-PATTERN
           (masgn
             (mlhs ({lvasgn ivasgn send} ...)+)
-            (array <(%<illegal_scalar_pattern>s _) ...>)
+            {
+              $(%<illegal_scalar_pattern>s _)
+              $(array <(%<illegal_scalar_pattern>s _) ...>)
+            }
           )
         PATTERN
         LOCAL_VARIABLE_ASSIGN_MSG = 'Do not use magic number local variables'
@@ -32,7 +35,8 @@ module RuboCop
         MULTIPLE_ASSIGN_MSG = 'Do not use magic numbers in multiple assignments'
         PROPERTY_MSG = 'Do not use magic numbers to set properties'
         DEFAULT_CONFIG = {
-          'AllowedAssignments' => %w[class_variables global_variables]
+          'AllowedAssignments' => %w[class_variables global_variables],
+          'PermittedValues' => []
         }.freeze
 
         def cop_config
@@ -76,7 +80,7 @@ module RuboCop
         private
 
         def illegal_scalar_argument_to_setter?(node)
-          method = node_matches_pattern?(
+          method, value = node_matches_pattern?(
             node: node,
             pattern: format(
               MAGIC_NUMBER_ARGUMENT_TO_SETTER_PATTERN,
@@ -84,17 +88,19 @@ module RuboCop
             )
           )
 
-          method&.end_with?('=')
+          method&.end_with?('=') && illegal_scalar_expression?(value)
         end
 
         def illegal_multi_assign_right_hand_side?(node)
-          node_matches_pattern?(
+          multiple_assignment_value = node_matches_pattern?(
             node: node,
             pattern: format(
               MAGIC_NUMBER_MULTI_ASSIGN_PATTERN,
               illegal_scalar_pattern: illegal_scalar_pattern
             )
           )
+
+          illegal_scalar_expressions(multiple_assignment_value).any?
         end
 
         def illegal_scalar_value?(node)
@@ -103,7 +109,29 @@ module RuboCop
           # multiassignment nodes contain individual assignments in their AST
           # representations, but they aren't aware of their values, so we need to
           # allow for expressionless assignments
-          forbidden_numerics.include?(node.expression&.type)
+          illegal_scalar_expression?(node.expression)
+        end
+
+        def illegal_scalar_expression?(expression)
+          return false unless expression
+          return false unless forbidden_numerics.include?(expression.type)
+
+          !permitted_value?(expression.value)
+        end
+
+        def illegal_scalar_expressions(expression)
+          return [] unless expression
+
+          expressions = expression.array_type? ? expression.children : [expression]
+          expressions.select { |child_expression| illegal_scalar_expression?(child_expression) }
+        end
+
+        def permitted_value?(value)
+          permitted_values.include?(value)
+        end
+
+        def permitted_values
+          Array(cop_config['PermittedValues'])
         end
       end
     end

data/lib/rubocop/magic_numbers/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module MagicNumbers
-    VERSION = '0.5.0'
+    VERSION = '0.6.0'
   end
 end

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-magic_numbers
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Gavin Morrice
 - Fell Sunderland
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-03-05 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -62,7 +61,6 @@ licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -77,8 +75,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.6
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: rubocop/magic_numbers implements a rubocop cop for detecting the use of bare
   numbers when linting