svn commit: r437274 - in /james/jsieve/trunk/src: main/java/org/apache/jsieve/samples/james/Actions.java test/java/org/apache/jsieve/samples/james/junit/ActionDispatcherTest.java

[bago]

Author: bago
Date: Sat Aug 26 19:02:57 2006
New Revision: 437274

URL: http://svn.apache.org/viewvc?rev=437274&view=rev
Log:
Fix failing test (it was not ran from previous tests)

Modified:
    
james/jsieve/trunk/src/main/java/org/apache/jsieve/samples/james/Actions.java
    
james/jsieve/trunk/src/test/java/org/apache/jsieve/samples/james/junit/ActionDispatcherTest.java

Modified: 
james/jsieve/trunk/src/main/java/org/apache/jsieve/samples/james/Actions.java
URL: 
http://svn.apache.org/viewvc/james/jsieve/trunk/src/main/java/org/apache/jsieve/samples/james/Actions.java?rev=437274&r1=437273&r2=437274&view=diff
==============================================================================
--- 
james/jsieve/trunk/src/main/java/org/apache/jsieve/samples/james/Actions.java 
(original)
+++ 
james/jsieve/trunk/src/main/java/org/apache/jsieve/samples/james/Actions.java 
Sat Aug 26 19:02:57 2006
@@ -298,7 +298,10 @@
     protected static MailAddress getSoleRecipient(Mail aMail)
             throws MessagingException
     {
-        if (1 != aMail.getRecipients().size())
+         if (aMail.getRecipients() == null) {
+          throw new MessagingException("Invalid number of recipients - 0"
+              + ". Exactly 1 recipient is expected.");
+         } else if (1 != aMail.getRecipients().size())
             throw new MessagingException("Invalid number of recipients - "
                     + new Integer(aMail.getRecipients().size()).toString()
                     + ". Exactly 1 recipient is expected.");

Modified: 
james/jsieve/trunk/src/test/java/org/apache/jsieve/samples/james/junit/ActionDispatcherTest.java
URL: 
http://svn.apache.org/viewvc/james/jsieve/trunk/src/test/java/org/apache/jsieve/samples/james/junit/ActionDispatcherTest.java?rev=437274&r1=437273&r2=437274&view=diff
==============================================================================
--- 
james/jsieve/trunk/src/test/java/org/apache/jsieve/samples/james/junit/ActionDispatcherTest.java
 (original)
+++ 
james/jsieve/trunk/src/test/java/org/apache/jsieve/samples/james/junit/ActionDispatcherTest.java
 Sat Aug 26 19:02:57 2006
@@ -20,8 +20,13 @@
 package org.apache.jsieve.samples.james.junit;
 
 import java.lang.reflect.InvocationTargetException;
+import java.util.Arrays;
+import java.util.Properties;
 
 import javax.mail.MessagingException;
+import javax.mail.Session;
+import javax.mail.internet.MimeMessage;
+import javax.mail.internet.ParseException;
 
 import junit.framework.TestCase;
 
@@ -32,6 +37,7 @@
 import org.apache.jsieve.samples.james.junit.utils.ActionAbsent;
 import org.apache.jsieve.samples.james.junit.utils.MockMailetContext;
 import org.apache.mailet.Mail;
+import org.apache.mailet.MailAddress;
 import org.apache.mailet.MailetContext;
 
 /**
@@ -74,11 +80,16 @@
 
     /**
      * Test execute of ActionKeep
+     * @throws MessagingException 
      */
-    public void testExecuteActionKeep()
+    public void testExecuteActionKeep() throws MessagingException
     {
         boolean isTestPassed = false;
         Mail aMail = new MailImpl();
+        aMail.setRecipients(Arrays.asList(new MailAddress[] { new 
MailAddress("a","a.com")}));
+        MimeMessage mimeMessage = new 
MimeMessage(Session.getDefaultInstance(new Properties()));
+        mimeMessage.setText("TEST");
+                               aMail.setMessage(mimeMessage);
         MailetContext aMailetContext = new MockMailetContext();
         Action action = new ActionKeep();
         try



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]