You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by rc...@apache.org on 2023/12/06 08:12:05 UTC

(james-project) 05/06: JAMES-2586 Enable PostgresMailboxManager annotation tests

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

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

commit a38f239a50ab317f5fa19a409a8fd689ac084a1a
Author: Benoit TELLIER <bt...@linagora.com>
AuthorDate: Tue Dec 5 16:08:15 2023 +0100

    JAMES-2586 Enable PostgresMailboxManager annotation tests
---
 .../src/test/java/org/apache/james/mailbox/MailboxManagerTest.java  | 4 +++-
 .../apache/james/mailbox/postgres/PostgresMailboxManagerTest.java   | 6 ------
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java b/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
index 6e8325ce5c..8bc9d387ea 100644
--- a/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
+++ b/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
@@ -651,7 +651,9 @@ public abstract class MailboxManagerTest<T extends MailboxManager> {
 
             mailboxManager.updateAnnotations(inbox, session, annotations);
 
-            assertThat(mailboxManager.getAllAnnotations(inbox, session)).isEqualTo(annotations);
+            assertThat(mailboxManager.getAllAnnotations(inbox, session))
+                .hasSize(annotations.size())
+                .containsAnyElementsOf(annotations);
         }
 
         @Test
diff --git a/mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresMailboxManagerTest.java b/mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresMailboxManagerTest.java
index d7fc4f355e..537a124c96 100644
--- a/mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresMailboxManagerTest.java
+++ b/mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresMailboxManagerTest.java
@@ -38,11 +38,6 @@ class PostgresMailboxManagerTest extends MailboxManagerTest<PostgresMailboxManag
     class HookTests {
     }
 
-    @Disabled("//TODO https://github.com/apache/james-project/pull/1822")
-    @Nested
-    class AnnotationTests {
-    }
-
     @RegisterExtension
     static PostgresExtension postgresExtension = PostgresExtension.withoutRowLevelSecurity(PostgresMailboxAggregateModule.MODULE);
 
@@ -61,7 +56,6 @@ class PostgresMailboxManagerTest extends MailboxManagerTest<PostgresMailboxManag
         return new StoreSubscriptionManager(provideMailboxManager().getMapperFactory(), provideMailboxManager().getMapperFactory(), provideMailboxManager().getEventBus());
     }
 
-
     @Override
     protected EventBus retrieveEventBus(PostgresMailboxManager mailboxManager) {
         return mailboxManager.getEventBus();


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