You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by bt...@apache.org on 2021/03/08 05:44:34 UTC

[james-project] 24/30: JAMES-3508 IMAPServerTest: Ensure Appended messages can be read

This is an automated email from the ASF dual-hosted git repository.

btellier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit 642531778d8c4b2088bde5450c4647ec7b5d7c8e
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Wed Mar 3 15:59:30 2021 +0700

    JAMES-3508 IMAPServerTest: Ensure Appended messages can be read
---
 .../james/imapserver/netty/IMAPServerTest.java     | 36 +++++++++++++++++-----
 1 file changed, 29 insertions(+), 7 deletions(-)

diff --git a/server/protocols/protocols-imap4/src/test/java/org/apache/james/imapserver/netty/IMAPServerTest.java b/server/protocols/protocols-imap4/src/test/java/org/apache/james/imapserver/netty/IMAPServerTest.java
index 9ef933c..8e30c08 100644
--- a/server/protocols/protocols-imap4/src/test/java/org/apache/james/imapserver/netty/IMAPServerTest.java
+++ b/server/protocols/protocols-imap4/src/test/java/org/apache/james/imapserver/netty/IMAPServerTest.java
@@ -19,6 +19,7 @@
 
 package org.apache.james.imapserver.netty;
 
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
@@ -51,6 +52,7 @@ class IMAPServerTest {
     private static final String _65K_MESSAGE = "header: value\r\n" + "012345678\r\n".repeat(6553);
     private static final Username USER = Username.of("user@domain.org");
     private static final String USER_PASS = "pass";
+    public static final String SMALL_MESSAGE = "header: value\r\n\r\nBODY";
 
     private static XMLConfiguration getConfig(InputStream configStream) throws Exception {
         FileBasedConfigurationBuilder<XMLConfiguration> builder = new FileBasedConfigurationBuilder<>(XMLConfiguration.class)
@@ -119,30 +121,42 @@ class IMAPServerTest {
         }
 
         @Test
-        void smallAppendsShouldWork() {
+        void smallAppendsShouldWork() throws Exception {
             assertThatCode(() ->
                 testIMAPClient.connect("127.0.0.1", port)
                     .login(USER.asString(), USER_PASS)
-                    .append("INBOX", "header: value\r\n\r\nBODY"))
+                    .append("INBOX", SMALL_MESSAGE))
                 .doesNotThrowAnyException();
+
+            assertThat(testIMAPClient.select("INBOX")
+                    .readFirstMessage())
+                .contains("\r\n" + SMALL_MESSAGE + ")\r\n");
         }
 
         @Test
-        void mediumAppendsShouldWork() {
+        void mediumAppendsShouldWork() throws Exception {
             assertThatCode(() ->
                 testIMAPClient.connect("127.0.0.1", port)
                     .login(USER.asString(), USER_PASS)
                     .append("INBOX", _65K_MESSAGE))
                 .doesNotThrowAnyException();
+
+            assertThat(testIMAPClient.select("INBOX")
+                    .readFirstMessage())
+                .contains("\r\n" + _65K_MESSAGE + ")\r\n");
         }
 
         @Test
-        void largeAppendsShouldWork() {
+        void largeAppendsShouldWork() throws Exception {
             assertThatCode(() ->
                 testIMAPClient.connect("127.0.0.1", port)
                     .login(USER.asString(), USER_PASS)
                     .append("INBOX", _129K_MESSAGE))
                 .doesNotThrowAnyException();
+
+            assertThat(testIMAPClient.select("INBOX")
+                    .readFirstMessage())
+                .contains("\r\n" + _129K_MESSAGE + ")\r\n");
         }
     }
 
@@ -163,21 +177,29 @@ class IMAPServerTest {
         }
 
         @Test
-        void smallAppendsShouldWork() {
+        void smallAppendsShouldWork() throws Exception {
             assertThatCode(() ->
                 testIMAPClient.connect("127.0.0.1", port)
                     .login(USER.asString(), USER_PASS)
-                    .append("INBOX", "header: value\r\n\r\nBODY"))
+                    .append("INBOX", SMALL_MESSAGE))
                 .doesNotThrowAnyException();
+
+            assertThat(testIMAPClient.select("INBOX")
+                    .readFirstMessage())
+                .contains("\r\n" + SMALL_MESSAGE + ")\r\n");
         }
 
         @Test
-        void mediumAppendsShouldWork() {
+        void mediumAppendsShouldWork() throws Exception {
             assertThatCode(() ->
                 testIMAPClient.connect("127.0.0.1", port)
                     .login(USER.asString(), USER_PASS)
                     .append("INBOX", _65K_MESSAGE))
                 .doesNotThrowAnyException();
+
+            assertThat(testIMAPClient.select("INBOX")
+                    .readFirstMessage())
+                .contains("\r\n" + _65K_MESSAGE + ")\r\n");
         }
 
         @Test


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org