You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ma...@apache.org on 2015/11/02 14:47:53 UTC
svn commit: r1711989 -
/james/project/trunk/server/data/data-library/src/test/java/org/apache/james/rrt/lib/AbstractRecipientRewriteTableTest.java
Author: matthieu
Date: Mon Nov 2 13:47:53 2015
New Revision: 1711989
URL: http://svn.apache.org/viewvc?rev=1711989&view=rev
Log:
JAMES-1595 Test mapping sorting in AbstractRecipientRewriteTable
Modified:
james/project/trunk/server/data/data-library/src/test/java/org/apache/james/rrt/lib/AbstractRecipientRewriteTableTest.java
Modified: james/project/trunk/server/data/data-library/src/test/java/org/apache/james/rrt/lib/AbstractRecipientRewriteTableTest.java
URL: http://svn.apache.org/viewvc/james/project/trunk/server/data/data-library/src/test/java/org/apache/james/rrt/lib/AbstractRecipientRewriteTableTest.java?rev=1711989&r1=1711988&r2=1711989&view=diff
==============================================================================
--- james/project/trunk/server/data/data-library/src/test/java/org/apache/james/rrt/lib/AbstractRecipientRewriteTableTest.java (original)
+++ james/project/trunk/server/data/data-library/src/test/java/org/apache/james/rrt/lib/AbstractRecipientRewriteTableTest.java Mon Nov 2 13:47:53 2015
@@ -21,7 +21,6 @@ package org.apache.james.rrt.lib;
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.fail;
-import java.util.Arrays;
import java.util.Map;
import org.apache.james.lifecycle.api.LifecycleUtil;
@@ -301,7 +300,7 @@ public abstract class AbstractRecipientR
@Test
public void sortMappingsShouldReturnEmptyWhenEmpty() {
- assertThat(AbstractRecipientRewriteTable.sortMappings("")).isEqualTo("");
+ assertThat(AbstractRecipientRewriteTable.sortMappings("")).isEmpty();
}
@Test
@@ -312,19 +311,26 @@ public abstract class AbstractRecipientR
@Test
public void sortMappingsShouldReturnSameStringWhenTwoDomainAliases() {
- String firstAliasMapping = RecipientRewriteTable.ALIASDOMAIN_PREFIX + "first";
- String secondAliasMapping = RecipientRewriteTable.ALIASDOMAIN_PREFIX + "second";
- String mappings = MappingsImpl.fromCollection(Arrays.asList(firstAliasMapping, secondAliasMapping)).serialize();
- assertThat(AbstractRecipientRewriteTable.sortMappings(mappings)).isEqualTo(mappings);
+ MappingsImpl mappings = MappingsImpl.builder()
+ .add(RecipientRewriteTable.ALIASDOMAIN_PREFIX + "first")
+ .add(RecipientRewriteTable.ALIASDOMAIN_PREFIX + "second")
+ .build();
+ assertThat(AbstractRecipientRewriteTable.sortMappings(mappings.serialize())).isEqualTo(mappings.serialize());
}
@Test
public void sortMappingsShouldPutDomainAliasFirstWhenVariousMappings() {
String regexMapping = RecipientRewriteTable.REGEX_PREFIX + "first";
String domainMapping = RecipientRewriteTable.ALIASDOMAIN_PREFIX + "second";
- String inputMappings = MappingsImpl.fromCollection(Arrays.asList(regexMapping, domainMapping)).serialize();
- String expectedMappings = MappingsImpl.fromCollection(Arrays.asList(domainMapping, regexMapping)).serialize();
- assertThat(AbstractRecipientRewriteTable.sortMappings(inputMappings)).isEqualTo(expectedMappings);
+ MappingsImpl mappings = MappingsImpl.builder()
+ .add(regexMapping)
+ .add(domainMapping)
+ .build();
+ assertThat(AbstractRecipientRewriteTable.sortMappings(mappings.serialize()))
+ .isEqualTo(MappingsImpl.builder()
+ .add(domainMapping)
+ .add(regexMapping)
+ .build().serialize());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org