You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tn...@apache.org on 2014/06/30 12:57:49 UTC

svn commit: r1606679 - /commons/proper/email/trunk/src/test/java/org/apache/commons/mail/SimpleEmailTest.java

Author: tn
Date: Mon Jun 30 10:57:48 2014
New Revision: 1606679

URL: http://svn.apache.org/r1606679
Log:
Disable test as it is dependent on order of execution.

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

Modified: commons/proper/email/trunk/src/test/java/org/apache/commons/mail/SimpleEmailTest.java
URL: http://svn.apache.org/viewvc/commons/proper/email/trunk/src/test/java/org/apache/commons/mail/SimpleEmailTest.java?rev=1606679&r1=1606678&r2=1606679&view=diff
==============================================================================
--- commons/proper/email/trunk/src/test/java/org/apache/commons/mail/SimpleEmailTest.java (original)
+++ commons/proper/email/trunk/src/test/java/org/apache/commons/mail/SimpleEmailTest.java Mon Jun 30 10:57:48 2014
@@ -24,6 +24,7 @@ import java.io.IOException;
 
 import org.apache.commons.mail.mocks.MockSimpleEmail;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 
 /**
@@ -121,8 +122,15 @@ public class SimpleEmailTest extends Abs
     }
 
     @Test
+    @Ignore
     public void testDefaultMimeCharset() throws EmailException, IOException
     {
+        /*
+         * disabling this test as it is dependent on execution order.
+         * MimeUtility.getDefaultMIMECharset does some internal caching and if
+         * mail.mime.charset is not defined, reverts to the default java charset
+         * which is basically the system default file encoding.
+         */
         System.setProperty(EmailConstants.MAIL_MIME_CHARSET, "utf-8");
 
         // ====================================================================
@@ -156,7 +164,7 @@ public class SimpleEmailTest extends Abs
         validateSend(
                 this.fakeMailServer,
                 strSubject,
-                this.email.getMsg(),
+                this.email.getMsg().substring(0, 13), // only check the start, the ä will be encoded
                 this.email.getFromAddress(),
                 this.email.getToAddresses(),
                 this.email.getCcAddresses(),