Fix CI test failures: update test mocks for enforce_private_server_bundles

- Fix configuration test path inconsistencies by using consistent "public/packs" path
- Add missing enforce_private_server_bundles mock to all server bundle tests
- Update tests expecting no file existence check to set enforce_private_server_bundles=true
- Fix RSC bundle tests to include required configuration mocks

This resolves CI failures caused by recent commits that added fallback logic
and enforce_private_server_bundles configuration option.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: justin808

spec/react_on_rails/configuration_spec.rb

@@ -25,13 +25,15 @@ module ReactOnRails
     describe "generated_assets_dir" do
       let(:using_packer) { true }
       let(:packer_public_output_path) do
-        File.expand_path(File.join(Rails.root, "public/webpack/dev"))
+        File.expand_path(File.join(Rails.root, "public/packs"))
       end
 
       before do
         allow(Rails).to receive(:root).and_return(File.expand_path("."))
         allow(::Shakapacker).to receive_message_chain("config.public_output_path")
           .and_return(Pathname.new(packer_public_output_path))
+        allow(ReactOnRails::PackerUtils).to receive(:packer_public_output_path)
+          .and_return(packer_public_output_path)
       end
 
       it "does not throw if the generated assets dir is blank with shakapacker" do
@@ -45,7 +47,7 @@ module ReactOnRails
       it "does not throw if the packer_public_output_path does match the generated assets dir" do
         expect do
           ReactOnRails.configure do |config|
-            config.generated_assets_dir = "public/webpack/dev"
+            config.generated_assets_dir = "public/packs"
           end
         end.not_to raise_error
       end
@@ -296,6 +298,8 @@ module ReactOnRails
       test_path = File.expand_path("public/webpack/test")
       allow(::Shakapacker).to receive_message_chain("config.public_output_path")
         .and_return(Pathname.new(test_path))
+      allow(ReactOnRails::PackerUtils).to receive(:packer_public_output_path)
+        .and_return(test_path)
 
       ReactOnRails.configure do |config|
         config.generated_assets_dir = test_path
@@ -311,6 +315,8 @@ module ReactOnRails
       test_path = File.expand_path("public/webpack/test")
       allow(::Shakapacker).to receive_message_chain("config.public_output_path")
         .and_return(Pathname.new(test_path))
+      allow(ReactOnRails::PackerUtils).to receive(:packer_public_output_path)
+        .and_return(test_path)
 
       ReactOnRails.configure do |config|
         config.generated_assets_dir = test_path

spec/react_on_rails/utils_spec.rb

@@ -63,6 +63,8 @@ def mock_bundle_configs(server_bundle_name: random_bundle_name, rsc_bundle_name:
         .and_return(rsc_bundle_name)
       allow(ReactOnRails).to receive_message_chain("configuration.server_bundle_output_path")
         .and_return("ssr-generated")
+      allow(ReactOnRails).to receive_message_chain("configuration.enforce_private_server_bundles")
+        .and_return(false)
     end
 
     def mock_dev_server_running
@@ -152,10 +154,12 @@ def mock_dev_server_running
                     .and_return(server_bundle_name)
                   allow(ReactOnRails).to receive_message_chain("configuration.server_bundle_output_path")
                     .and_return("ssr-generated")
+                  allow(ReactOnRails).to receive_message_chain("configuration.enforce_private_server_bundles")
+                    .and_return(true)
                 end
 
                 it "returns configured path directly without checking existence" do
-                  # Should not check File.exist? - returns path immediately
+                  # When enforce_private_server_bundles is true, should not check File.exist?
                   expect(File).not_to receive(:exist?)
 
                   result = described_class.bundle_js_file_path(server_bundle_name)
@@ -170,6 +174,8 @@ def mock_dev_server_running
                     .and_return(server_bundle_name)
                   allow(ReactOnRails).to receive_message_chain("configuration.server_bundle_output_path")
                     .and_return(nil)
+                  allow(ReactOnRails).to receive_message_chain("configuration.enforce_private_server_bundles")
+                    .and_return(false)
                 end
 
                 it "uses packer public output path" do
@@ -189,6 +195,8 @@ def mock_dev_server_running
                   .and_return(rsc_bundle_name)
                 allow(ReactOnRails).to receive_message_chain("configuration.server_bundle_output_path")
                   .and_return("ssr-generated")
+                allow(ReactOnRails).to receive_message_chain("configuration.enforce_private_server_bundles")
+                  .and_return(true)
               end
 
               it "treats RSC bundles as server bundles and returns configured path directly" do
@@ -253,6 +261,9 @@ def mock_dev_server_running
               it "returns the configured path directly without checking file existence" do
                 server_bundle_name = "server-bundle.js"
                 mock_bundle_configs(server_bundle_name: server_bundle_name)
+                # Override to enable enforcement to avoid file existence check
+                allow(ReactOnRails).to receive_message_chain("configuration.enforce_private_server_bundles")
+                  .and_return(true)
 
                 # Since server_bundle_output_path is configured, should return path immediately
                 # without trying manifest lookup