Ask if node is assignment, instead of creating new list

rubocop · Jan 18, 2019 · f79c30f · f79c30f
1 parent 5371b80
commit f79c30f
Showing 1 changed file with 1 addition and 13 deletions.
diff --git a/lib/rubocop/cop/layout/rescue_ensure_alignment.rb b/lib/rubocop/cop/layout/rescue_ensure_alignment.rb
@@ -30,17 +30,6 @@ class RescueEnsureAlignment < Cop
         ANCESTOR_TYPES = %i[kwbegin def defs class module].freeze
         RUBY_2_5_ANCESTOR_TYPES = (ANCESTOR_TYPES + %i[block]).freeze
         ANCESTOR_TYPES_WITH_ACCESS_MODIFIERS = %i[def defs].freeze
-        ASSIGNMENT_TYPES = %i[
-          lvasgn
-          ivasgn
-          cvasgn
-          gvasgn
-          casgn
-          masgn
-          op_asgn
-          and_asgn
-          or_asgn
-        ].freeze
 
         def on_resbody(node)
           check(node) unless modifier?(node)
@@ -151,8 +140,7 @@ def ancestor_node(node)
         def assignment_node(node)
           assignment_node = node.ancestors.first
           return nil unless
-            assignment_node &&
-            ASSIGNMENT_TYPES.include?(assignment_node.type)
+            assignment_node && assignment_node.assignment?
 
           assignment_node
         end