Stop Thread.new in DRb::TimerIdConv::TimerHolder2#on_gc

and add more stop_pool after stop_service

lib/drb/drb.rb

@@ -1215,6 +1215,7 @@ module DRb
 
     def kill
       @thread.kill
+      @thread.join
     end
 
     def method_missing(msg, *arg, &blk)

test/drb/drbtest.rb

@@ -42,6 +42,7 @@ class DRbService
     server.stop_service
     manager.instance_variable_get(:@queue)&.push(nil)
     manager.instance_variable_get(:@thread)&.join
+    DRb::DRbConn.stop_pool
   end
 end
 

test/drb/test_drb.rb

@@ -329,6 +329,7 @@ class TestDRbAnyToS < Test::Unit::TestCase
     assert_match(/\A#<DRbTests::TestDRbAnyToS::BO:0x[0-9a-f]+>\z/, server.any_to_s(BO.new))
     server.stop_service
     server.thread.join
+    DRb::DRbConn.stop_pool
   end
 end
 
@@ -344,6 +345,7 @@ class TestDRbTCP < Test::Unit::TestCase
     client.close
     server.stop_service
     server.thread.join
+    DRb::DRbConn.stop_pool
   end
 end
 

test/drb/test_drbobject.rb

@@ -12,6 +12,7 @@ module DRbObjectTest
 
   def teardown
     DRb.stop_service
+    DRb::DRbConn.stop_pool
   end
 
   def drb_eq(obj)
@@ -43,7 +44,19 @@ class TestDRbObjectTimerIdConv < Test::Unit::TestCase
   include DRbObjectTest
 
   def setup
-    DRb.start_service(nil, nil, {:idconv => DRb::TimerIdConv.new})
+    @idconv = DRb::TimerIdConv.new
+    DRb.start_service(nil, nil, {:idconv => @idconv})
+  end
+
+  def teardown
+    super
+    # stop DRb::TimerIdConv::TimerHolder2#on_gc
+    @idconv.instance_eval do
+      @holder.instance_eval do
+        @expires = nil
+      end
+    end
+    GC.start
   end
 end