Add three new rubocop rules

Style/SpaceBeforeBlockBraces
Style/SpaceInsideBlockBraces
Style/SpaceInsideHashLiteralBraces

Fix all violations in the repository.

4 files changed, 46 insertions, 46 deletions

diff --git a/actionpack/test/dispatch/request/json_params_parsing_test.rb b/actionpack/test/dispatch/request/json_params_parsing_test.rb
index e24b1e12e3..d0cd32a242 100644
--- a/actionpack/test/dispatch/request/json_params_parsing_test.rb
+++ b/actionpack/test/dispatch/request/json_params_parsing_test.rb
@@ -18,21 +18,21 @@ class JsonParamsParsingTest < ActionDispatch::IntegrationTest
 
   test "parses json params for application json" do
     assert_parses(
-      {"person" => {"name" => "David"}},
+      { "person" => { "name" => "David" } },
       "{\"person\": {\"name\": \"David\"}}", "CONTENT_TYPE" => "application/json"
     )
   end
 
   test "parses boolean and number json params for application json" do
     assert_parses(
-      {"item" => {"enabled" => false, "count" => 10}},
+      { "item" => { "enabled" => false, "count" => 10 } },
       "{\"item\": {\"enabled\": false, \"count\": 10}}", "CONTENT_TYPE" => "application/json"
     )
   end
 
   test "parses json params for application jsonrequest" do
     assert_parses(
-      {"person" => {"name" => "David"}},
+      { "person" => { "name" => "David" } },
       "{\"person\": {\"name\": \"David\"}}", "CONTENT_TYPE" => "application/jsonrequest"
     )
   end
@@ -46,15 +46,15 @@ class JsonParamsParsingTest < ActionDispatch::IntegrationTest
 
   test "nils are stripped from collections" do
     assert_parses(
-      {"person" => []},
+      { "person" => [] },
       "{\"person\":[null]}", "CONTENT_TYPE" => "application/json"
     )
     assert_parses(
-      {"person" => ["foo"]},
+      { "person" => ["foo"] },
       "{\"person\":[\"foo\",null]}", "CONTENT_TYPE" => "application/json"
     )
     assert_parses(
-      {"person" => []},
+      { "person" => [] },
       "{\"person\":[null, null]}", "CONTENT_TYPE" => "application/json"
     )
   end
@@ -133,21 +133,21 @@ class RootLessJSONParamsParsingTest < ActionDispatch::IntegrationTest
 
   test "parses json params for application json" do
     assert_parses(
-      {"user" => {"username" => "sikachu"}, "username" => "sikachu"},
+      { "user" => { "username" => "sikachu" }, "username" => "sikachu" },
       "{\"username\": \"sikachu\"}", "CONTENT_TYPE" => "application/json"
     )
   end
 
   test "parses json params for application jsonrequest" do
     assert_parses(
-      {"user" => {"username" => "sikachu"}, "username" => "sikachu"},
+      { "user" => { "username" => "sikachu" }, "username" => "sikachu" },
       "{\"username\": \"sikachu\"}", "CONTENT_TYPE" => "application/jsonrequest"
     )
   end
 
   test "parses json with non-object JSON content" do
     assert_parses(
-      {"user" => {"_json" => "string content" }, "_json" => "string content" },
+      { "user" => { "_json" => "string content" }, "_json" => "string content" },
       "\"string content\"", "CONTENT_TYPE" => "application/json"
     )
   end
@@ -157,7 +157,7 @@ class RootLessJSONParamsParsingTest < ActionDispatch::IntegrationTest
       Mime::Type.unregister :json
       Mime::Type.register "application/json", :json, %w(application/vnd.rails+json)
       assert_parses(
-        {"user" => {"username" => "meinac"}, "username" => "meinac"},
+        { "user" => { "username" => "meinac" }, "username" => "meinac" },
         "{\"username\": \"meinac\"}", "CONTENT_TYPE" => "application/json"
       )
     ensure
