Add three new rubocop rules

Style/SpaceBeforeBlockBraces
Style/SpaceInsideBlockBraces
Style/SpaceInsideHashLiteralBraces

Fix all violations in the repository.

5 files changed, 26 insertions, 26 deletions

diff --git a/actionpack/test/journey/gtg/transition_table_test.rb b/actionpack/test/journey/gtg/transition_table_test.rb
index 23b05cc5f6..4c8b5032eb 100644
--- a/actionpack/test/journey/gtg/transition_table_test.rb
+++ b/actionpack/test/journey/gtg/transition_table_test.rb
@@ -95,7 +95,7 @@ module ActionDispatch
             parser  = Journey::Parser.new
             paths.map { |x|
               ast = parser.parse x
-              ast.each { |n| n.memo = ast}
+              ast.each { |n| n.memo = ast }
               ast
             }
           end
diff --git a/actionpack/test/journey/nfa/simulator_test.rb b/actionpack/test/journey/nfa/simulator_test.rb
index 918b971fec..183c892a53 100644
--- a/actionpack/test/journey/nfa/simulator_test.rb
+++ b/actionpack/test/journey/nfa/simulator_test.rb
@@ -49,7 +49,7 @@ module ActionDispatch
           parser  = Journey::Parser.new
           asts    = paths.map { |x|
             ast = parser.parse x
-            ast.each { |n| n.memo = ast}
+            ast.each { |n| n.memo = ast }
             ast
           }
 
diff --git a/actionpack/test/journey/path/pattern_test.rb b/actionpack/test/journey/path/pattern_test.rb
index 224ab689fc..d61a8c023a 100644
--- a/actionpack/test/journey/path/pattern_test.rb
+++ b/actionpack/test/journey/path/pattern_test.rb
@@ -194,7 +194,7 @@ module ActionDispatch
         end
 
         def test_to_regexp_with_strexp
-          path = Pattern.build("/:controller", { }, SEPARATORS, true)
+          path = Pattern.build("/:controller", {}, SEPARATORS, true)
           x = %r{\A/([^/.?]+)\Z}
 
           assert_equal(x.source, path.source)
diff --git a/actionpack/test/journey/route_test.rb b/actionpack/test/journey/route_test.rb
index 886cf857e8..b6414fd101 100644
--- a/actionpack/test/journey/route_test.rb
+++ b/actionpack/test/journey/route_test.rb
@@ -34,7 +34,7 @@ module ActionDispatch
 
       def test_ip_address
         path  = Path::Pattern.from_string "/messages/:id(.:format)"
