rubocop --only Layout/SpaceInsideHashLiteralBraces -a

Author: BookOfGreg

lib/react/rails/component_mount.rb

@@ -24,7 +24,7 @@ def teardown(controller)
       # are used by react_ujs to actually instantiate the React component
       # on the client.
       def react_component(name, props = {}, options = {}, &block)
-        options = {:tag => options} if options.is_a?(Symbol)
+        options = { :tag => options } if options.is_a?(Symbol)
         if options.fetch(:camelize_props, camelize_props_switch)
           props = React.camelize_props(props)
         end

lib/react/server_rendering.rb

@@ -13,7 +13,7 @@ module ServerRendering
     # This will clear all state such as loaded code, JS VM state, or options.
     # @return [void]
     def self.reset_pool
-      options = {size: pool_size, timeout: pool_timeout}
+      options = { size: pool_size, timeout: pool_timeout }
       @@pool = ConnectionPool.new(options) { self.renderer.new(self.renderer_options) }
     end
 

test/react/jsx/jsx_transformer_test.rb

@@ -28,7 +28,7 @@ class JSXTransformerTest < ActionDispatch::IntegrationTest
     end
 
     test 'accepts harmony: true option' do
-      React::JSX.transform_options = {harmony: true}
+      React::JSX.transform_options = { harmony: true }
       get '/assets/harmony_example.js'
       assert_response :success
       assert_match(/generateGreeting:\s*function\(\)/, @response.body, "object literal methods")
@@ -38,7 +38,7 @@ class JSXTransformerTest < ActionDispatch::IntegrationTest
     end
 
     test 'accepts strip_types: true option' do