@@ -171,7 +171,7 @@ class RootLessJSONParamsParsingTest < ActionDispatch::IntegrationTest
       Mime::Type.unregister :json
       Mime::Type.register "application/json", :json, %w(application/vnd.rails+json)
       assert_parses(
-        {"user" => {"username" => "meinac"}, "username" => "meinac"},
+        { "user" => { "username" => "meinac" }, "username" => "meinac" },
         "{\"username\": \"meinac\"}", "CONTENT_TYPE" => "application/vnd.rails+json"
       )
     ensure
diff --git a/actionpack/test/dispatch/request/multipart_params_parsing_test.rb b/actionpack/test/dispatch/request/multipart_params_parsing_test.rb
index 2714b7f50b..e572c722a0 100644
--- a/actionpack/test/dispatch/request/multipart_params_parsing_test.rb
+++ b/actionpack/test/dispatch/request/multipart_params_parsing_test.rb
@@ -32,11 +32,11 @@ class MultipartParamsParsingTest < ActionDispatch::IntegrationTest
   end
 
   test "parses bracketed parameters" do
-    assert_equal({ "foo" => { "baz" => "bar"}}, parse_multipart("bracketed_param"))
+    assert_equal({ "foo" => { "baz" => "bar" } }, parse_multipart("bracketed_param"))
   end
 
   test "parse single utf8 parameter" do
-    assert_equal({ "Iñtërnâtiônàlizætiøn_name" => "Iñtërnâtiônàlizætiøn_value"},
+    assert_equal({ "Iñtërnâtiônàlizætiøn_name" => "Iñtërnâtiônàlizætiøn_value" },
                  parse_multipart("single_utf8_param"), "request.request_parameters")
     assert_equal(
       "Iñtërnâtiônàlizætiøn_value",
@@ -45,10 +45,10 @@ class MultipartParamsParsingTest < ActionDispatch::IntegrationTest
 
   test "parse bracketed utf8 parameter" do
     assert_equal({ "Iñtërnâtiônàlizætiøn_name" => {
-      "Iñtërnâtiônàlizætiøn_nested_name" => "Iñtërnâtiônàlizætiøn_value"} },
+      "Iñtërnâtiônàlizætiøn_nested_name" => "Iñtërnâtiônàlizætiøn_value" } },
       parse_multipart("bracketed_utf8_param"), "request.request_parameters")
     assert_equal(
-      {"Iñtërnâtiônàlizætiøn_nested_name" => "Iñtërnâtiônàlizætiøn_value"},
+      { "Iñtërnâtiônàlizætiøn_nested_name" => "Iñtërnâtiônàlizætiøn_value" },
       TestController.last_parameters["Iñtërnâtiônàlizætiøn_name"], "request.parameters")
   end
 
diff --git a/actionpack/test/dispatch/request/query_string_parsing_test.rb b/actionpack/test/dispatch/request/query_string_parsing_test.rb
index 6763dbf243..5c992be216 100644
--- a/actionpack/test/dispatch/request/query_string_parsing_test.rb
+++ b/actionpack/test/dispatch/request/query_string_parsing_test.rb
@@ -29,92 +29,92 @@ class QueryStringParsingTest < ActionDispatch::IntegrationTest
 
   test "query string" do
     assert_parses(
-      {"action" => "create_customer", "full_name" => "David Heinemeier Hansson", "customerId" => "1"},
+      { "action" => "create_customer", "full_name" => "David Heinemeier Hansson", "customerId" => "1" },
       "action=create_customer&full_name=David%20Heinemeier%20Hansson&customerId=1"
     )
   end
 
   test "deep query string" do
     assert_parses(
-      {"x" => {"y" => {"z" => "10"}}},
+      { "x" => { "y" => { "z" => "10" } } },
       "x[y][z]=10"
     )
   end
 
   test "deep query string with array" do
