Remove visit_elsif methods from all visitors

Author: egiurleo

lib/syntax_tree/translation/parser.rb

@@ -238,7 +238,7 @@ def visit_aryptn(node)
           if !node.rest.value.nil?
             children << s(:match_rest, [visit(node.rest)], nil)
           elsif node.posts.empty? &&
-                node.rest.location.start_char == node.rest.location.end_char
+                  node.rest.location.start_char == node.rest.location.end_char
             # Here we have an implicit rest, as in [foo,]. parser has a specific
             # type for these patterns.
             type = :array_pattern_with_tail
@@ -409,7 +409,8 @@ def visit_begin(node)
         if node.bodystmt.empty?
           s(:kwbegin, [], location)
         elsif node.bodystmt.rescue_clause.nil? &&
-              node.bodystmt.ensure_clause.nil? && node.bodystmt.else_clause.nil?
+                node.bodystmt.ensure_clause.nil? &&
+                node.bodystmt.else_clause.nil?
           child = visit(node.bodystmt.statements)
 
           s(:kwbegin, child.type == :begin ? child.children : [child], location)
@@ -993,37 +994,6 @@ def visit_else(node)
         end
       end
 
-      # Visit an Elsif node.
-      def visit_elsif(node)
-        else_token =
-          case node.consequent
-          when Elsif
-            source_range_length(node.consequent.location.start_char, 5)
-          when Else
-            source_range_length(node.consequent.location.start_char, 4)
-          end
-
-        expression =
-          source_range(
-            node.location.start_char,
-            node.statements.location.end_char - 1
-          )
-
-        s(
-          :if,
-          [
-            visit(node.predicate),
-            visit(node.statements),
-            visit(node.consequent)
-          ],
-          source_map_condition(
-            keyword: source_range_length(node.location.start_char, 5),
-            else_token: else_token,
-            expression: expression
-          )
-        )
-      end
-
       # Visit an ENDBlock node.
       def visit_END(node)
         s(
@@ -1381,7 +1351,8 @@ def visit_if(node)
 
             else_token =
               case node.consequent
-              when Elsif
+              when IfNode
+                # The `IfNode` represents an `elsif` clause
                 source_range_length(node.consequent.location.start_char, 5)
               when Else
                 source_range_length(node.consequent.location.start_char, 4)

lib/syntax_tree/visitor.rb

@@ -131,9 +131,6 @@ class Visitor < BasicVisitor
     # Visit an Else node.
     alias visit_else visit_child_nodes
 
-    # Visit an Elsif node.
-    alias visit_elsif visit_child_nodes
-
     # Visit an EmbDoc node.
     alias visit_embdoc visit_child_nodes
 

lib/syntax_tree/visitor/field_visitor.rb

@@ -352,15 +352,6 @@ def visit_else(node)
         end
       end
 
-      def visit_elsif(node)
-        node(node, "elsif") do
-          field("predicate", node.predicate)
-          field("statements", node.statements)
-          field("consequent", node.consequent) if node.consequent
-          comments(node)
-        end
-      end
-
       def visit_embdoc(node)
         node(node, "embdoc") { field("value", node.value) }
       end

lib/syntax_tree/visitor/mutation_visitor.rb

@@ -322,14 +322,6 @@ def visit_else(node)
         )
       end
 
-      # Visit a Elsif node.
-      def visit_elsif(node)
-        node.copy(
-          statements: visit(node.statements),
-          consequent: visit(node.consequent)
-        )
-      end
-
       # Visit a EmbDoc node.
       def visit_embdoc(node)
         node.copy