amichair commented on code in PR #2444:
URL: https://github.com/apache/james-project/pull/2444#discussion_r1796120281


##########
server/data/data-library/src/test/java/org/apache/james/rrt/lib/RecipientRewriteTableContract.java:
##########
@@ -411,6 +411,19 @@ default void 
listSourcesShouldReturnWhenMultipleSourceMapping() throws Exception
         assertThat(virtualUserTable().listSources(mapping)).contains(source, 
source2);
     }
 
+    @Test
+    default void listSourcesShouldReturnWhenSourceHasMultipleMappings() throws 
Exception {

Review Comment:
   I'm not sure what you mean. This was a bug I stumbled upon in the previous 
code while working on this, so I added a test just to make sure it no longer 
occurs in the new design. Everything else I assume should be covered by 
existing tests...



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to