-    assert_parses({"x" => {"y" => {"z" => ["10"]}}}, "x[y][z][]=10")
-    assert_parses({"x" => {"y" => {"z" => ["10", "5"]}}}, "x[y][z][]=10&x[y][z][]=5")
+    assert_parses({ "x" => { "y" => { "z" => ["10"] } } }, "x[y][z][]=10")
+    assert_parses({ "x" => { "y" => { "z" => ["10", "5"] } } }, "x[y][z][]=10&x[y][z][]=5")
   end
 
   test "deep query string with array of hash" do
-    assert_parses({"x" => {"y" => [{"z" => "10"}]}}, "x[y][][z]=10")
-    assert_parses({"x" => {"y" => [{"z" => "10", "w" => "10"}]}}, "x[y][][z]=10&x[y][][w]=10")
-    assert_parses({"x" => {"y" => [{"z" => "10", "v" => {"w" => "10"}}]}}, "x[y][][z]=10&x[y][][v][w]=10")
+    assert_parses({ "x" => { "y" => [{ "z" => "10" }] } }, "x[y][][z]=10")
+    assert_parses({ "x" => { "y" => [{ "z" => "10", "w" => "10" }] } }, "x[y][][z]=10&x[y][][w]=10")
+    assert_parses({ "x" => { "y" => [{ "z" => "10", "v" => { "w" => "10" } }] } }, "x[y][][z]=10&x[y][][v][w]=10")
   end
 
   test "deep query string with array of hashes with one pair" do
-    assert_parses({"x" => {"y" => [{"z" => "10"}, {"z" => "20"}]}}, "x[y][][z]=10&x[y][][z]=20")
+    assert_parses({ "x" => { "y" => [{ "z" => "10" }, { "z" => "20" }] } }, "x[y][][z]=10&x[y][][z]=20")
   end
 
   test "deep query string with array of hashes with multiple pairs" do
     assert_parses(
-      {"x" => {"y" => [{"z" => "10", "w" => "a"}, {"z" => "20", "w" => "b"}]}},
+      { "x" => { "y" => [{ "z" => "10", "w" => "a" }, { "z" => "20", "w" => "b" }] } },
       "x[y][][z]=10&x[y][][w]=a&x[y][][z]=20&x[y][][w]=b"
     )
   end
 
   test "query string with nil" do
     assert_parses(
-      { "action" => "create_customer", "full_name" => ""},
+      { "action" => "create_customer", "full_name" => "" },
       "action=create_customer&full_name="
     )
   end
 
   test "query string with array" do
     assert_parses(
-      { "action" => "create_customer", "selected" => ["1", "2", "3"]},
+      { "action" => "create_customer", "selected" => ["1", "2", "3"] },
       "action=create_customer&selected[]=1&selected[]=2&selected[]=3"
     )
   end
 
   test "query string with amps" do
     assert_parses(
-      { "action" => "create_customer", "name" => "Don't & Does"},
+      { "action" => "create_customer", "name" => "Don't & Does" },
       "action=create_customer&name=Don%27t+%26+Does"
     )
   end
 
   test "query string with many equal" do
     assert_parses(
-      { "action" => "create_customer", "full_name" => "abc=def=ghi"},
+      { "action" => "create_customer", "full_name" => "abc=def=ghi" },
       "action=create_customer&full_name=abc=def=ghi"
     )
   end
 
   test "query string without equal" do
-    assert_parses({"action" => nil}, "action")
-    assert_parses({"action" => {"foo" => nil}}, "action[foo]")
-    assert_parses({"action" => {"foo" => { "bar" => nil }}}, "action[foo][bar]")
-    assert_parses({"action" => {"foo" => { "bar" => [] }}}, "action[foo][bar][]")
-    assert_parses({"action" => {"foo" => [] }}, "action[foo][]")
-    assert_parses({"action"=>{"foo"=>[{"bar"=>nil}]}}, "action[foo][][bar]")
+    assert_parses({ "action" => nil }, "action")
+    assert_parses({ "action" => { "foo" => nil } }, "action[foo]")
+    assert_parses({ "action" => { "foo" => { "bar" => nil } } }, "action[foo][bar]")
+    assert_parses({ "action" => { "foo" => { "bar" => [] } } }, "action[foo][bar][]")
+    assert_parses({ "action" => { "foo" => [] } }, "action[foo][]")
+    assert_parses({ "action"=>{ "foo"=>[{ "bar"=>nil }] } }, "action[foo][][bar]")
   end
 
   def test_array_parses_without_nil
