* test/io/nonblock/test_flush.rb: don't set Thread.abort_on_exception.

* test/net/imap/test_imap.rb: ensure disconnecting imap to terminate
  receiver thread..


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

ChangeLog

@@ -1,3 +1,10 @@
+Tue May  6 02:08:18 2008  Tanaka Akira  <akr@fsij.org>
+
+	* test/io/nonblock/test_flush.rb: don't set Thread.abort_on_exception.
+
+	* test/net/imap/test_imap.rb: ensure disconnecting imap to terminate
+	  receiver thread..
+
 Tue May  6 00:29:21 2008  Yukihiro Matsumoto  <matz@ruby-lang.org>
 
 	* iseq.c (insn_operand_intern): should handle Qundef embedded in

test/io/nonblock/test_flush.rb

@@ -5,7 +5,6 @@ begin
 rescue LoadError
 end
 
-Thread.abort_on_exception = true
 class TestIONonblock < Test::Unit::TestCase
   def test_flush
     r,w = IO.pipe

test/net/imap/test_imap.rb

@@ -67,6 +67,7 @@ class IMAPTest < Test::Unit::TestCase
   end
 
   def test_starttls
+    imap = nil
     if defined?(OpenSSL)
       starttls_test do |port|
         imap = Net::IMAP.new("localhost", :port => port)
@@ -74,6 +75,10 @@ class IMAPTest < Test::Unit::TestCase
         imap
       end
     end
+  ensure
+    if imap && !imap.disconnected?
+      imap.disconnect
+    end
   end
 
   private
@@ -105,9 +110,12 @@ class IMAPTest < Test::Unit::TestCase
       end
     end
     begin
-      imap = yield(port)
+      begin
-      imap.logout
+        imap = yield(port)
-      imap.disconnect
+        imap.logout
+      ensure
+        imap.disconnect if imap
+      end
     ensure
       ssl_server.close
     end
@@ -143,9 +151,12 @@ class IMAPTest < Test::Unit::TestCase
       end
     end
     begin
-      imap = yield(port)
+      begin
-      imap.logout
+        imap = yield(port)
-      imap.disconnect
+        imap.logout
+      ensure
+        imap.disconnect if imap
+      end
     ensure
       server.close
     end