From f34ce620a79db7f21131a912fb6f456ded2d218f Mon Sep 17 00:00:00 2001
From: nahi <nahi@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Fri, 9 Jan 2004 04:26:05 +0000
Subject:         * lib/test/unit/collector/dir.rb: do not ignore
 exceptions(LoadError           and SystemExitError) while loading a testcase.
  smell of bug.

        * test/testunit/collector/test_dir.rb: add new test of the LoadError.

        * test/drb/{test_drbssl.rb,test_drbunix.rb}: do not define testcase if
          openssl is not installed.

        * test/testunit/collector/test_dir.rb: assert_raises -> assert_raise.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@5423 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 test/drb/test_drbssl.rb             | 12 +++++++++++-
 test/drb/test_drbunix.rb            | 12 +++++++++++-
 test/testunit/collector/test_dir.rb | 29 +++++++++++++++++------------
 3 files changed, 39 insertions(+), 14 deletions(-)

(limited to 'test')

diff --git a/test/drb/test_drbssl.rb b/test/drb/test_drbssl.rb
index 0ca177fe58..1e7e49a72c 100644
--- a/test/drb/test_drbssl.rb
+++ b/test/drb/test_drbssl.rb
@@ -1,6 +1,13 @@
 $:.unshift(File.dirname(File.expand_path(__FILE__)))
 require 'drbtest'
-require 'drb/ssl'
+
+begin
+  require 'drb/ssl'
+rescue LoadError
+end
+
+if Object.const_defined?("OpenSSL")
+
 
 class DRbSSLService < DRbService
   %w(ut_drb_drbssl.rb ut_array_drbssl.rb).each do |nm|
@@ -63,3 +70,6 @@ class TestDRbSSLAry < Test::Unit::TestCase
     @there = @ext.front
   end
 end
+
+
+end
diff --git a/test/drb/test_drbunix.rb b/test/drb/test_drbunix.rb
index e05637a2b7..88f47ddbb7 100644
--- a/test/drb/test_drbunix.rb
+++ b/test/drb/test_drbunix.rb
@@ -1,6 +1,13 @@
 $:.unshift(File.dirname(File.expand_path(__FILE__)))
 require 'drbtest'
-require 'drb/unix'
+
+begin
+  require 'drb/unix'
+rescue LoadError
+end
+
+if Object.const_defined?("UNIXServer")
+
 
 class DRbUNIXService < DRbService
   %w(ut_drb_drbunix.rb ut_array_drbunix.rb).each do |nm|
@@ -46,3 +53,6 @@ class TestDRbUNIXAry < Test::Unit::TestCase
     @there = @ext.front
   end
 end
+
+
+end
diff --git a/test/testunit/collector/test_dir.rb b/test/testunit/collector/test_dir.rb
index e9b42d7966..64bcabaea6 100644
--- a/test/testunit/collector/test_dir.rb
+++ b/test/testunit/collector/test_dir.rb
@@ -148,7 +148,7 @@ module Test
             return false if(@required.include?(file))
             begin
               e = find(file)
-            rescue Errno::ENOENT => e
+            rescue Errno::ENOENT => ex
               if(/\.rb\Z/ =~ file)
                 raise LoadError, file
               end
@@ -213,10 +213,10 @@ module Test
             end
           end
           assert_equal('/', fs.pwd)
-          assert_raises(Errno::ENOENT) do
+          assert_raise(Errno::ENOENT) do
             fs.chdir('bogus')
           end
-          assert_raises(Errno::ENOTDIR) do
+          assert_raise(Errno::ENOTDIR) do
             fs.chdir('a')
           end
           fs.chdir('b')
@@ -248,10 +248,10 @@ module Test
           assert_equal(['.', '..', 'a', 'b', 'e', 'f'], fs.entries('.').sort)
           assert_equal(['.', '..', 'a', 'b', 'e', 'f'], fs.entries('b/..').sort)
           assert_equal(['.', '..', 'c', 'd'], fs.entries('b').sort)
-          assert_raises(Errno::ENOENT) do
+          assert_raise(Errno::ENOENT) do
             fs.entries('z')
           end
-          assert_raises(Errno::ENOTDIR) do
+          assert_raise(Errno::ENOTDIR) do
             fs.entries('a')
           end
           fs.chdir('f')
@@ -275,7 +275,7 @@ module Test
           end
           assert_equal([], c)
 
-          assert_raises(LoadError) do
+          assert_raise(LoadError) do
             fs.require('bogus')
           end
           
@@ -353,13 +353,18 @@ module Test
         end
 
         def test_collect_file
-          expected = TestSuite.new('test_1.rb')
-          expected << @t1.suite
-          assert_equal(expected, @c.collect('test_1.rb'))
+          expected = [@t1.suite]
+          subsuites = []
+	  @c.collect_file('test_1.rb', subsuites, @c.find_test_cases)
+          assert_equal(expected, subsuites)
           
-          expected = TestSuite.new('t4.rb')
-          expected << @t4.suite
-          assert_equal(expected, @c.collect('t4.rb'))
+          expected = [@t4.suite]
+	  subsuites = []
+          @c.collect_file('t4.rb', subsuites, @c.find_test_cases)
+          assert_equal(expected, subsuites)
+	  assert_raise(LoadError) do
+	    @c.collect_file('tloaderr.rb', [], [])
+	  end
         end
 
         def test_nil_pattern
-- 
cgit v1.2.3