-      React::JSX.transform_options = {strip_types: true, harmony: true}
+      React::JSX.transform_options = { strip_types: true, harmony: true }
       get '/assets/flow_types_example.js'
       assert_response :success
       assert_match(/\(i\s*,\s*name\s*\)\s*\{/, @response.body, "type annotations are removed")
@@ -49,7 +49,7 @@ class JSXTransformerTest < ActionDispatch::IntegrationTest
       custom_path =     Rails.root.join("vendor/assets/react/custom")
       replacing_path =  custom_path.join("CustomTransformer.js")
 
-      React::JSX.transform_options = {asset_path: "custom/CustomTransformer.js"}
+      React::JSX.transform_options = { asset_path: "custom/CustomTransformer.js" }
 
       FileUtils.mkdir_p(custom_path)
       FileUtils.cp(hidden_path, replacing_path)

test/react/jsx_test.rb

@@ -59,7 +59,7 @@ class JSXTransformTest < ActionDispatch::IntegrationTest
     end
 
     def test_babel_transformer_accepts_babel_transformation_options
-      React::JSX.transform_options = {blacklist: ['spec.functionName', 'validation.react', "strict"]}
+      React::JSX.transform_options = { blacklist: ['spec.functionName', 'validation.react', "strict"] }
       SprocketsHelpers.manually_expire_asset('example.js')
       get '/assets/example.js'
       assert_response :success

test/react/rails/component_mount_test.rb

@@ -12,7 +12,7 @@ class ComponentMountTest < ActionDispatch::IntegrationTest
     end
 
     test '#react_component accepts React props' do
-      html = @helper.react_component('Foo', {bar: 'value'})
+      html = @helper.react_component('Foo', { bar: 'value' })
       expected_props = %w(data-react-class="Foo" data-react-props="{&quot;bar&quot;:&quot;value&quot;}")
       expected_props.each do |segment|
         assert html.include?(segment)
@@ -22,7 +22,7 @@ class ComponentMountTest < ActionDispatch::IntegrationTest
     test '#react_component accepts React props with camelize_props' do
       React::Rails::ComponentMount.camelize_props_switch = true
       helper = React::Rails::ComponentMount.new
-      html = helper.react_component('Foo', {foo_bar: 'value'})
+      html = helper.react_component('Foo', { foo_bar: 'value' })
       expected_props = %w(data-react-class="Foo" data-react-props="{&quot;fooBar&quot;:&quot;value&quot;}")
       expected_props.each do |segment|
         assert html.include?(segment)
@@ -32,15 +32,15 @@ class ComponentMountTest < ActionDispatch::IntegrationTest
     test '#react_component allows camelize_props to be passed in as an option' do
       React::Rails::ComponentMount.camelize_props_switch = false
       helper = React::Rails::ComponentMount.new
-      html = helper.react_component('Foo', {foo_bar: 'value'}, camelize_props: true)
+      html = helper.react_component('Foo', { foo_bar: 'value' }, camelize_props: true)
       expected_props = %w(data-react-class="Foo" data-react-props="{&quot;fooBar&quot;:&quot;value&quot;}")
       expected_props.each do |segment|
         assert html.include?(segment)
       end
 
       React::Rails::ComponentMount.camelize_props_switch = true
       helper = React::Rails::ComponentMount.new
-      html = helper.react_component('Foo', {foo_bar: 'value'}, camelize_props: false)
+      html = helper.react_component('Foo', { foo_bar: 'value' }, camelize_props: false)
       expected_props = %w(data-react-class="Foo" data-react-props="{&quot;foo_bar&quot;:&quot;value&quot;}")
       expected_props.each do |segment|
         assert html.include?(segment)
@@ -50,7 +50,7 @@ class ComponentMountTest < ActionDispatch::IntegrationTest
     test '#react_component accepts React props with camelize_props containing nested arrays' do
       React::Rails::ComponentMount.camelize_props_switch = true
       helper = React::Rails::ComponentMount.new
-      html = helper.react_component('Foo', {foo_bar: [{user_name: 'Ryan'}, {user_name: 'Matt'}], bar_foo: 1})
+      html = helper.react_component('Foo', { foo_bar: [{ user_name: 'Ryan' }, { user_name: 'Matt' }], bar_foo: 1 })
       expected_props = %w(data-react-class="Foo" data-react-props="{&quot;fooBar&quot;:[{&quot;userName&quot;:&quot;Ryan&quot;},{&quot;userName&quot;:&quot;Matt&quot;}],&quot;barFoo&quot;:1}")
       expected_props.each do |segment|
         assert html.include?(segment)
@@ -70,27 +70,27 @@ class ComponentMountTest < ActionDispatch::IntegrationTest
     end
 
     test '#react_component accepts string props with prerender: true' do
-      html = @helper.react_component('Todo', {todo: 'render on the server'}.to_json, prerender: true)
+      html = @helper.react_component('Todo', { todo: 'render on the server' }.to_json, prerender: true)
       assert(html.include?('data-react-class="Todo"'), "it includes attrs for UJS")
       assert(html.include?('>render on the server</li>'), "it includes rendered HTML")
       assert(html.include?('data-reactid'), "it includes React properties")
     end
 
     test '#react_component passes :static to BundleRenderer' do
-      html = @helper.react_component('Todo', {todo: 'render on the server'}.to_json, prerender: :static)
+      html = @helper.react_component('Todo', { todo: 'render on the server' }.to_json, prerender: :static)
       assert(html.include?('>render on the server</li>'), "it includes rendered HTML")
       assert(!html.include?('data-reactid'), "it DOESNT INCLUDE React properties")
     end
 
     test '#react_component does not include HTML properties with a static render' do
-      html = @helper.react_component('Todo', {todo: 'render on the server'}.to_json, prerender: :static)
+      html = @helper.react_component('Todo', { todo: 'render on the server' }.to_json, prerender: :static)
       assert_equal('<div><li>render on the server</li></div>', html)
     end
 
     test '#react_component accepts HTML options and HTML tag' do
       assert @helper.react_component('Foo', {}, :span).match(/<span\s.*><\/span>/)
 
-      html = @helper.react_component('Foo', {}, {class: 'test', tag: :span, data: {foo: 1}})
+      html = @helper.react_component('Foo', {}, { class: 'test', tag: :span, data: { foo: 1 } })
       assert html.match(/<span\s.*><\/span>/)
       assert html.include?('class="test"')
       assert html.include?('data-foo="1"')
@@ -110,7 +110,7 @@ def self.react_rails_prerenderer
 
     test "it uses the controller's react_rails_prerenderer, if available" do
       @helper.setup(DummyController)
-      rendered_component = @helper.react_component('Foo', {"ok" => true}, prerender: :static)
+      rendered_component = @helper.react_component('Foo', { "ok" => true }, prerender: :static)
       assert_equal %|<div>rendered Foo with {&quot;ok&quot;:true}</div>|, rendered_component
     end
   end

test/react/rails/view_helper_test.rb

@@ -11,13 +11,13 @@ class ViewHelperHelper
 class ViewHelperTest < ActionView::TestCase
   test "view helper can be called directly" do
     expected_html = %{<div data-react-class="Component" data-react-props="{&quot;a&quot;:&quot;b&quot;}"></div>}
-    rendered_html = ViewHelperHelper.react_component("Component", {a: "b"})
+    rendered_html = ViewHelperHelper.react_component("Component", { a: "b" })
     assert_equal(expected_html, rendered_html)
   end
 
   test 'view helper accepts block usage' do
     expected_html = %{<div data-react-class="Component" data-react-props="{&quot;a&quot;:&quot;b&quot;}">content</div>}
-    rendered_html = ViewHelperHelper.react_component("Component", {a: "b"}) do
+    rendered_html = ViewHelperHelper.react_component("Component", { a: "b" }) do
       "content"
     end
     assert_equal(expected_html, rendered_html)

test/react/server_rendering/bundle_renderer_test.rb

@@ -31,33 +31,33 @@ class BundleRendererTest < ActiveSupport::TestCase
     end
 
     test '#render returns HTML' do
-      result = @renderer.render("Todo", {todo: "write tests"}, nil)
+      result = @renderer.render("Todo", { todo: "write tests" }, nil)
       assert_match(/<li.*write tests<\/li>/, result)
       assert_match(/data-react-checksum/, result)
     end
 
     test '#render accepts strings' do
-      result = @renderer.render("Todo", {todo: "write more tests"}.to_json, nil)
+      result = @renderer.render("Todo", { todo: "write more tests" }.to_json, nil)
       assert_match(/<li.*write more tests<\/li>/, result)
     end
 
     test '#render accepts :static pre-render option' do
-      result = @renderer.render("Todo", {todo: "write more tests"}, :static)
+      result = @renderer.render("Todo", { todo: "write more tests" }, :static)
       assert_match(/<li>write more tests<\/li>/, result)
       assert_no_match(/data-react-checksum/, result)
     end
 
     test '#render replays console messages' do
-      result = @renderer.render("TodoListWithConsoleLog", {todos: ["log some messages"]}, nil)
+      result = @renderer.render("TodoListWithConsoleLog", { todos: ["log some messages"] }, nil)
       assert_match(/<script class="react-rails-console-replay">$/, result)
       assert_match(/console.log.apply\(console, \["got initial state"\]\);$/, result)
       assert_match(/console.warn.apply\(console, \["mounted component"\]\);$/, result)
       assert_match(/console.error.apply\(console, \["rendered!","foo"\]\);$/, result)
     end
 
     test '#render console messages can be disabled' do
-      no_log_renderer = React::ServerRendering::BundleRenderer.new({replay_console: false})
-      result = no_log_renderer.render("TodoListWithConsoleLog", {todos: ["log some messages"]}, nil)
+      no_log_renderer = React::ServerRendering::BundleRenderer.new({ replay_console: false })
+      result = no_log_renderer.render("TodoListWithConsoleLog", { todos: ["log some messages"] }, nil)
       assert_no_match(/console.log.apply\(console, \["got initial state"\]\)/, result)
       assert_no_match(/console.warn.apply\(console, \["mounted component"\]\)/, result)
       assert_no_match(/console.error.apply\(console, \["rendered!","foo"\]\)/, result)
@@ -97,15 +97,15 @@ class BundleRendererTest < ActiveSupport::TestCase
 
         additional_renderer = React::ServerRendering::BundleRenderer.new(code: additional_code)
 
-        assert_match(/drink more caffeine<\/span>/, additional_renderer.render("WithoutSprockets", {label: "drink more caffeine"}, nil))
+        assert_match(/drink more caffeine<\/span>/, additional_renderer.render("WithoutSprockets", { label: "drink more caffeine" }, nil))
       end
 
       # These use cases don't apply to Webpacker since the require.context comes from a pack:
       test '.new accepts any filenames' do
         limited_renderer = React::ServerRendering::BundleRenderer.new(files: ["react-server.js", "react_ujs.js", "components/Todo.js"])
-        assert_match(/get a real job<\/li>/, limited_renderer.render("Todo", {todo: "get a real job"}, nil))
+        assert_match(/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)
+          limited_renderer.render("TodoList", { todos: [] }, nil)
         end
         assert_match(/ReferenceError/, err.to_s, "it doesnt load other files")
       end
@@ -121,7 +121,7 @@ class BundleRendererTest < ActiveSupport::TestCase
 
           @renderer = React::ServerRendering::BundleRenderer.new(files: legacy_rendering_files)
 
-          result = @renderer.render("Todo", {todo: "write tests"}, nil)
+          result = @renderer.render("Todo", { todo: "write tests" }, nil)
           assert_match(/<li.*write tests<\/li>/, result)
           assert_match(/data-react-checksum/, result)
         ensure

test/react/server_rendering/console_replay_test.rb

@@ -4,7 +4,7 @@
   class ConsoleReplayTest < ActionDispatch::IntegrationTest
     setup do
       WebpackerHelpers.compile
-      React::ServerRendering.renderer_options = {replay_console: true}
+      React::ServerRendering.renderer_options = { replay_console: true }
       React::ServerRendering.reset_pool
     end
 

test/react/server_rendering/exec_js_renderer_test.rb

@@ -22,13 +22,13 @@ class ExecJSRendererTest < ActiveSupport::TestCase
   end
 
   test '#render returns HTML' do
-    result = @renderer.render("Todo", {todo: "write tests"}.to_json, {})
+    result = @renderer.render("Todo", { todo: "write tests" }.to_json, {})
     assert_match(/<li.*write tests<\/li>/, result)
     assert_match(/data-react-checksum/, result)
   end
 
   test '#render accepts render_function:' do
-    result = @renderer.render("Todo", {todo: "write more tests"}.to_json, render_function: "renderToStaticMarkup")
+    result = @renderer.render("Todo", { todo: "write more tests" }.to_json, render_function: "renderToStaticMarkup")
     assert_match(/<li>write more tests<\/li>/, result)
     assert_no_match(/data-react-checksum/, result)
   end
@@ -43,7 +43,7 @@ def @renderer.after_render(name, props, opts)
     end
 
     error = assert_raises(React::ServerRendering::PrerenderError) do
-      @renderer.render("Todo", {todo: "write tests"}.to_json, {})
+      @renderer.render("Todo", { todo: "write tests" }.to_json, {})
     end
 
     assert_match(/before_render/, error.message)
@@ -60,7 +60,7 @@ def @renderer.after_render(name, props, opts)
     end
 
     error = assert_raises(React::ServerRendering::PrerenderError) do
-      @renderer.render("Todo", {todo: "write tests"}.to_json, {})
+      @renderer.render("Todo", { todo: "write tests" }.to_json, {})
     end
 
     assert_match(/after_render/, error.message)
@@ -69,7 +69,7 @@ def @renderer.after_render(name, props, opts)
 
   test '.new accepts code:' do
     dummy_renderer = React::ServerRendering::ExecJSRenderer.new(code: DUMMY_IMPLEMENTATION)
-    result = dummy_renderer.render("Todo", {todo: "get a real job"}.to_json, {})
+    result = dummy_renderer.render("Todo", { todo: "get a real job" }.to_json, {})
     assert_equal("serverRender was called", result)
   end
 end

test/react/server_rendering_test.rb

@@ -27,7 +27,7 @@ class ReactServerRenderingTest < ActiveSupport::TestCase
   end
 
   test '.render returns a rendered string' do
-    props = {"props" => true}
+    props = { "props" => true }
     result = React::ServerRendering.render("MyComponent", props, "prerender-opts")
     assert_equal("TEST rendered MyComponent with #{props} and prerender-opts", result)
   end