From: Eric Wong <normalperson@yhbt.net>
Date: Wed, 31 Oct 2012 19:20:53 +0000 (+0000)
Subject: test: fix some warnings found with "ruby -w -c"
X-Git-Tag: v3.5.0-rc1~2
X-Git-Url: https://repo.or.cz/w/ruby-mogilefs-client.git/commitdiff_plain/3e0b092efc2c5a5383a5d2063b830fe5d5e652f9

test: fix some warnings found with "ruby -w -c"

Less noise should be better.
---

diff --git a/test/socket_test.rb b/test/socket_test.rb
index 7818506..d0aff32 100644
--- a/test/socket_test.rb
+++ b/test/socket_test.rb
@@ -95,7 +95,7 @@ module SocketTest
   def test_read_in_full
     sock = MogileFS::Socket.tcp(@host, @port)
     accepted = @srv.accept
-    thr = Thread.new do
+    Thread.new do
       accepted.write "HELLO"
       sleep 0.1
       accepted.write " "
diff --git a/test/test_fresh.rb b/test/test_fresh.rb
index a87ebd5..7585ab0 100644
--- a/test/test_fresh.rb
+++ b/test/test_fresh.rb
@@ -44,7 +44,7 @@ class TestMogFresh < Test::Unit::TestCase
       opts[:content_md5] = [ Digest::MD5.digest(content) ].pack('m').rstrip
       opts[:class] = "check"
     end
-    rv = @client.new_file(key, opts) do |http_file|
+    @client.new_file(key, opts) do |http_file|
       http_file << content
     end
 
@@ -106,7 +106,7 @@ class TestMogFresh < Test::Unit::TestCase
     line = buf.split(/\r?\n/).grep(/\screate_open\s/)[0]
     assert_equal 0, buf.split(/\r?\n/).grep(/\screate_close\s/).size
     assert_equal 0, buf.split(/\r?\n/).grep(/\sfarewell\s/).size
-    assert_match /\bhello=world\b/, line
+    assert_match(/\bhello=world\b/, line)
     assert_equal 1, io.write('.')
     assert_nil io.close
 
@@ -114,6 +114,6 @@ class TestMogFresh < Test::Unit::TestCase
     buf = ""
     buf << socket.readpartial(666) until buf =~ /\.\r?\n\z/
     line = buf.split(/\r?\n/).grep(/\screate_close\s/)[0]
-    assert_match /\bfarewell=goodnight\b/, line
+    assert_match(/\bfarewell=goodnight\b/, line)
   end
 end