react_on_rails 16.2.0.beta.4 → 16.2.0.beta.8

data/react_on_rails_pro/spec/dummy/config/initializers/react_on_rails.rb

@@ -1,9 +1,11 @@
 # frozen_string_literal: true
 
-# For documentation of parameters see: docs/basics/configuration.md
+# ⚠️ TEST CONFIGURATION - Do not copy directly for production apps
+# This is the Pro dummy app configuration used for testing React on Rails Pro features.
+# See docs/api-reference/configuration.md for production configuration guidance.
+
+# Advanced: Custom rendering extension to add values to railsContext
 module RenderingExtension
-  # Return a Hash that contains custom values from the view context that will get passed to
-  # all calls to react_component and redux_store for rendering
   def self.custom_context(view_context)
     if view_context.controller.is_a?(ActionMailer::Base)
       {}
@@ -15,6 +17,7 @@ module RenderingExtension
   end
 end
 
+# Advanced: Custom props extension for client-side hydration
 module RenderingPropsExtension
   def self.adjust_props_for_client_side_hydration(_component_name, props)
     if props.instance_of?(Hash)
@@ -26,19 +29,32 @@ module RenderingPropsExtension
 end
 
 ReactOnRails.configure do |config|
+  ################################################################################
+  # Essential Configuration
+  ################################################################################
   config.server_bundle_js_file = "server-bundle.js"
+  config.components_subdirectory = "ror-auto-load-components"
+  config.auto_load_bundle = true
+
+  ################################################################################
+  # Pro Feature Testing: Server Bundle Security
+  ################################################################################
+  # Testing private server bundle enforcement (recommended for production)
+  config.enforce_private_server_bundles = true
+  config.server_bundle_output_path = "ssr-generated"
+
+  ################################################################################
+  # Test-specific Advanced Configuration
+  ################################################################################
+  # Testing with fixed DOM IDs for easier test assertions
   config.random_dom_id = false # default is true
 
-  # Next 2 lines are commented out because we've set test.compile to true
-  # config.build_test_command = "yarn run build:test"
-  # config.webpack_generated_files = %w[server-bundle.js manifest.json]
+  # Testing advanced rendering customization
   config.rendering_extension = RenderingExtension
-
   config.rendering_props_extension = RenderingPropsExtension
 
-  config.auto_load_bundle = true
-  config.components_subdirectory = "ror-auto-load-components"
-
-  config.enforce_private_server_bundles = true
-  config.server_bundle_output_path = "ssr-generated"
+  # NOTE: build_test_command and webpack_generated_files are commented out
+  # because we've set test.compile to true in shakapacker.yml
+  # config.build_test_command = "yarn run build:test"
+  # config.webpack_generated_files = %w[server-bundle.js manifest.json]
 end

data/react_on_rails_pro/spec/dummy/config.ru

@@ -2,6 +2,6 @@
 
 # This file is used by Rack-based servers to start the application.
 
-require ::File.expand_path("config/environment", __dir__)
+require File.expand_path("config/environment", __dir__)
 
 run Rails.application

data/react_on_rails_pro/spec/dummy/package.json

@@ -17,7 +17,7 @@
     "@loadable/server": "^5.16.2",
     "@loadable/webpack-plugin": "^5.15.2",
     "@sentry/node": "^7.120.0",
-    "@shakacode-tools/react-on-rails-pro-node-renderer": "link:.yalc/@shakacode-tools/react-on-rails-pro-node-renderer",
+    "react-on-rails-pro-node-renderer": "link:.yalc/react-on-rails-pro-node-renderer",
     "@shakacode/use-ssr-computation.macro": "^1.2.4",
     "@shakacode/use-ssr-computation.runtime": "^2.0.0",
     "@webpack-cli/serve": "^1.6.0",
@@ -98,7 +98,7 @@
     "test": "yarn run build:test && yarn run lint && rspec",
     "lint": "cd ../.. && nps lint",
     "e2e-test": "playwright test",
