backport: Fix a crash in the minitest rewriter (#235)

Original patch: sorbet/sorbet#8446

---

Co-authored-by: Trevor Elliott <[email protected]>

Author: varungandhi-src

ast/Trees.cc

@@ -1343,11 +1343,20 @@ core::NameRef Literal::asString() const {
 
 core::NameRef Literal::asSymbol() const {
     ENFORCE(isSymbol());
+    return asName();
+}
+
+core::NameRef Literal::asName() const {
+    ENFORCE(isName());
     auto t = core::cast_type_nonnull<core::NamedLiteralType>(value);
     core::NameRef res = t.asName();
     return res;
 }
 
+bool Literal::isName() const {
+    return isString() || isSymbol();
+}
+
 bool Literal::isSymbol() const {
     return core::isa_type<core::NamedLiteralType>(value) &&
            core::cast_type_nonnull<core::NamedLiteralType>(value).literalKind ==

ast/Trees.h

@@ -1040,6 +1040,8 @@ EXPRESSION(Literal) {
 
     std::string toStringWithTabs(const core::GlobalState &gs, int tabs = 0) const;
     std::string showRaw(const core::GlobalState &gs, int tabs = 0) const;
+    bool isName() const;
+    core::NameRef asName() const;
     std::string nodeName() const;
     bool isString() const;
     bool isSymbol() const;

rewriter/Minitest.cc

@@ -0,0 +1,20 @@
+# typed: true
+
+class Test
+  extend T::Sig
+  def self.test_each(arg, &blk); end
+  def self.it(name, &blk); end
+  def self.describe(name, &blk); end
+end
+
+class Foo < Test
+  # The unclosed `do` block here should be a recoverable parse error.
+  test_each([[1, 2], [3,4]]) do |(a,b)|
+                           # ^^ error: Hint: this "do" token
+
+  it "it block 1" do
+  end
+
+  it "it block 2" do
+  end
+end # error: unexpected token "end of file"

test/scip/testdata/minitest.rb test/scip/testdata/minitest_1.rbtest/scip/testdata/minitest.rb renamed to test/scip/testdata/minitest_1.rb

@@ -0,0 +1,30 @@
+ # typed: true
+ 
+ class Test
+#      ^^^^ definition [..] Test#
+   extend T::Sig
+#  ^^^^^^ reference [..] Kernel#extend().
+   def self.test_each(arg, &blk); end
+#           ^^^^^^^^^ definition [..] `<Class:Test>`#test_each().
+   def self.it(name, &blk); end
+#           ^^ definition [..] `<Class:Test>`#it().
+   def self.describe(name, &blk); end
+#           ^^^^^^^^ definition [..] `<Class:Test>`#describe().
+ end
+ 
+ class Foo < Test
+#      ^^^ definition [..] Foo#
+#            ^^^^ reference [..] Test#
+   # The unclosed `do` block here should be a recoverable parse error.
+   test_each([[1, 2], [3,4]]) do |(a,b)|
+#  ^^^^^^^^^ reference [..] `<Class:Test>`#test_each().
+                            # ^^ error: Hint: this "do" token
+ 
+   it "it block 1" do
+#     ^^^^^^^^^^^^ definition [..] Foo#`<it 'it block 1'>`().
+   end
+ 
+   it "it block 2" do
+#     ^^^^^^^^^^^^ definition [..] Foo#`<it 'it block 2'>`().
+   end
+ end # error: unexpected token "end of file"

test/scip/testdata/minitest.snapshot.rb test/scip/testdata/minitest_1.snapshot.rbtest/scip/testdata/minitest.snapshot.rb renamed to test/scip/testdata/minitest_1.snapshot.rb

@@ -0,0 +1,24 @@
+# typed: true
+extend T::Sig
+
+class Test
+  extend T::Sig
+
+  def self.test_each(iter, &blk); end
+  def self.it(name, &blk); end
+  def self.describe(name, &blk); end
+
+  test_each([[1,2], [3,4]]) do |(a,b)|
+
+    describe "d" do
+      it "b" do
+        T.reveal_type(a) # error: Revealed type: `Integer`
+      end
+    end
+
+    it "a" do
+      T.reveal_type(a) # error: Revealed type: `Integer`
+    end
+
+  end
+end

test/scip/testdata/minitest_2.rb

@@ -0,0 +1,45 @@
+ # typed: true
+ extend T::Sig
+#^^^^^^ reference [..] Kernel#extend().
+#       ^ reference [..] T#
+#          ^^^ reference [..] T#Sig#
+ 
+ class Test
+#      ^^^^ definition [..] Test#
+   extend T::Sig
+#  ^^^^^^ reference [..] Kernel#extend().
+ 
+   def self.test_each(iter, &blk); end
+#           ^^^^^^^^^ definition [..] `<Class:Test>`#test_each().
+   def self.it(name, &blk); end
+#           ^^ definition [..] `<Class:Test>`#it().
+   def self.describe(name, &blk); end
+#           ^^^^^^^^ definition [..] `<Class:Test>`#describe().
+ 
+   test_each([[1,2], [3,4]]) do |(a,b)|
+#  ^^^^^^^^^ reference [..] `<Class:Test>`#test_each().
+#                                 ^ definition local 1~#2288740619
+#                                 ^ definition local 1~#416088458
+#                                   ^ definition local 2~#2288740619
+#                                   ^ definition local 2~#416088458
+ 
+     describe "d" do
+       it "b" do
+#         ^^^ definition [..] Test#`<it 'b'>`().
+         T.reveal_type(a) # error: Revealed type: `Integer`
+#        ^ reference [..] T#
+#          ^^^^^^^^^^^ reference [..] `<Class:T>`#reveal_type().
+#                      ^ reference local 1~#416088458
+       end
+     end
+ 
+     it "a" do
+#       ^^^ definition [..] Test#`<it 'a'>`().
+       T.reveal_type(a) # error: Revealed type: `Integer`
+#      ^ reference [..] T#
+#        ^^^^^^^^^^^ reference [..] `<Class:T>`#reveal_type().
+#                    ^ reference local 1~#2288740619
+     end
+ 
+   end
+ end