Fix test failure with nil Rails constant

justin808 · claude · justin808 · commit fc3017615865 · 2025-10-09T13:30:11.000-10:00
Use safe navigation operator (&.) when checking Rails.root to handle cases where Rails constant exists but is set to nil (e.g., via stub_const). This prevents NoMethodError when defined?(Rails) is true but Rails is nil. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/lib/shakapacker/instance.rb b/lib/shakapacker/instance.rb
@@ -7,10 +7,10 @@ class Shakapacker::Instance
 
   def initialize(root_path: nil, config_path: nil)
     # Use Rails.root if Rails is defined and no root_path is provided
-    @root_path = root_path || (defined?(Rails) && Rails.root) || Pathname.new(Dir.pwd)
+    @root_path = root_path || (defined?(Rails) && Rails&.root) || Pathname.new(Dir.pwd)
 
     # Use Rails.root.join for config_path if Rails is defined and no config_path is provided
-    default_config_path = if defined?(Rails) && Rails.root
+    default_config_path = if defined?(Rails) && Rails&.root
       Rails.root.join("config/shakapacker.yml")
                           else
                             Pathname.new(File.join(Dir.pwd, "config/shakapacker.yml"))