-        route = Route.build("name", nil, path, {ip: "192.168.1.1"}, [],
+        route = Route.build("name", nil, path, { ip: "192.168.1.1" }, [],
                           controller: "foo", action: "bar")
         assert_equal "192.168.1.1", route.ip
       end
@@ -57,7 +57,7 @@ module ActionDispatch
 
       def test_connects_all_match
         path  = Path::Pattern.from_string "/:controller(/:action(/:id(.:format)))"
-        route = Route.build("name", nil, path, {action: "bar"}, [], controller: "foo")
+        route = Route.build("name", nil, path, { action: "bar" }, [], controller: "foo")
 
         assert_equal "/foo/bar/10", route.format(
           controller: "foo",
@@ -67,28 +67,28 @@ module ActionDispatch
 
       def test_extras_are_not_included_if_optional
         path  = Path::Pattern.from_string "/page/:id(/:action)"
-        route = Route.build("name", nil, path, { }, [], action: "show")
+        route = Route.build("name", nil, path, {}, [], action: "show")
 
         assert_equal "/page/10", route.format(id: 10)
       end
 
       def test_extras_are_not_included_if_optional_with_parameter
         path  = Path::Pattern.from_string "(/sections/:section)/pages/:id"
-        route = Route.build("name", nil, path, { }, [], action: "show")
+        route = Route.build("name", nil, path, {}, [], action: "show")
 
         assert_equal "/pages/10", route.format(id: 10)
       end
 
       def test_extras_are_not_included_if_optional_parameter_is_nil
         path  = Path::Pattern.from_string "(/sections/:section)/pages/:id"
-        route = Route.build("name", nil, path, { }, [], action: "show")
+        route = Route.build("name", nil, path, {}, [], action: "show")
 
         assert_equal "/pages/10", route.format(id: 10, section: nil)
       end
 
       def test_score
         constraints = {}
-        defaults = {controller: "pages", action: "show"}
+        defaults = { controller: "pages", action: "show" }
 
         path = Path::Pattern.from_string "/page/:id(/:action)(.:format)"
         specific = Route.build "name", nil, path, constraints, [:controller, :action], defaults
@@ -96,7 +96,7 @@ module ActionDispatch
         path = Path::Pattern.from_string "/:controller(/:action(/:id))(.:format)"
         generic = Route.build "name", nil, path, constraints, [], {}
 
-        knowledge = {id: 20, controller: "pages", action: "show"}
+        knowledge = { id: 20, controller: "pages", action: "show" }
 
         routes = [specific, generic]
 
diff --git a/actionpack/test/journey/router_test.rb b/actionpack/test/journey/router_test.rb
index c75475bffb..2b99637f56 100644
--- a/actionpack/test/journey/router_test.rb
+++ b/actionpack/test/journey/router_test.rb
@@ -58,31 +58,31 @@ module ActionDispatch
         get "/foo/:id", id: /\d/, anchor: false, to: "foo#bar"
 
         assert_raises(ActionController::UrlGenerationError) do
-          @formatter.generate(nil, { controller: "foo", action: "bar", id: "10" }, { })
+          @formatter.generate(nil, { controller: "foo", action: "bar", id: "10" }, {})
         end
       end
 
       def test_required_parts_are_verified_when_building
         get "/foo/:id", id: /\d+/, anchor: false, to: "foo#bar"
 
-        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar", id: "10" }, { })
+        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar", id: "10" }, {})
         assert_equal "/foo/10", path
 
         assert_raises(ActionController::UrlGenerationError) do
-          @formatter.generate(nil, { id: "aa" }, { })
+          @formatter.generate(nil, { id: "aa" }, {})
         end
       end
 
       def test_only_required_parts_are_verified
         get "/foo(/:id)", id: /\d/, to: "foo#bar"
 
-        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar", id: "10" }, { })
+        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar", id: "10" }, {})
         assert_equal "/foo/10", path
 
-        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar" }, { })
+        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar" }, {})
         assert_equal "/foo", path
 
-        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar", id: "aa" }, { })
+        path, _ = @formatter.generate(nil, { controller: "foo", action: "bar", id: "aa" }, {})
         assert_equal "/foo/aa", path
       end
 
@@ -91,7 +91,7 @@ module ActionDispatch
         get "/foo/:id", as: route_name, id: /\d+/, to: "foo#bar"
 
         error = assert_raises(ActionController::UrlGenerationError) do
-          @formatter.generate(route_name, { }, { })
+          @formatter.generate(route_name, {}, {})
         end
 
         assert_match(/missing required keys: \[:id\]/, error.message)
@@ -101,7 +101,7 @@ module ActionDispatch
         route_name = "gorby_thunderhorse"
 
         error = assert_raises(ActionController::UrlGenerationError) do
-          @formatter.generate(route_name, { }, { })
+          @formatter.generate(route_name, {}, {})
         end
 
         assert_no_match(/missing required keys: \[\]/, error.message)
@@ -130,12 +130,12 @@ module ActionDispatch
 
         env = rails_env "PATH_INFO" => "/foo/10"
         router.recognize(env) do |r, params|
-          assert_equal({id: "10", controller: "foo", action: "bar"}, params)
+          assert_equal({ id: "10", controller: "foo", action: "bar" }, params)
         end
 
         env = rails_env "PATH_INFO" => "/foo"
         router.recognize(env) do |r, params|
-          assert_equal({id: nil, controller: "foo", action: "bar"}, params)
+          assert_equal({ id: nil, controller: "foo", action: "bar" }, params)
         end
       end
 
@@ -242,9 +242,9 @@ module ActionDispatch
         get "/:controller(/:action)", to: "foo#bar"
 
         path, params = @formatter.generate(
-          nil, {id: 1, controller: "tasks", action: "show"}, {})
+          nil, { id: 1, controller: "tasks", action: "show" }, {})
         assert_equal "/tasks/show", path
-        assert_equal({id: 1}, params)
+        assert_equal({ id: 1 }, params)
       end
 
       def test_generate_escapes
@@ -277,7 +277,7 @@ module ActionDispatch
                  relative_url_root: nil
         }, {})
         assert_equal "/tasks/show", path
-        assert_equal({id: 1, relative_url_root: nil}, params)
+        assert_equal({ id: 1, relative_url_root: nil }, params)
       end
 
       def test_generate_missing_keys_no_matches_different_format_keys
@@ -297,7 +297,7 @@ module ActionDispatch
         }
         request_parameters = primarty_parameters.merge(redirection_parameters).merge(missing_parameters)
 
-        message = "No route matches #{Hash[request_parameters.sort_by{|k,v|k.to_s}].inspect} missing required keys: #{[missing_key.to_sym].inspect}"
+        message = "No route matches #{Hash[request_parameters.sort_by { |k,v|k.to_s }].inspect} missing required keys: #{[missing_key.to_sym].inspect}"
 
         error = assert_raises(ActionController::UrlGenerationError) do
           @formatter.generate(
@@ -311,7 +311,7 @@ module ActionDispatch
 
         path, params = @formatter.generate(
           nil,
-          {controller: "tasks", id: 10},
+          { controller: "tasks", id: 10 },
           action: "index")
         assert_equal "/tasks/index/10", path
         assert_equal({}, params)
@@ -322,7 +322,7 @@ module ActionDispatch
 
         path, params = @formatter.generate(
           "tasks",
-          {controller: "tasks"},
+          { controller: "tasks" },
           controller: "tasks", action: "index")
         assert_equal "/tasks", path
         assert_equal({}, params)