Make on_dir_autoloaded internal

fxn · Sep 17, 2023 · 7f56fa8 · 7f56fa8
1 parent 53c27f9
commit 7f56fa8
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/lib/zeitwerk/kernel.rb b/lib/zeitwerk/kernel.rb
@@ -31,7 +31,7 @@ def require(path)
         loader.__on_file_autoloaded(path) if required
         required
       else
-        loader.on_dir_autoloaded(path)
+        loader.__on_dir_autoloaded(path)
         true
       end
     else

diff --git a/lib/zeitwerk/loader/callbacks.rb b/lib/zeitwerk/loader/callbacks.rb
@@ -37,9 +37,8 @@ module Zeitwerk::Loader::Callbacks
   # Invoked from our decorated Kernel#require when a managed directory is
   # autoloaded.
   #
-  # @private
   # @sig (String) -> void
-  def on_dir_autoloaded(dir)
+  internal def on_dir_autoloaded(dir)
     # Module#autoload does not serialize concurrent requires in CRuby < 3.2, and
     # we handle directories ourselves without going through Kernel#require, so
     # the callback needs to account for concurrency.

diff --git a/test/lib/zeitwerk/test_callbacks.rb b/test/lib/zeitwerk/test_callbacks.rb
@@ -78,7 +78,7 @@ def loader.__on_file_autoloaded(file)
   end
 
   test "autoloading a directory triggers on_dir_autoloaded (Object)" do
-    def loader.on_dir_autoloaded(dir)
+    def loader.__on_dir_autoloaded(dir)
       if dir == File.expand_path("m")
         $on_dir_autoloaded_called = true
       end
@@ -94,7 +94,7 @@ def loader.on_dir_autoloaded(dir)
   end
 
   test "autoloading a directory triggers on_dir_autoloaded (Namespace)" do
-    def loader.on_dir_autoloaded(dir)
+    def loader.__on_dir_autoloaded(dir)
       if dir == File.expand_path("m")
         $on_dir_autoloaded_called = true
       end