From 2c90568a066c3064b054827ac7239a57c12a5841 Mon Sep 17 00:00:00 2001
From: Mitesh Jain <mjain@crri.co.in>
Date: Sat, 21 May 2011 17:57:48 +0530
Subject: [PATCH] Duplicable test is looking fine now.

---
 .../test/core_ext/duplicable_test.rb          | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/activesupport/test/core_ext/duplicable_test.rb b/activesupport/test/core_ext/duplicable_test.rb
index 24e0ccd9b3646..e48e6a7c45565 100644
--- a/activesupport/test/core_ext/duplicable_test.rb
+++ b/activesupport/test/core_ext/duplicable_test.rb
@@ -4,22 +4,27 @@
 require 'active_support/core_ext/numeric/time'
 
 class DuplicableTest < Test::Unit::TestCase
-  NO  = [nil, false, true, :symbol, 1, 2.3, BigDecimal.new('4.56'), Class.new, Module.new, 5.seconds]
+  RAISE_DUP  = [nil, false, true, :symbol, 1, 2.3, BigDecimal.new('4.56'), 5.seconds]
   YES = ['1', Object.new, /foo/, [], {}, Time.now]
+  NO = [Class.new, Module.new]
 
   def test_duplicable
-    NO.each do |v|
+    (RAISE_DUP + NO).each do |v|
       assert !v.duplicable?
-      begin
-        v.dup
-        fail
-      rescue Exception
-      end
     end
 
     YES.each do |v|
       assert v.duplicable?
+    end
+
+    (YES + NO).each do |v|
       assert_nothing_raised { v.dup }
     end
+
+    RAISE_DUP.each do |v|
+      assert_raises(TypeError) do
+        v.dup
+      end
+    end
   end
 end