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