-    "preinstall": "yarn run link-source && yalc add --link react-on-rails-pro && cd .yalc/react-on-rails-pro && yalc add --link react-on-rails && cd ../.. && yalc add --link @shakacode-tools/react-on-rails-pro-node-renderer",
+    "preinstall": "yarn run link-source && yalc add --link react-on-rails-pro && cd .yalc/react-on-rails-pro && yalc add --link react-on-rails && cd ../.. && yalc add --link react-on-rails-pro-node-renderer",
     "link-source": "cd ../../.. && yarn && yarn run yalc:publish && cd react_on_rails_pro && yarn && yalc publish",
     "postinstall": "test -f post-yarn-install.local && ./post-yarn-install.local || true",
     "build:test": "rm -rf public/webpack/test && rm -rf ssr-generated && RAILS_ENV=test NODE_ENV=test bin/shakapacker",

data/react_on_rails_pro/spec/dummy/spec/helpers/react_on_rails_pro_helper_spec.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "async"
+require "async/queue"
 require "rails_helper"
 require "support/script_tag_utils"
 
@@ -327,10 +329,10 @@ describe ReactOnRailsProHelper do
       HTML
     end
 
+    # mock_chunks can be an Async::Queue or an Array
     def mock_request_and_response(mock_chunks = chunks, count: 1)
       # Reset connection instance variables to ensure clean state for tests
       ReactOnRailsPro::Request.instance_variable_set(:@connection, nil)
-      ReactOnRailsPro::Request.instance_variable_set(:@connection_without_retries, nil)
       original_httpx_plugin = HTTPX.method(:plugin)
       allow(HTTPX).to receive(:plugin) do |*args|
         original_httpx_plugin.call(:mock_stream).plugin(*args)
