remove redundant curlies from hash arguments

Author: fxn

actioncable/test/channel/base_test.rb

@@ -74,7 +74,7 @@ def rm_rf
   setup do
     @user = User.new "lifo"
     @connection = TestConnection.new(@user)
-    @channel = ChatChannel.new @connection, "{id: 1}", { id: 1 }
+    @channel = ChatChannel.new @connection, "{id: 1}", id: 1
   end
 
   test "should subscribe to a channel on initialize" do

actioncable/test/channel/periodic_timers_test.rb

@@ -60,7 +60,7 @@ def ping
 
   test "timer start and stop" do
     @connection.server.event_loop.expects(:timer).times(3).returns(stub(shutdown: nil))
-    channel = ChatChannel.new @connection, "{id: 1}", { id: 1 }
+    channel = ChatChannel.new @connection, "{id: 1}", id: 1
 
     channel.unsubscribe_from_channel
     assert_equal [], channel.send(:active_periodic_timers)

actioncable/test/channel/rejection_test.rb

@@ -16,7 +16,7 @@ def subscribed
 
   test "subscription rejection" do
     @connection.expects(:subscriptions).returns mock().tap { |m| m.expects(:remove_subscription).with instance_of(SecretChannel) }
-    @channel = SecretChannel.new @connection, "{id: 1}", { id: 1 }
+    @channel = SecretChannel.new @connection, "{id: 1}", id: 1
 
     expected = { "identifier" => "{id: 1}", "type" => "reject_subscription" }
     assert_equal expected, @connection.last_transmission

actioncable/test/channel/stream_test.rb

@@ -52,7 +52,7 @@ class StreamTest < ActionCable::TestCase
       run_in_eventmachine do
         connection = TestConnection.new
         connection.expects(:pubsub).returns mock().tap { |m| m.expects(:subscribe).with("test_room_1", kind_of(Proc), kind_of(Proc)).returns stub_everything(:pubsub) }
-        channel = ChatChannel.new connection, "{id: 1}", { id: 1 }
+        channel = ChatChannel.new connection, "{id: 1}", id: 1
 
         connection.expects(:pubsub).returns mock().tap { |m| m.expects(:unsubscribe) }
         channel.unsubscribe_from_channel
@@ -84,7 +84,7 @@ class StreamTest < ActionCable::TestCase
       run_in_eventmachine do
         connection = TestConnection.new
 
-        ChatChannel.new connection, "{id: 1}", { id: 1 }
+        ChatChannel.new connection, "{id: 1}", id: 1
         assert_nil connection.last_transmission
 
         wait_for_async

actioncable/test/connection/base_test.rb

@@ -119,8 +119,8 @@ def call(*)
 
       env = Rack::MockRequest.env_for(
         "/test",
-        { "HTTP_CONNECTION" => "upgrade", "HTTP_UPGRADE" => "websocket",
-          "HTTP_HOST" => "localhost", "HTTP_ORIGIN" => "http://rubyonrails.org", "rack.hijack" => CallMeMaybe.new }
+        "HTTP_CONNECTION" => "upgrade", "HTTP_UPGRADE" => "websocket",
+          "HTTP_HOST" => "localhost", "HTTP_ORIGIN" => "http://rubyonrails.org", "rack.hijack" => CallMeMaybe.new
       )
 
       connection = ActionCable::Connection::Base.new(@server, env)

actionpack/lib/action_dispatch/journey/route.rb

@@ -32,7 +32,7 @@ def self.call(_); true; end
           def self.verb; ""; end
         end
 
-        VERB_TO_CLASS = VERBS.each_with_object({ all: All }) do |verb, hash|
+        VERB_TO_CLASS = VERBS.each_with_object(all: All) do |verb, hash|
           klass = const_get verb
           hash[verb]                 = klass
           hash[verb.downcase]        = klass

actionpack/lib/action_dispatch/routing/mapper.rb

@@ -2122,7 +2122,7 @@ def frame; @hash; end
 
       def initialize(set) #:nodoc:
         @set = set
-        @scope = Scope.new({ path_names: @set.resources_path_names })
+        @scope = Scope.new(path_names: @set.resources_path_names)
         @concerns = {}
       end
 

actionpack/lib/action_dispatch/routing/route_set.rb

@@ -731,7 +731,7 @@ def recognize_path(path, environment = {})
         extras = environment[:extras] || {}
 
         begin
-          env = Rack::MockRequest.env_for(path, {method: method})
+          env = Rack::MockRequest.env_for(path, method: method)
         rescue URI::InvalidURIError => e
           raise ActionController::RoutingError, e.message
         end

actionpack/lib/action_dispatch/testing/assertions/routing.rb

@@ -202,7 +202,7 @@ def recognized_request_for(path, extras = {}, msg)
           request.request_method = method if method
 
           params = fail_on(ActionController::RoutingError, msg) do
-            @routes.recognize_path(path, { method: method, extras: extras })
+            @routes.recognize_path(path, method: method, extras: extras)
           end
           request.path_parameters = params.with_indifferent_access
 

actionpack/lib/action_dispatch/testing/integration.rb

@@ -298,7 +298,7 @@ def process_with_kwargs(http_method, path, *args)
             process(http_method, path, *args)
           else
             non_kwarg_request_warning if args.any?
-            process(http_method, path, { params: args[0], headers: args[1] })
+            process(http_method, path, params: args[0], headers: args[1])
           end
         end