Work around Ruby 3.0.0 bug and get tests running without a crash (although they will not all pass).

Author: cfis

test/embed_ruby.cpp

@@ -1,4 +1,5 @@
 #include <rice/rice.hpp>
+#include <ruby/version.h>
 
 void embed_ruby()
 {
@@ -17,6 +18,18 @@ void embed_ruby()
 
     initialized__ = true;
 
-    rb_eval_string("GC.stress = true");
+    // Because Ruby 3 no longer initializes the GC module when embedding, calling GC.stress 
+    // results in a crash.
+    // See https://bugs.ruby-lang.org/issues/17643
+    if (RUBY_API_VERSION_MAJOR == 3 &&
+        RUBY_API_VERSION_MINOR == 0 &&
+        RUBY_API_VERSION_TEENY == 0)
+    {
+      // do nothing
+    }
+    else
+    {
+      rb_eval_string("GC.stress = true");
+    }
   }
 }

test/test_Ownership.cpp

@@ -143,10 +143,10 @@ TESTCASE(TransferPointer)
                           my_class = factory.transfer_pointer
                           my_class.set_flag(i)
                           my_class = nil
-                        end
-                        GC.start)";
+                        end)";
 
   m.instance_eval(code);
+  rb_gc_start();
 
   ASSERT_EQUAL(10, MyClass::constructorCalls);
   ASSERT_EQUAL(0, MyClass::copyConstructorCalls);
@@ -167,10 +167,10 @@ TESTCASE(KeepPointer)
                         10.times do |i|
                           my_class = factory.keep_pointer
                           my_class.set_flag(i)
-                        end
-                        GC.start)";
+                        end)";
 
   m.instance_eval(code);
+  rb_gc_start();
 
   ASSERT_EQUAL(1, MyClass::constructorCalls);
   ASSERT_EQUAL(0, MyClass::copyConstructorCalls);
@@ -191,10 +191,10 @@ TESTCASE(KeepReference)
                         10.times do |i|
                           my_class = factory.keep_reference
                           my_class.set_flag(i)
-                        end
-                        GC.start)";
+                        end)";
 
   m.instance_eval(code);
+  rb_gc_start();
 
   ASSERT_EQUAL(1, MyClass::constructorCalls);
   ASSERT_EQUAL(0, MyClass::copyConstructorCalls);
@@ -215,10 +215,10 @@ TESTCASE(CopyReference)
                         10.times do |i|
                           my_class = factory.copy_reference
                           my_class.set_flag(i)
-                        end
-                        GC.start)";
+                        end)";
 
   m.instance_eval(code);
+  rb_gc_start();
 
   ASSERT_EQUAL(1, MyClass::constructorCalls);
   ASSERT_EQUAL(10, MyClass::copyConstructorCalls);
@@ -239,10 +239,10 @@ TESTCASE(TransferValue)
                         10.times do |i|
                           my_class = factory.value
                           my_class.set_flag(i)
-                        end
-                        GC.start)";
+                        end)";
 
   m.instance_eval(code);
+  rb_gc_start();
 
   ASSERT_EQUAL(10, MyClass::constructorCalls);
   ASSERT_EQUAL(10, MyClass::copyConstructorCalls);
@@ -263,10 +263,10 @@ TESTCASE(MoveValue)
                         10.times do |i|
                           my_class = factory.move_value
                           my_class.set_flag(i)
-                        end
-                        GC.start)";
+                        end)";
 
   m.instance_eval(code);
+  rb_gc_start();
 
   ASSERT_EQUAL(10, MyClass::constructorCalls);
   ASSERT_EQUAL(10, MyClass::copyConstructorCalls);