Merge pull request #18 from eugeneius/fix_warnings

Fix warnings

Author: mfazekas

lib/net/ssh/multi/pending_connection.rb

@@ -43,7 +43,7 @@ def remote(port, host, remote_port, remote_host="127.0.0.1")
 
       def replay_on(session)
         forward = session.forward
-        @recordings.each {|args| forward.send *args}
+        @recordings.each { |args| forward.send(*args) }
       end
     end
 

lib/net/ssh/multi/server.rb

@@ -56,6 +56,7 @@ def initialize(master, host, options={})
 
       @gateway = @options.delete(:via)
       @failed = false
+      @session = @ready_session = nil
     end
 
     # Returns the value of the server property with the given +key+. Server

lib/net/ssh/multi/session_actions.rb

@@ -118,8 +118,8 @@ def open_channel(type="session", *extra, &on_confirm)
     #   end
     def exec(command, &block)
       open_channel do |channel|
-        channel.exec(command) do |ch, success|
-          raise "could not execute command: #{command.inspect} (#{ch[:host]})" unless success
+        channel.exec(command) do |c, success|
+          raise "could not execute command: #{command.inspect} (#{c[:host]})" unless success
 
           channel.on_data do |ch, data|
             if block

test/channel_test.rb

@@ -3,7 +3,7 @@
 
 class ChannelTest < Minitest::Test
   def test_each_should_iterate_over_each_component_channel
-    channels = [c1 = mock('channel'), c2 = mock('channel'), c3 = mock('channel')]
+    channels = [mock('channel'), mock('channel'), mock('channel')]
     channel = Net::SSH::Multi::Channel.new(mock('session'), channels)
     result = []
     channel.each { |c| result << c }

test/session_actions_test.rb

@@ -94,7 +94,7 @@ def test_exec_with_block_should_pass_data_and_extended_data_to_block
     c.expects(:on_extended_data).yields(c, 1, "stderr")
     c.expects(:on_request)
     results = {}
-    @session.exec("something") do |c, stream, data|
+    @session.exec("something") do |_, stream, data|
       results[stream] = data
     end
     assert_equal({:stdout => "stdout", :stderr => "stderr"}, results)