[#41278] [BUG:1.9] BINARY should not be ASCII-compatible — Yugui <yugui@...>

WXVndWkbJEIkRyQ5ISMbKEIKCgo+IBskQiRHISIkKiQqJGAkTSQzJDMkXiRHJE41RE9AJEclKyVQ

15 messages 2010/05/11

[#41407] [Bug #3339] win32ole test failure — Usaku NAKAMURA <redmine@...>

Bug #3339: win32ole test failure

20 messages 2010/05/25
[#41411] Re: [Bug #3339] win32ole test failure — Masaki Suketa <masaki.suketa@...> 2010/05/25

助田です。

[#41412] Re: [Bug #3339] win32ole test failure — "U.Nakamura" <usa@...> 2010/05/25

こんにちは、なかむら(う)です。

[ruby-dev:41484] Re: trunk/1.9.2 で test_io.rb が test_print_separators で固まる

From: Tomoyuki Chikanaga <chikanag@...>
Date: 2010-05-31 11:35:17 UTC
List: ruby-dev #41484
近永と申します。

その後いくつかのマシン上で追試してみて

2.6.9-89 まで → 固まる
2.6.18-164 以降 → 再現しない

と Kernel Version が新しいと発生しなくなるようでしたので
環境の問題だろうと考えて深追いはしないことにしました。

それはいいのですが、test_io.rb の $SAFE = 4 のテストが Timeout.timeout 内の
Thread#kill で例外が発生するために本来確認したい SecurityError の発生を
テストできていない問題がありますので、以下のパッチをご検討ください。

Index: test/ruby/test_io.rb
===================================================================
--- test/ruby/test_io.rb        (revision 28108)
+++ test/ruby/test_io.rb        (working copy)
@@ -740,12 +740,12 @@
   end

   def safe_4
-    Thread.new do
-      Timeout.timeout(10) do
-        $SAFE = 4
-        yield
-      end
-    end.join
+    unless (Thread.new do
+      $SAFE = 4
+      yield
+    end.join(10))
+      flunk("timeout in safe_4")
+    end
   end

   def pipe(wp, rp)
@@ -1461,15 +1461,16 @@
   def test_print_separators
     $, = ':'
     $\ = "\n"
-    r, w = IO.pipe
-    w.print('a')
-    w.print('a','b','c')
-    w.close
-    assert_equal("a\n", r.gets)
-    assert_equal("a:b:c\n", r.gets)
-    assert_nil r.gets
-    r.close
-
+    pipe(proc do |w|
+      w.print('a')
+      w.print('a','b','c')
+      w.close
+    end, proc do |r|
+      assert_equal("a\n", r.gets)
+      assert_equal("a:b:c\n", r.gets)
+      assert_nil r.gets
+      r.close
+    end)
   ensure
     $, = nil
     $\ = nil


Chikanaga Tomoyuki
Nippon Control System Corp.


In This Thread

Prev Next