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 2023/01/04 02:31:22 UTC

[james-project] 03/03: JAMES-3872 Fix failed JpaMessageMapperTest

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 a7ae09187f16bc9484c1500d84257d9a1229f856
Author: Quan Tran <hq...@linagora.com>
AuthorDate: Tue Jan 3 23:04:11 2023 +0700

    JAMES-3872 Fix failed JpaMessageMapperTest
---
 .../james/mailbox/cassandra/mail/CassandraMessageMapperTest.java       | 2 ++
 .../org/apache/james/mailbox/store/mail/model/MessageMapperTest.java   | 3 ---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/mailbox/cassandra/src/test/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageMapperTest.java b/mailbox/cassandra/src/test/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageMapperTest.java
index 028d4cf0d0..e6bc711bdf 100644
--- a/mailbox/cassandra/src/test/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageMapperTest.java
+++ b/mailbox/cassandra/src/test/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageMapperTest.java
@@ -51,6 +51,7 @@ import org.apache.james.mailbox.model.MessageRange;
 import org.apache.james.mailbox.model.ParsedAttachment;
 import org.apache.james.mailbox.model.ThreadId;
 import org.apache.james.mailbox.store.FlagsUpdateCalculator;
+import org.apache.james.mailbox.store.mail.AttachmentMapper;
 import org.apache.james.mailbox.store.mail.MessageMapper;
 import org.apache.james.mailbox.store.mail.MessageMapper.FetchType;
 import org.apache.james.mailbox.store.mail.model.MailboxMessage;
@@ -445,6 +446,7 @@ class CassandraMessageMapperTest extends MessageMapperTest {
 
     @Test
     void messagesRetrievedUsingFetchTypeAttachmentsMetadataShouldHaveAttachmentsMetadataLoaded() throws MailboxException {
+        AttachmentMapper attachmentMapper = mapperProvider.createAttachmentMapper();
         MessageId messageId = mapperProvider.generateMessageId();
         String content = "Subject: Test1 \n\nBody1\n.\n";
         ParsedAttachment attachment1 = ParsedAttachment.builder()
diff --git a/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/MessageMapperTest.java b/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/MessageMapperTest.java
index 2a49fc1af6..1efac851ea 100644
--- a/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/MessageMapperTest.java
+++ b/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/MessageMapperTest.java
@@ -53,7 +53,6 @@ import org.apache.james.mailbox.model.ThreadId;
 import org.apache.james.mailbox.model.UidValidity;
 import org.apache.james.mailbox.model.UpdatedFlags;
 import org.apache.james.mailbox.store.FlagsUpdateCalculator;
-import org.apache.james.mailbox.store.mail.AttachmentMapper;
 import org.apache.james.mailbox.store.mail.MailboxMapper;
 import org.apache.james.mailbox.store.mail.MessageMapper;
 import org.apache.james.mailbox.store.mail.MessageMapper.FetchType;
@@ -85,7 +84,6 @@ public abstract class MessageMapperTest {
     protected MapperProvider mapperProvider;
     protected MessageMapper messageMapper;
     private MailboxMapper mailboxMapper;
-    protected AttachmentMapper attachmentMapper;
 
     protected Mailbox benwaInboxMailbox;
     protected Mailbox benwaWorkMailbox;
@@ -109,7 +107,6 @@ public abstract class MessageMapperTest {
 
         this.messageMapper = mapperProvider.createMessageMapper();
         this.mailboxMapper = mapperProvider.createMailboxMapper();
-        this.attachmentMapper = mapperProvider.createAttachmentMapper();
 
         initData();
     }


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