@@ -340,9 +342,19 @@ describe ReactOnRailsProHelper do
       chunks_read.clear
       mock_streaming_response(%r{http://localhost:3800/bundles/[a-f0-9]{32}-test/render/[a-f0-9]{32}}, 200,
                               count: count) do |yielder|
-        mock_chunks.each do |chunk|
-          chunks_read << chunk
-          yielder.call("#{chunk.to_json}\n")
+        if mock_chunks.is_a?(Async::Queue)
+          loop do
+            chunk = mock_chunks.dequeue
+            break if chunk.nil?
+
+            chunks_read << chunk
+            yielder.call("#{chunk.to_json}\n")
+          end
+        else
+          mock_chunks.each do |chunk|
+            chunks_read << chunk
+            yielder.call("#{chunk.to_json}\n")
+          end
         end
       end
     end
@@ -429,18 +441,35 @@ describe ReactOnRailsProHelper do
 
         allow(mocked_stream).to receive(:write) do |chunk|
           written_chunks << chunk
-          # Ensures that any chunk received is written immediately to the stream
-          expect(written_chunks.count).to eq(chunks_read.count) # rubocop:disable RSpec/ExpectInHook
         end
         allow(mocked_stream).to receive(:close)
         mocked_response = instance_double(ActionDispatch::Response)
         allow(mocked_response).to receive(:stream).and_return(mocked_stream)
         allow(self).to receive(:response).and_return(mocked_response)
-        mock_request_and_response
+      end
+
+      def execute_stream_view_containing_react_components
+        queue = Async::Queue.new
+        mock_request_and_response(queue)
+
+        Sync do |parent|
+          parent.async { stream_view_containing_react_components(template: template_path) }
+
+          chunks_to_write = chunks.dup
+          while (chunk = chunks_to_write.shift)
+            queue.enqueue(chunk)
+            sleep 0.05
+
+            # Ensures that any chunk received is written immediately to the stream
+            expect(written_chunks.count).to eq(chunks_read.count)
+          end
+          queue.close
+          sleep 0.05
+        end
       end
 
       it "writes the chunk to stream as soon as it is received" do
-        stream_view_containing_react_components(template: template_path)
+        execute_stream_view_containing_react_components
         expect(self).to have_received(:render_to_string).once.with(template: template_path)
         expect(chunks_read.count).to eq(chunks.count)
         expect(written_chunks.count).to eq(chunks.count)
@@ -449,7 +478,7 @@ describe ReactOnRailsProHelper do
       end
 
       it "prepends the rails context to the first chunk only" do
-        stream_view_containing_react_components(template: template_path)
+        execute_stream_view_containing_react_components
         initial_result = written_chunks.first
         expect(initial_result).to script_tag_be_included(rails_context_tag)
 
@@ -465,7 +494,7 @@ describe ReactOnRailsProHelper do
       end
 
       it "prepends the component specification tag to the first chunk only" do
-        stream_view_containing_react_components(template: template_path)
+        execute_stream_view_containing_react_components
         initial_result = written_chunks.first
         expect(initial_result).to script_tag_be_included(react_component_specification_tag)
 
@@ -476,7 +505,7 @@ describe ReactOnRailsProHelper do
       end
 
       it "renders the rails view content in the first chunk" do
-        stream_view_containing_react_components(template: template_path)
+        execute_stream_view_containing_react_components
         initial_result = written_chunks.first
         expect(initial_result).to include("<h1>Header Rendered In View</h1>")
         written_chunks[1..].each do |chunk|

data/react_on_rails_pro/spec/dummy/spec/rails_helper.rb

@@ -53,7 +53,7 @@ RSpec.configure do |config|
 
   # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
   # For React on Rails Pro, using loadable-stats.json
-  config.fixture_paths = ["#{::Rails.root}/spec/fixtures"]
+  config.fixture_paths = ["#{Rails.root}/spec/fixtures"]
 
   # If you're not using ActiveRecord, or you'd prefer not to run each of your
   # examples within a transaction, remove the following line or assign false

data/react_on_rails_pro/spec/dummy/spec/requests/renderer_console_logging_spec.rb

@@ -15,11 +15,11 @@ describe "Console logging from server" do
       html_nodes = Nokogiri::HTML(response.body)
       expected = <<~JS
         console.log.apply(console, ["[SERVER] RENDERED ReduxSharedStoreApp to dom node with id: ReduxSharedStoreApp-react-component-0"]);
-        console.log.apply(console, ["[SERVER] This is a script:\\\"</div>\\\"(/script> <script>alert('WTF1')(/script>"]);
-        console.log.apply(console, ["[SERVER] Script2:\\\"</div>\\\"(/script xx> <script>alert('WTF2')(/script xx>"]);
-        console.log.apply(console, ["[SERVER] Script3:\\\"</div>\\\"(/script xx> <script>alert('WTF3')(/script xx>"]);
-        console.log.apply(console, ["[SERVER] Script4\\\"</div>\\\"(/script <script>alert('WTF4')(/script>"]);
-        console.log.apply(console, ["[SERVER] Script5:\\\"</div>\\\"(/script> <script>alert('WTF5')(/script>"]);
+        console.log.apply(console, ["[SERVER] This is a script:\\"</div>\\"(/script> <script>alert('WTF1')(/script>"]);
+        console.log.apply(console, ["[SERVER] Script2:\\"</div>\\"(/script xx> <script>alert('WTF2')(/script xx>"]);
+        console.log.apply(console, ["[SERVER] Script3:\\"</div>\\"(/script xx> <script>alert('WTF3')(/script xx>"]);
+        console.log.apply(console, ["[SERVER] Script4\\"</div>\\"(/script <script>alert('WTF4')(/script>"]);
+        console.log.apply(console, ["[SERVER] Script5:\\"</div>\\"(/script> <script>alert('WTF5')(/script>"]);
         console.log.apply(console, ["[SERVER] railsContext.serverSide is ","true"]);
       JS
 

data/react_on_rails_pro/spec/dummy/spec/system/integration_spec.rb

@@ -11,7 +11,7 @@ def change_text_expect_dom_selector(dom_selector, expect_no_change: false)
     find("input").set new_text
     within("h3") do
       if expect_no_change
-        expect(subject).not_to have_content new_text
+        expect(subject).to have_no_content new_text
       else
         expect(subject).to have_content new_text
       end
@@ -110,7 +110,7 @@ describe "Turbolinks across pages", :js do
   it "changes name in message according to input" do
     visit "/client_side_hello_world"
     change_text_expect_dom_selector("#HelloWorld-react-component-0")
-    click_link "Hello World Component Server Rendered, with extra options"
+    click_on "Hello World Component Server Rendered, with extra options"
     change_text_expect_dom_selector("#my-hello-world-id")
   end
 end
@@ -174,19 +174,19 @@ describe "React Router", :js do
 
   before do
     visit "/"
-    click_link "React Router"
+    click_on "React Router"
   end
 
   context "when rendering /react_router" do
     it { is_expected.to have_text("Woohoo, we can use react-router here!") }
 
     it "clicking links correctly renders other pages" do
-      click_link "Router First Page"
+      click_on "Router First Page"
       expect(page).to have_current_path("/react_router/first_page")
       first_page_header_text = page.find(:css, "h2#first-page").text
       expect(first_page_header_text).to eq("React Router First Page")
 
-      click_link "Router Second Page"
+      click_on "Router Second Page"
       expect(page).to have_current_path("/react_router/second_page")
       second_page_header_text = page.find(:css, "h2#second-page").text
       expect(second_page_header_text).to eq("React Router Second Page")
@@ -239,12 +239,12 @@ describe "async render function returns component", :js do
   end
 end
 
-describe "Manual client hydration", :js, type: :system do
+describe "Manual client hydration", :js do
   before { visit "/xhr_refresh" }
 
   it "HelloWorldRehydratable onChange should trigger" do
     within("form") do
-      click_button "refresh"
+      click_on "refresh"
     end
     within("#HelloWorldRehydratable-react-component-1") do
       find("input").set "Should update"
@@ -396,20 +396,20 @@ shared_examples "streamed component tests" do |path, selector|
 
   it "hydrates the component" do
     visit path
-    expect(page.html).to match(/client-bundle[^\"]*.js/)
+    expect(page.html).to match(/client-bundle[^"]*.js/)
     change_text_expect_dom_selector(selector)
   end
 
   it "renders the page completely on server and displays content on client even without JavaScript" do
     # Don't add client-bundle.js to the page to ensure that the app is not hydrated
     visit "#{path}?skip_js_packs=true"
-    expect(page.html).not_to match(/client-bundle[^\"]*.js/)
+    expect(page.html).not_to match(/client-bundle[^"]*.js/)
     # Ensure that the component state is not updated
     change_text_expect_dom_selector(selector, expect_no_change: true)
 
-    expect(page).not_to have_text "Loading branch1"
-    expect(page).not_to have_text "Loading branch2"
-    expect(page).not_to have_text(/Loading branch1 at level \d+/)
+    expect(page).to have_no_text "Loading branch1"
+    expect(page).to have_no_text "Loading branch2"
+    expect(page).to have_no_text(/Loading branch1 at level \d+/)
     expect(page).to have_text(/branch1 \(level \d+\)/, count: 5)
   end
 
@@ -417,8 +417,8 @@ shared_examples "streamed component tests" do |path, selector|
     # visit waits for the page to load, so we ensure that the page is loaded before checking the hydration status
     visit "#{path}?skip_js_packs=true"
     expect(page).to have_text "HydrationStatus: Streaming server render"
-    expect(page).not_to have_text "HydrationStatus: Hydrated"
-    expect(page).not_to have_text "HydrationStatus: Page loaded"
+    expect(page).to have_no_text "HydrationStatus: Hydrated"
+    expect(page).to have_no_text "HydrationStatus: Page loaded"
   end
 end
 
@@ -430,4 +430,10 @@ end
 describe "React Router Sixth Page", :js do
   it_behaves_like "streamed component tests", "/server_router/streaming-server-component",
                   "#ServerComponentRouter-react-component-0"
+
+  # Skip the test that fails without JavaScript - being addressed in another PR
+  it "renders the page completely on server and displays content on client even without JavaScript", # rubocop:disable RSpec/NoExpectationExample
+     skip: "Being addressed in another PR" do
+    # This test is overridden to skip it
+  end
 end

data/react_on_rails_pro/spec/dummy/spec/system/renderer_integration_spec.rb

@@ -9,14 +9,14 @@ describe "Shared Redux store example", :server_rendering do
 
   context "with enabled JS", :js do
     it "Has correct heading and text inside the text input" do
-      expect(page).to have_selector("h3", text: /\ARedux Hello, Mr. Server Side Rendering!\z/)
-      expect(page).to have_selector("input[type='text'][value='Mr. Server Side Rendering']")
+      expect(page).to have_css("h3", text: /\ARedux Hello, Mr. Server Side Rendering!\z/)
+      expect(page).to have_css("input[type='text'][value='Mr. Server Side Rendering']")
     end
 
     it "updates header in reaction to text input changes" do
       new_value = "new value"
       all("input[type='text']")[0].set(new_value)
-      expect(page).to have_selector("h3", text: /\ARedux Hello, #{new_value}!\z/)
+      expect(page).to have_css("h3", text: /\ARedux Hello, #{new_value}!\z/)
     end
   end
 end

data/react_on_rails_pro/spec/dummy/yarn.lock

@@ -1228,10 +1228,6 @@
   dependencies:
     "@sentry/types" "7.120.0"
 
-"@shakacode-tools/react-on-rails-pro-node-renderer@link:.yalc/@shakacode-tools/react-on-rails-pro-node-renderer":
-  version "0.0.0"
-  uid ""
-
 "@shakacode/use-ssr-computation.macro@^1.2.4":
   version "1.2.4"
   resolved "https://registry.yarnpkg.com/@shakacode/use-ssr-computation.macro/-/use-ssr-computation.macro-1.2.4.tgz#b247d683e3133126dbdb42060c26e1c34a8b625d"
@@ -5461,6 +5457,10 @@ react-is@^16.12.0, react-is@^16.13.1, react-is@^16.7.0:
   resolved "https://registry.yarnpkg.com/react-is/-/react-is-16.13.1.tgz#789729a4dc36de2999dc156dd6c1d9c18cea56a4"
   integrity sha512-24e6ynE2H+OKt4kqsOvNd8kBpV65zoxbA4BVsEOB3ARVWQki/DHzaUoC5KuON/BiccDaCCTZBuOcfZs70kR8bQ==
 
+"react-on-rails-pro-node-renderer@link:.yalc/react-on-rails-pro-node-renderer":
+  version "0.0.0"
+  uid ""
+
 "react-on-rails-pro@link:.yalc/react-on-rails-pro":
   version "0.0.0"
   uid ""

data/react_on_rails_pro/spec/execjs-compatible-dummy/config/environments/production.rb

@@ -55,7 +55,7 @@ Rails.application.configure do
 
   # Log to STDOUT by default
   config.logger = ActiveSupport::Logger.new($stdout)
-                                       .tap  { |logger| logger.formatter = ::Logger::Formatter.new }
+                                       .tap  { |logger| logger.formatter = Logger::Formatter.new }
                                        .then { |logger| ActiveSupport::TaggedLogging.new(logger) }
 
   # Prepend all log lines with the following tags.

data/react_on_rails_pro/spec/execjs-compatible-dummy/config/initializers/react_on_rails.rb

@@ -1,58 +1,31 @@
 # frozen_string_literal: true
 
-# See https://github.com/shakacode/react_on_rails/blob/master/docs/guides/configuration.md
-# for many more options.
+# ⚠️ TEST CONFIGURATION - Do not copy directly for production apps
+# This is the ExecJS-compatible dummy app for testing legacy webpacker compatibility.
+# See docs/api-reference/configuration.md for production configuration guidance.
 
 ReactOnRails.configure do |config|
-  # This configures the script to run to build the production assets by webpack. Set this to nil
-  # if you don't want react_on_rails building this file for you.
-  # If nil, then the standard shakacode/webpacker assets:precompile will run
-  # config.build_production_command = nil
-
-  ################################################################################
   ################################################################################
-  # TEST CONFIGURATION OPTIONS
-  # Below options are used with the use of this test helper:
-  # ReactOnRails::TestHelper.configure_rspec_to_compile_assets(config)
+  # Essential Configuration
   ################################################################################
+  # Configure server bundle for server-side rendering
+  config.server_bundle_js_file = "server-bundle.js"
 
-  # If you are using this in your spec_helper.rb (or rails_helper.rb):
-  #
-  # ReactOnRails::TestHelper.configure_rspec_to_compile_assets(config)
-  #
-  # with rspec then this controls what yarn command is run
-  # to automatically refresh your webpack assets on every test run.
-  #
-  # Alternately, you can remove the `ReactOnRails::TestHelper.configure_rspec_to_compile_assets`
-  # and set the config/webpacker.yml option for test to true.
+  # Test configuration
   config.build_test_command = "RAILS_ENV=test bin/webpacker"
 
   ################################################################################
+  # File System Based Component Registry (Optional - Disabled for this test)
   ################################################################################
-  # SERVER RENDERING OPTIONS
-  ################################################################################
-  # This is the file used for server rendering of React when using `(prerender: true)`
-  # If you are never using server rendering, you should set this to "".
-  # Note, there is only one server bundle, unlike JavaScript where you want to minimize the size
-  # of the JS sent to the client. For the server rendering, React on Rails creates a pool of
-  # JavaScript execution instances which should handle any component requested.
-  #
-  # While you may configure this to be the same as your client bundle file, this file is typically
-  # different. You should have ONE server bundle which can create all of your server rendered
-  # React components.
-  #
-  config.server_bundle_js_file = "server-bundle.js"
+  # Uncomment to enable automatic component registration:
+  # config.components_subdirectory = "ror_components"
+  # config.auto_load_bundle = true
+  config.auto_load_bundle = false
 
   ################################################################################
+  # Advanced Configuration
   ################################################################################
-  # FILE SYSTEM BASED COMPONENT REGISTRY
-  ################################################################################
-  # `components_subdirectory`  is the name of the matching directories that contain automatically registered components
-  # for use in the Rails views. The default is nil, you can enable the feature by updating it in the next line.
-  # config.components_subdirectory = "ror_components"
-  #
-  # For automated component registry, `render_component` view helper method tries to load bundle for component from
-  # generated directory. default is false, you can pass option at the time of individual usage or update the default
-  # in the following line
-  config.auto_load_bundle = false
+  # Most options have sensible defaults. For advanced configuration including
+  # component loading strategies, server bundle security, and more, see:
+  # https://github.com/shakacode/react_on_rails/blob/master/docs/api-reference/configuration.md
 end

data/react_on_rails_pro/spec/react_on_rails_pro/assets_precompile_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative "./spec_helper"
+require_relative "spec_helper"
 require_relative "../../lib/react_on_rails_pro/assets_precompile"
 
 describe ReactOnRailsPro::AssetsPrecompile do
@@ -46,8 +46,6 @@ describe ReactOnRailsPro::AssetsPrecompile do
 
       ror_pro_config = instance_double(ReactOnRailsPro::Configuration)
 
-      allow(ror_pro_config).to receive(:dependency_globs).and_return([expected_parameters.last])
-
       adapter = Module.new do
         def self.cache_keys
           %w[a b]
@@ -58,7 +56,8 @@ describe ReactOnRailsPro::AssetsPrecompile do
         end
       end
 
-      allow(ror_pro_config).to receive(:remote_bundle_cache_adapter).and_return(adapter)
+      allow(ror_pro_config).to receive_messages(dependency_globs: [expected_parameters.last],
+                                                remote_bundle_cache_adapter: adapter)
 
       stub_const("ReactOnRailsPro::VERSION", "2.2.0")
 
@@ -136,11 +135,10 @@ describe ReactOnRailsPro::AssetsPrecompile do
       it "calls build_bundles & cache_bundles if cached bundles can't be fetched" do
         instance = described_class.instance
 
-        allow(instance).to receive(:fetch_and_unzip_cached_bundles).and_return(false)
         expect(instance).to receive(:fetch_and_unzip_cached_bundles).once
-        allow(instance).to receive(:build_bundles).and_return(nil)
         expect(instance).to receive(:build_bundles).once
-        allow(instance).to receive(:cache_bundles).and_return(nil)
+        allow(instance).to receive_messages(fetch_and_unzip_cached_bundles: false, build_bundles: nil,
+                                            cache_bundles: nil)
         expect(instance).to receive(:cache_bundles).once
 
         instance.build_or_fetch_bundles
@@ -179,9 +177,9 @@ describe ReactOnRailsPro::AssetsPrecompile do
       unique_variable = { unique_key: "a unique value" }
 
       instance = described_class.instance
-      allow(instance).to receive(:remote_bundle_cache_adapter).and_return(adapter_double)
-      allow(instance).to receive(:zipped_bundles_filename).and_return(unique_variable)
-      allow(instance).to receive(:zipped_bundles_filepath).and_return("zipped_bundles_filepath")
+      allow(instance).to receive_messages(remote_bundle_cache_adapter: adapter_double,
+                                          zipped_bundles_filename: unique_variable,
+                                          zipped_bundles_filepath: "zipped_bundles_filepath")
 
       allow(File).to receive(:binwrite).and_return(true)
       expect(File).to receive(:binwrite).once
@@ -197,8 +195,7 @@ describe ReactOnRailsPro::AssetsPrecompile do
       allow(File).to receive(:exist?).and_return(false)
 
       instance = described_class.instance
-      allow(instance).to receive(:fetch_bundles).and_return(false)
-      allow(instance).to receive(:zipped_bundles_filepath).and_return("a")
+      allow(instance).to receive_messages(fetch_bundles: false, zipped_bundles_filepath: "a")
 
       expect(instance.fetch_and_unzip_cached_bundles).to be(false)
     end
@@ -256,10 +253,10 @@ describe ReactOnRailsPro::AssetsPrecompile do
       zipped_bundles_filepath = Pathname.new(Dir.tmpdir).join("foobar")
 
       instance = described_class.instance
-      allow(instance).to receive(:remote_bundle_cache_adapter).and_return(adapter_double)
-      allow(instance).to receive(:zipped_bundles_filename).and_return("zipped_bundles_filename")
-      allow(instance).to receive(:zipped_bundles_filepath).and_return(zipped_bundles_filepath)
-      allow(instance).to receive(:remove_extra_files_cache_dir).and_return(nil)
+      allow(instance).to receive_messages(remote_bundle_cache_adapter: adapter_double,
+                                          zipped_bundles_filename: "zipped_bundles_filename",
+                                          zipped_bundles_filepath: zipped_bundles_filepath,
+                                          remove_extra_files_cache_dir: nil)
 
       expect(instance.cache_bundles).to be_truthy
 
@@ -289,8 +286,8 @@ describe ReactOnRailsPro::AssetsPrecompile do
 
       instance = described_class.instance
 
-      allow(instance).to receive(:remote_bundle_cache_adapter).and_return(adapter)
-      allow(instance).to receive(:extra_files_path).and_return(Pathname.new(Dir.pwd).join("extra_files_cache_dir"))
+      allow(instance).to receive_messages(remote_bundle_cache_adapter: adapter,
+                                          extra_files_path: Pathname.new(Dir.pwd).join("extra_files_cache_dir"))
       copied_gemfile_path = Pathname.new(Dir.pwd).join("extra_files_cache_dir", "Gemfile")
       copied_assets_precompile_path = Pathname.new(Dir.pwd).join("extra_files_cache_dir",
                                                                  "lib---react_on_rails_pro---assets_precompile.rb")

data/react_on_rails_pro/spec/react_on_rails_pro/cache_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative "./spec_helper"
+require_relative "spec_helper"
 
 class TestingCache
   def call; end

data/react_on_rails_pro/spec/react_on_rails_pro/configuration_spec.rb

@@ -2,7 +2,7 @@
 
 require_relative "spec_helper"
 
-module ReactOnRailsPro
+module ReactOnRailsPro # rubocop:disable Metrics/ModuleLength
   RSpec.describe Configuration do
     after do
       ReactOnRailsPro.instance_variable_set(:@configuration, nil)
@@ -205,7 +205,8 @@ module ReactOnRailsPro
 
         expect(ReactOnRailsPro.configuration.rsc_bundle_js_file).to eq("rsc-bundle.js")
         expect(ReactOnRailsPro.configuration.react_client_manifest_file).to eq("react-client-manifest.json")
-        expect(ReactOnRailsPro.configuration.react_server_client_manifest_file).to eq("react-server-client-manifest.json")
+        expect(ReactOnRailsPro.configuration.react_server_client_manifest_file)
+          .to eq("react-server-client-manifest.json")
       end
 
       it "allows setting rsc_bundle_js_file" do
@@ -229,7 +230,8 @@ module ReactOnRailsPro
           config.react_server_client_manifest_file = "custom-server-client-manifest.json"
         end
 
-        expect(ReactOnRailsPro.configuration.react_server_client_manifest_file).to eq("custom-server-client-manifest.json")
+        expect(ReactOnRailsPro.configuration.react_server_client_manifest_file)
+          .to eq("custom-server-client-manifest.json")
       end
 
       it "allows nil values for RSC configuration options" do