-    assert_parses({"action" => ["1"]}, "action[]=1&action[]")
+    assert_parses({ "action" => ["1"] }, "action[]=1&action[]")
   end
 
   test "perform_deep_munge" do
     old_perform_deep_munge = ActionDispatch::Request::Utils.perform_deep_munge
     ActionDispatch::Request::Utils.perform_deep_munge = false
     begin
-      assert_parses({"action" => nil}, "action")
-      assert_parses({"action" => {"foo" => nil}}, "action[foo]")
-      assert_parses({"action" => {"foo" => {"bar" => nil}}}, "action[foo][bar]")
-      assert_parses({"action" => {"foo" => {"bar" => [nil]}}}, "action[foo][bar][]")
-      assert_parses({"action" => {"foo" => [nil]}}, "action[foo][]")
-      assert_parses({"action" => {"foo" => [{"bar" => nil}]}}, "action[foo][][bar]")
-      assert_parses({"action" => ["1",nil]}, "action[]=1&action[]")
+      assert_parses({ "action" => nil }, "action")
+      assert_parses({ "action" => { "foo" => nil } }, "action[foo]")
+      assert_parses({ "action" => { "foo" => { "bar" => nil } } }, "action[foo][bar]")
+      assert_parses({ "action" => { "foo" => { "bar" => [nil] } } }, "action[foo][bar][]")
+      assert_parses({ "action" => { "foo" => [nil] } }, "action[foo][]")
+      assert_parses({ "action" => { "foo" => [{ "bar" => nil }] } }, "action[foo][][bar]")
+      assert_parses({ "action" => ["1",nil] }, "action[]=1&action[]")
     ensure
       ActionDispatch::Request::Utils.perform_deep_munge = old_perform_deep_munge
     end
@@ -129,14 +129,14 @@ class QueryStringParsingTest < ActionDispatch::IntegrationTest
 
   test "query string with many ampersands" do
     assert_parses(
-      { "action" => "create_customer", "full_name" => "David Heinemeier Hansson"},
+      { "action" => "create_customer", "full_name" => "David Heinemeier Hansson" },
       "&action=create_customer&&&full_name=David%20Heinemeier%20Hansson"
     )
   end
 
   test "unbalanced query string with array" do
     assert_parses(
-      {"location" => ["1", "2"], "age_group" => ["2"]},
+      { "location" => ["1", "2"], "age_group" => ["2"] },
       "location[]=1&location[]=2&age_group[]=2"
     )
   end
diff --git a/actionpack/test/dispatch/request/session_test.rb b/actionpack/test/dispatch/request/session_test.rb
index cc3c4d48f3..311b80ea0a 100644
--- a/actionpack/test/dispatch/request/session_test.rb
+++ b/actionpack/test/dispatch/request/session_test.rb
@@ -19,7 +19,7 @@ module ActionDispatch
         s = Session.create(store, req, {})
         s["foo"] = "bar"
         assert_equal "bar", s["foo"]
-        assert_equal({"foo" => "bar"}, s.to_hash)
+        assert_equal({ "foo" => "bar" }, s.to_hash)
       end
 
       def test_create_merges_old
@@ -98,7 +98,7 @@ module ActionDispatch
         assert_equal "2", session.fetch(:two, "2")
         assert_nil session.fetch(:two, nil)
 
-        assert_equal "three", session.fetch(:three) {|el| el.to_s }
+        assert_equal "three", session.fetch(:three) { |el| el.to_s }
 
         assert_raise KeyError do
           session.fetch(:three)