Fix Rubobop issue in Ruby v3

Author: ahangarha

lib/generators/react/install_generator.rb

@@ -21,14 +21,14 @@ class InstallGenerator < ::Rails::Generators::Base
       # Make an empty `components/` directory in the right place:
       def create_directory
         components_dir = if shakapacker?
-          Pathname.new(javascript_dir).parent.to_s
-        else
-          javascript_dir
-        end
-        empty_directory File.join(components_dir, 'components')
-        unless options[:skip_git]
-          create_file File.join(components_dir, 'components/.keep')
-        end
+                           Pathname.new(javascript_dir).parent.to_s
+                         else
+                           javascript_dir
+                         end
+        empty_directory File.join(components_dir, "components")
+        return if options[:skip_git]
+
+        create_file File.join(components_dir, "components/.keep")
       end
 
       # Add requires, setup UJS
@@ -42,10 +42,10 @@ def setup_react
 
       def create_server_rendering
         if options[:skip_server_rendering]
-          return
+          nil
         elsif shakapacker?
-          ssr_manifest_path = File.join(javascript_dir, 'server_rendering.js')
-          template('server_rendering_pack.js', ssr_manifest_path)
+          ssr_manifest_path = File.join(javascript_dir, "server_rendering.js")
+          template("server_rendering_pack.js", ssr_manifest_path)
         else
           ssr_manifest_path = File.join(javascript_dir, "server_rendering.js")
           template("server_rendering.js", ssr_manifest_path)
@@ -112,8 +112,6 @@ def setup_react_shakapacker
         end
       end
 
-      private
-
       def shakapacker_source_path
         Shakapacker.config.source_entry_path
       end

test/react/server_rendering/bundle_renderer_test.rb

@@ -105,6 +105,7 @@ class BundleRendererTest < ActiveSupport::TestCase
       test ".new accepts any filenames" do
         limited_renderer = React::ServerRendering::BundleRenderer.new(files: ["react-server.js", "react_ujs.js",
                                                                               "components/Todo.js"])
+
         assert_match(%r{get a real job</li>}, limited_renderer.render("Todo", { todo: "get a real job" }, nil))
         err = assert_raises React::ServerRendering::PrerenderError do
           limited_renderer.render("TodoList", { todos: [] }, nil)

test/server_rendered_html_test.rb

@@ -95,7 +95,7 @@ def wait_to_ensure_asset_pipeline_detects_changes
       # Make sure console messages are replayed when requested
       React::ServerRendering.renderer_options = { replay_console: true }
       React::ServerRendering.reset_pool
-      get "/server/console_example" 
+      get "/server/console_example"
 
       assert_match(/Console Logged/, response.body)
       assert_match(/console.log.apply\(console, \["got initial state"\]\)/, response.body)