Add tests for $LOAD_PATH.dup and Ractor.make_shareable

Author: byroot

CHANGELOG.md

@@ -1,5 +1,7 @@
 # Unreleased
 
+* Ensure `$LOAD_PATH.dup` is Ractor shareable to fix an conflit with `did_you_mean`.
+* Allow to ignore direcotries using absolute paths.
 * Support YAML and JSON CompileCache on TruffleRuby.
 * Support LoadPathCache on TruffleRuby.
 

lib/bootsnap/load_path_cache/change_observer.rb

@@ -56,16 +56,10 @@ def uniq!(*args)
         end
 
         def dup
-          new_obj = super
-          new_obj.remove_instance_variable(:@lpc_observer)
-          new_obj
+          [] + self
         end
 
-        def clone
-          new_obj = super
-          ChangeObserver.unregister(new_obj)
-          new_obj
-        end
+        alias_method :clone, :dup
       end
 
       def self.register(arr, observer)

test/load_path_cache/change_observer_test.rb

@@ -10,6 +10,7 @@ class ChangeObserverTest < Minitest::Test
       def setup
         super
         @observer = Object.new
+        @observer.instance_variable_set(:@mutex, Mutex.new)
         @arr = []
         ChangeObserver.register(@arr, @observer)
       end
@@ -78,6 +79,20 @@ def test_register_twice_observes_once
         assert_equal(%w(a), @arr)
       end
 
+      def test_dup_returns_ractor_shareable_instance
+        return unless defined?(Ractor)
+
+        ChangeObserver.register(@arr, @observer)
+        Ractor.make_shareable(@arr.dup.freeze)
+      end
+
+      def test_clone_returns_ractor_shareable_instance
+        return unless defined?(Ractor)
+
+        ChangeObserver.register(@arr, @observer)
+        Ractor.make_shareable(@arr.clone.freeze)
+      end
+
       def test_uniq_without_block
         @observer.expects(:reinitialize).never
         @arr.uniq!