Author: britter
Date: Fri Jul 26 21:39:22 2013
New Revision: 1507480

URL: http://svn.apache.org/r1507480
Log:
Test exception case in separate test method

Modified:
    
commons/proper/email/trunk/src/test/java/org/apache/commons/mail/EmailTest.java

Modified: 
commons/proper/email/trunk/src/test/java/org/apache/commons/mail/EmailTest.java
URL: 
http://svn.apache.org/viewvc/commons/proper/email/trunk/src/test/java/org/apache/commons/mail/EmailTest.java?rev=1507480&r1=1507479&r2=1507480&view=diff
==============================================================================
--- 
commons/proper/email/trunk/src/test/java/org/apache/commons/mail/EmailTest.java 
(original)
+++ 
commons/proper/email/trunk/src/test/java/org/apache/commons/mail/EmailTest.java 
Fri Jul 26 21:39:22 2013
@@ -442,24 +442,12 @@ public class EmailTest extends AbstractE
         // retrieve and verify
         assertEquals(arrExpected.size(), this.email.getToAddresses().size());
         assertEquals(arrExpected.toString(), 
this.email.getToAddresses().toString());
+    }
 
-        // ====================================================================
-        // Test Exceptions
-        // ====================================================================
-        // reset the mail class
-        MockEmailConcrete anotherEmail = new MockEmailConcrete();
-
-        // bad encoding
-        try
-        {
-            // set a dodgy encoding scheme
-            anotherEmail.addTo("m...@home.com", "m...@home.com", 
"bad.encoding\uc5ec\n");
-            fail("setting invalid charset should have failed!");
-        }
-        catch (IllegalCharsetNameException  e)
-        {
-            // expected runtime exception.
-        }
+    @Test(expected = IllegalCharsetNameException.class)
+    public void testAddToBadEncoding() throws Exception
+    {
+        this.email.addTo("m...@home.com", "m...@home.com", 
"bad.encoding\uc5ec\n");
     }
 
     @Test


Reply via email to