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 bt...@apache.org on 2018/08/03 00:58:21 UTC
[6/8] james-project git commit: JAMES-2425
MailRepositoryStore::getByPath should create mail repository when repository
exist
JAMES-2425 MailRepositoryStore::getByPath should create mail repository when repository exist
Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/6592d7ce
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/6592d7ce
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/6592d7ce
Branch: refs/heads/master
Commit: 6592d7cebacb4f8b5d4cfae82991700e41f69cac
Parents: 31662d5
Author: benwa <bt...@linagora.com>
Authored: Wed Aug 1 16:56:50 2018 +0700
Committer: Benoit Tellier <bt...@linagora.com>
Committed: Fri Aug 3 07:57:22 2018 +0700
----------------------------------------------------------------------
.../org/apache/james/utils/InMemoryMailRepositoryStore.java | 8 +++-----
.../apache/james/utils/InMemoryMailRepositoryStoreTest.java | 8 ++++++++
2 files changed, 11 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/james-project/blob/6592d7ce/server/container/guice/guice-common/src/main/java/org/apache/james/utils/InMemoryMailRepositoryStore.java
----------------------------------------------------------------------
diff --git a/server/container/guice/guice-common/src/main/java/org/apache/james/utils/InMemoryMailRepositoryStore.java b/server/container/guice/guice-common/src/main/java/org/apache/james/utils/InMemoryMailRepositoryStore.java
index b93ea77..b6e1491 100644
--- a/server/container/guice/guice-common/src/main/java/org/apache/james/utils/InMemoryMailRepositoryStore.java
+++ b/server/container/guice/guice-common/src/main/java/org/apache/james/utils/InMemoryMailRepositoryStore.java
@@ -103,11 +103,9 @@ public class InMemoryMailRepositoryStore implements MailRepositoryStore, Configu
@Override
public Stream<MailRepository> getByPath(MailRepositoryPath path) {
- return destinationToRepositoryAssociations
- .keySet()
- .stream()
- .filter((MailRepositoryUrl key) -> key.getPath().equals(path))
- .map(destinationToRepositoryAssociations::get);
+ return urlStore.listDistinct()
+ .filter(url -> url.getPath().equals(path))
+ .map(this::select);
}
@Override
http://git-wip-us.apache.org/repos/asf/james-project/blob/6592d7ce/server/container/guice/guice-common/src/test/java/org/apache/james/utils/InMemoryMailRepositoryStoreTest.java
----------------------------------------------------------------------
diff --git a/server/container/guice/guice-common/src/test/java/org/apache/james/utils/InMemoryMailRepositoryStoreTest.java b/server/container/guice/guice-common/src/test/java/org/apache/james/utils/InMemoryMailRepositoryStoreTest.java
index cb6cb00..44b91a9 100644
--- a/server/container/guice/guice-common/src/test/java/org/apache/james/utils/InMemoryMailRepositoryStoreTest.java
+++ b/server/container/guice/guice-common/src/test/java/org/apache/james/utils/InMemoryMailRepositoryStoreTest.java
@@ -197,6 +197,14 @@ public class InMemoryMailRepositoryStoreTest {
}
@Test
+ public void getByPathShouldReturnRepositoryWhenUrlExists() {
+ urlStore.add(FILE_REPO);
+
+ assertThat(repositoryStore.getByPath(FILE_REPO.getPath()))
+ .isNotEmpty();
+ }
+
+ @Test
public void getShouldReturnPreviouslyCreatedMailRepository() {
MailRepository mailRepository = repositoryStore.select(FILE_REPO);
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org