Add comprehensive tests for Phase 3 RSC utility methods

Add tests for the three RSC utility methods moved to Pro gem:

- rsc_support_enabled?: Tests enabled/disabled states
- rsc_bundle_js_file_path: Tests delegation to bundle_js_file_path,
  caching behavior in development vs production
- react_client_manifest_file_path: Tests packer integration with
  dev server running/not running, caching behavior
- react_server_client_manifest_file_path: Tests delegation,
  caching behavior, error handling when manifest is nil

Total: ~210 lines of tests ensuring all RSC utility methods work
correctly in the Pro gem with proper caching and error handling.

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

Co-Authored-By: Claude <noreply@anthropic.com>

Author: AbanoubGhadban

react_on_rails_pro/spec/react_on_rails_pro/utils_spec.rb

@@ -222,6 +222,218 @@ module ReactOnRailsPro
       it { expect(printable_cache_key).to eq("1_2_3_4_5") }
     end
 
+    describe ".rsc_support_enabled?" do
+      context "when RSC support is enabled" do
+        before do
+          allow(ReactOnRailsPro.configuration).to receive(:enable_rsc_support).and_return(true)
+          described_class.instance_variable_set(:@rsc_support_enabled, nil)
+        end
+
+        it "returns true" do
+          expect(described_class.rsc_support_enabled?).to be(true)
+        end
+      end
+
+      context "when RSC support is disabled" do
+        before do
+          allow(ReactOnRailsPro.configuration).to receive(:enable_rsc_support).and_return(false)
+          described_class.instance_variable_set(:@rsc_support_enabled, nil)
+        end
+
+        it "returns false" do
+          expect(described_class.rsc_support_enabled?).to be(false)
+        end
+      end
+    end
+
+    describe ".rsc_bundle_js_file_path" do
+      before do
+        described_class.instance_variable_set(:@rsc_bundle_path, nil)
+        allow(ReactOnRailsPro.configuration).to receive(:rsc_bundle_js_file).and_return("rsc-bundle.js")
+      end
+
+      after do
+        described_class.instance_variable_set(:@rsc_bundle_path, nil)
+      end
+
+      it "calls bundle_js_file_path with the rsc_bundle_js_file name" do
+        allow(ReactOnRails::Utils).to receive(:bundle_js_file_path).with("rsc-bundle.js")
+                                                                   .and_return("/some/path/rsc-bundle.js")
+
+        result = described_class.rsc_bundle_js_file_path
+
+        expect(ReactOnRails::Utils).to have_received(:bundle_js_file_path).with("rsc-bundle.js")
+        expect(result).to eq("/some/path/rsc-bundle.js")
+      end
+
+      it "caches the path when not in development" do
+        allow(Rails.env).to receive(:development?).and_return(false)
+        allow(ReactOnRails::Utils).to receive(:bundle_js_file_path).with("rsc-bundle.js")
+                                                                   .and_return("/some/path/rsc-bundle.js")
+
+        result1 = described_class.rsc_bundle_js_file_path
+        result2 = described_class.rsc_bundle_js_file_path
+
+        expect(ReactOnRails::Utils).to have_received(:bundle_js_file_path).once.with("rsc-bundle.js")
+        expect(result1).to eq("/some/path/rsc-bundle.js")
+        expect(result2).to eq("/some/path/rsc-bundle.js")
+      end
+
+      it "does not cache the path in development" do
+        allow(Rails.env).to receive(:development?).and_return(true)
+        allow(ReactOnRails::Utils).to receive(:bundle_js_file_path).with("rsc-bundle.js")
+                                                                   .and_return("/some/path/rsc-bundle.js")
+
+        result1 = described_class.rsc_bundle_js_file_path
+        result2 = described_class.rsc_bundle_js_file_path
+
+        expect(ReactOnRails::Utils).to have_received(:bundle_js_file_path).twice.with("rsc-bundle.js")
+        expect(result1).to eq("/some/path/rsc-bundle.js")
+        expect(result2).to eq("/some/path/rsc-bundle.js")
+      end
+    end
+
+    describe ".react_client_manifest_file_path" do
+      before do
+        described_class.instance_variable_set(:@react_client_manifest_path, nil)
+        allow(ReactOnRailsPro.configuration).to receive(:react_client_manifest_file)
+          .and_return("react-client-manifest.json")
+      end
+
+      after do
+        described_class.instance_variable_set(:@react_client_manifest_path, nil)
+      end
+
+      context "when using packer" do
+        let(:public_output_path) { "/path/to/public/webpack/dev" }
+
+        before do
+          allow(::Shakapacker).to receive_message_chain("config.public_output_path")
+            .and_return(Pathname.new(public_output_path))
+          allow(::Shakapacker).to receive_message_chain("config.public_path")
+            .and_return(Pathname.new("/path/to/public"))
+        end
+
+        context "when dev server is running" do
+          before do
+            allow(::Shakapacker).to receive(:dev_server).and_return(
+              instance_double(
+                ::Shakapacker::DevServer,
+                running?: true,
+                protocol: "http",
+                host_with_port: "localhost:3035"
+              )
+            )
+          end
+
+          it "returns manifest URL with dev server path" do
+            expected_url = "http://localhost:3035/webpack/dev/react-client-manifest.json"
+            expect(described_class.react_client_manifest_file_path).to eq(expected_url)
+          end
+        end
+
+        context "when dev server is not running" do
+          before do
+            allow(::Shakapacker).to receive_message_chain("dev_server.running?")
+              .and_return(false)
+          end
+
+          it "returns file path to the manifest" do
+            expected_path = File.join(public_output_path, "react-client-manifest.json")
+            expect(described_class.react_client_manifest_file_path).to eq(expected_path)
+          end
+        end
+      end
+
+      it "caches the path when not in development" do
+        allow(Rails.env).to receive(:development?).and_return(false)
+        allow(ReactOnRails::PackerUtils).to receive(:asset_uri_from_packer)
+          .with("react-client-manifest.json")
+          .and_return("/some/path/react-client-manifest.json")
+
+        result1 = described_class.react_client_manifest_file_path
+        result2 = described_class.react_client_manifest_file_path
+
+        expect(ReactOnRails::PackerUtils).to have_received(:asset_uri_from_packer).once
+        expect(result1).to eq("/some/path/react-client-manifest.json")
+        expect(result2).to eq("/some/path/react-client-manifest.json")
+      end
+
+      it "does not cache the path in development" do
+        allow(Rails.env).to receive(:development?).and_return(true)
+        allow(ReactOnRails::PackerUtils).to receive(:asset_uri_from_packer)
+          .with("react-client-manifest.json")
+          .and_return("/some/path/react-client-manifest.json")
+
+        result1 = described_class.react_client_manifest_file_path
+        result2 = described_class.react_client_manifest_file_path
+
+        expect(ReactOnRails::PackerUtils).to have_received(:asset_uri_from_packer).twice
+        expect(result1).to eq("/some/path/react-client-manifest.json")
+        expect(result2).to eq("/some/path/react-client-manifest.json")
+      end
+    end
+
+    describe ".react_server_client_manifest_file_path" do
+      let(:asset_name) { "react-server-client-manifest.json" }
+
+      before do
+        described_class.instance_variable_set(:@react_server_manifest_path, nil)
+        allow(ReactOnRailsPro.configuration).to receive(:react_server_client_manifest_file).and_return(asset_name)
+        allow(Rails.env).to receive(:development?).and_return(false)
+      end
+
+      after do
+        described_class.instance_variable_set(:@react_server_manifest_path, nil)
+      end
+
+      it "calls bundle_js_file_path with the correct asset name and returns its value" do
+        allow(ReactOnRails::Utils).to receive(:bundle_js_file_path).with(asset_name)
+                                                                   .and_return("/some/path/#{asset_name}")
+
+        result = described_class.react_server_client_manifest_file_path
+
+        expect(ReactOnRails::Utils).to have_received(:bundle_js_file_path).with(asset_name)
+        expect(result).to eq("/some/path/#{asset_name}")
+      end
+
+      it "caches the path when not in development" do
+        allow(ReactOnRails::Utils).to receive(:bundle_js_file_path).with(asset_name)
+                                                                   .and_return("/some/path/#{asset_name}")
+
+        result1 = described_class.react_server_client_manifest_file_path
+        result2 = described_class.react_server_client_manifest_file_path
+
+        expect(ReactOnRails::Utils).to have_received(:bundle_js_file_path).once.with(asset_name)
+        expect(result1).to eq("/some/path/#{asset_name}")
+        expect(result2).to eq("/some/path/#{asset_name}")
+      end
+
+      it "does not cache the path in development" do
+        allow(Rails.env).to receive(:development?).and_return(true)
+        allow(ReactOnRails::Utils).to receive(:bundle_js_file_path).with(asset_name)
+                                                                   .and_return("/some/path/#{asset_name}")
+
+        result1 = described_class.react_server_client_manifest_file_path
+        result2 = described_class.react_server_client_manifest_file_path
+
+        expect(ReactOnRails::Utils).to have_received(:bundle_js_file_path).twice.with(asset_name)
+        expect(result1).to eq("/some/path/#{asset_name}")
+        expect(result2).to eq("/some/path/#{asset_name}")
+      end
+
+      context "when manifest file name is nil" do
+        before do
+          allow(ReactOnRailsPro.configuration).to receive(:react_server_client_manifest_file).and_return(nil)
+        end
+
+        it "raises an error" do
+          expect { described_class.react_server_client_manifest_file_path }
+            .to raise_error(ReactOnRailsPro::Error, /react_server_client_manifest_file is nil/)
+        end
+      end
+    end
+
     describe ".pro_attribution_comment" do
       context "when license is valid and not in grace period" do
         before do