* test/ruby/test_thread.rb: fix typos.

* test/ruby/envutil.rb (rubyexec): move Open3.popen3 call into timeout
  block.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@16182 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

ChangeLog

@@ -1,3 +1,10 @@
+Thu Apr 24 23:00:58 2008  Yusuke Endoh  <mame@tsg.ne.jp>
+
+	* test/ruby/test_thread.rb: fix typos.
+
+	* test/ruby/envutil.rb (rubyexec): move Open3.popen3 call into timeout
+	  block.
+
 Thu Apr 24 22:34:52 2008  Yusuke Endoh  <mame@tsg.ne.jp>
 
 	* test/ruby/test_comparable.rb: new tests for Comparable, to achieve

test/ruby/envutil.rb

@@ -43,16 +43,17 @@ module EnvUtil
     c = "C"
     env = {}
     LANG_ENVS.each {|lc| env[lc], ENV[lc] = ENV[lc], c}
-    stdin, stdout, stderr = Open3.popen3(*([ruby] + args))
-    env.each_pair {|lc, v|
-      if v
-        ENV[lc] = v
-      else
-        ENV.delete(lc)
-      end
-    }
-    env = nil
+    stdin = stdout = stderr = nil
     Timeout.timeout(10) do
+      stdin, stdout, stderr = Open3.popen3(*([ruby] + args))
+      env.each_pair {|lc, v|
+        if v
+          ENV[lc] = v
+        else
+          ENV.delete(lc)
+        end
+      }
+      env = nil
       yield(stdin, stdout, stderr)
     end
 

test/ruby/test_thread.rb

@@ -109,12 +109,12 @@ class TestThread < Test::Unit::TestCase
     t1.priority = -1
     t2 = Thread.new { loop { c2 += 1 } }
     t2.priority = -3
-    assert(-1, t1.priority)
-    assert(-3, t2.priority)
+    assert_equal(-1, t1.priority)
+    assert_equal(-3, t2.priority)
     sleep 0.5
     t1.kill
     t2.kill
-    #assert(c1 > c2 * 2, "[ruby-dev:33124]")
+    assert(c1 > c2 * 2, "[ruby-dev:33124]")
   end
 
   def test_new
@@ -384,7 +384,7 @@ class TestThread < Test::Unit::TestCase
   def test_select_wait
     assert_nil(IO.select(nil, nil, nil, 1))
     t = Thread.new do
-      assert_nil(IO.select(nil, nil, nil, nil))
+      IO.select(nil, nil, nil, nil)
     end
     sleep 0.5
     t.kill