You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by br...@apache.org on 2013/07/26 22:52:42 UTC
svn commit: r1507432 -
/commons/proper/email/trunk/src/test/java/org/apache/commons/mail/EmailTest.java
Author: britter
Date: Fri Jul 26 20:52:42 2013
New Revision: 1507432
URL: http://svn.apache.org/r1507432
Log:
Split up tests into separate methods
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=1507432&r1=1507431&r2=1507432&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 20:52:42 2013
@@ -1190,12 +1190,8 @@ public class EmailTest extends AbstractE
}
@Test
- public void testSendEx()
+ public void testSendNoHostName()
{
- // ====================================================================
- // Test Exceptions (in getMailSession)
- // ====================================================================
- // hostname not set
try
{
this.getMailServer();
@@ -1209,8 +1205,11 @@ public class EmailTest extends AbstractE
this.fakeMailServer.stop();
assertTrue(true);
}
+ }
- // bad hostname
+ @Test
+ public void testSendBadHostName()
+ {
try
{
this.getMailServer();
@@ -1237,11 +1236,11 @@ public class EmailTest extends AbstractE
this.fakeMailServer.stop();
assertTrue(true);
}
+ }
- // ====================================================================
- // Test Exceptions (in send)
- // ====================================================================
- // from add not set
+ @Test
+ public void testSendFromNotSet()
+ {
try
{
this.getMailServer();
@@ -1258,8 +1257,11 @@ public class EmailTest extends AbstractE
this.fakeMailServer.stop();
assertTrue(true);
}
+ }
- // destination (to/cc/bcc) dd not set
+ @Test
+ public void testSendDestinationNotSet()
+ {
try
{
this.getMailServer();
@@ -1276,8 +1278,11 @@ public class EmailTest extends AbstractE
this.fakeMailServer.stop();
assertTrue(true);
}
+ }
- // bad auth set
+ @Test
+ public void testSendBadAuthSet()
+ {
try
{
this.getMailServer();
@@ -1296,9 +1301,11 @@ public class EmailTest extends AbstractE
this.fakeMailServer.stop();
assertTrue(true);
}
-
- // validate that the correct smtp port is visible in the exception message
- // in case ssl connection is used
+ }
+
+ @Test
+ public void testSendCorrectSmtpPortContainedInException()
+ {
try
{
this.getMailServer();
@@ -1318,7 +1325,6 @@ public class EmailTest extends AbstractE
this.fakeMailServer.stop();
assertTrue(true);
}
-
}
@Test