From 7f5a09f40e3d41dac9c23a5283efe7bbbccd8c7c Mon Sep 17 00:00:00 2001
From: Kevin Newton <kddnewton@gmail.com>
Date: Tue, 5 Mar 2024 08:08:16 -0500
Subject: [PATCH] Implement case/in for ripper translation

---
 lib/prism/translation/ripper.rb | 23 +++++++++++++++++++++--
 test/prism/ripper_test.rb       | 17 -----------------
 2 files changed, 21 insertions(+), 19 deletions(-)

diff --git a/lib/prism/translation/ripper.rb b/lib/prism/translation/ripper.rb
index e3cd3004460..0872ed9f9e6 100644
--- a/lib/prism/translation/ripper.rb
+++ b/lib/prism/translation/ripper.rb
@@ -539,7 +539,14 @@ def visit_case_node(node)
       # case foo; in bar; end
       # ^^^^^^^^^^^^^^^^^^^^^
       def visit_case_match_node(node)
-        raise NoMethodError, __method__
+        predicate = visit(node.predicate)
+        clauses =
+          node.conditions.reverse_each.inject(nil) do |consequent, condition|
+            on_in(*visit(condition), consequent)
+          end
+
+        bounds(node.location)
+        on_case(predicate, clauses)
       end
 
       # class Foo; end
@@ -1124,7 +1131,19 @@ def visit_implicit_rest_node(node)
       # case foo; in bar; end
       # ^^^^^^^^^^^^^^^^^^^^^
       def visit_in_node(node)
-        raise NoMethodError, __method__
+        # This is a special case where we're not going to call on_in directly
+        # because we don't have access to the consequent. Instead, we'll return
+        # the component parts and let the parent node handle it.
+        pattern = visit(node.pattern)
+        statements =
+          if node.statements.nil?
+            bounds(node.location)
+            on_stmts_add(on_stmts_new, on_void_stmt)
+          else
+            visit(node.statements)
+          end
+
+        [pattern, statements]
       end
 
       # foo[bar] += baz
diff --git a/test/prism/ripper_test.rb b/test/prism/ripper_test.rb
index a0b7d3a25f7..4636fa6d58e 100644
--- a/test/prism/ripper_test.rb
+++ b/test/prism/ripper_test.rb
@@ -369,18 +369,7 @@ class RipperFixturesTest < RipperTestCase
       seattlerb/call_stabby_with_braces_block.txt
       seattlerb/call_trailing_comma.txt
       seattlerb/case_in.txt
-      seattlerb/case_in_31.txt
       seattlerb/case_in_37.txt
-      seattlerb/case_in_42.txt
-      seattlerb/case_in_42_2.txt
-      seattlerb/case_in_47.txt
-      seattlerb/case_in_67.txt
-      seattlerb/case_in_86.txt
-      seattlerb/case_in_86_2.txt
-      seattlerb/case_in_array_pat_const.txt
-      seattlerb/case_in_array_pat_const2.txt
-      seattlerb/case_in_array_pat_paren_assign.txt
-      seattlerb/case_in_const.txt
       seattlerb/case_in_else.txt
       seattlerb/case_in_find.txt
       seattlerb/case_in_find_array.txt
@@ -390,9 +379,6 @@ class RipperFixturesTest < RipperTestCase
       seattlerb/case_in_hash_pat_paren_true.txt
       seattlerb/case_in_hash_pat_rest.txt
       seattlerb/case_in_hash_pat_rest_solo.txt
-      seattlerb/case_in_if_unless_post_mod.txt
-      seattlerb/case_in_multiple.txt
-      seattlerb/case_in_or.txt
       seattlerb/class_comments.txt
       seattlerb/defn_arg_forward_args.txt
       seattlerb/defn_args_forward_args.txt
@@ -509,8 +495,6 @@ class RipperFixturesTest < RipperTestCase
       seattlerb/parse_line_to_ary.txt
       seattlerb/parse_opt_call_args_assocs_comma.txt
       seattlerb/parse_opt_call_args_lit_comma.txt
-      seattlerb/parse_pattern_019.txt
-      seattlerb/parse_pattern_044.txt
       seattlerb/parse_pattern_051.txt
       seattlerb/parse_pattern_058.txt
       seattlerb/parse_pattern_058_2.txt
@@ -730,7 +714,6 @@ class RipperFixturesTest < RipperTestCase
       whitequark/parser_bug_645.txt
       whitequark/parser_drops_truncated_parts_of_squiggly_heredoc.txt
       whitequark/parser_slash_slash_n_escaping_in_literals.txt
-      whitequark/pattern_matching__FILE__LINE_literals.txt
       whitequark/pattern_matching_blank_else.txt
       whitequark/pattern_matching_else.txt
       whitequark/pattern_matching_single_line_allowed_omission_of_parentheses.txt