This is an automated email from the ASF dual-hosted git repository. gaul pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/jclouds.git
commit 311a4102dad7b44d74c0a99b1cbaade8506e2096 Author: jixinchi <[email protected]> AuthorDate: Tue Nov 14 17:29:16 2023 +0800 fix typo and add unit test --- .../org/jclouds/blobstore/config/LocalBlobStore.java | 1 - .../blobstore/strategy/internal/ListContainerTest.java | 16 ++++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java b/blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java index 9b1ef849bd..d44f93d748 100644 --- a/blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java +++ b/blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java @@ -367,7 +367,6 @@ public final class LocalBlobStore implements BlobStore { if (!contents.contains(md)) { contents.add(md); } - contents.add(md); } return contents; } diff --git a/blobstore/src/test/java/org/jclouds/blobstore/strategy/internal/ListContainerTest.java b/blobstore/src/test/java/org/jclouds/blobstore/strategy/internal/ListContainerTest.java index cb9f79e0a1..78af809efe 100644 --- a/blobstore/src/test/java/org/jclouds/blobstore/strategy/internal/ListContainerTest.java +++ b/blobstore/src/test/java/org/jclouds/blobstore/strategy/internal/ListContainerTest.java @@ -193,6 +193,22 @@ public class ListContainerTest { assertThat(Iterables.get(results, 0).getType()).isNotEqualTo(StorageType.RELATIVE_PATH); } + public void testListBlobEndsWithDelimiterAndDelimiterFilter() { + String containerName = "testListBlobEndsWithDelimiterAndDelimiterFilter"; + blobStore.createContainerInLocation(null, containerName); + blobStore.putBlob(containerName, blobStore.blobBuilder("foo/").type(StorageType.FOLDER) + .payload(ByteSource.empty()).build()); + blobStore.putBlob(containerName, blobStore.blobBuilder("bar/text").type(StorageType.BLOB) + .payload(ByteSource.empty()).build()); + PageSet<? extends StorageMetadata> results = blobStore.list(containerName, + ListContainerOptions.Builder.delimiter("/")); + assertThat(results.size()).isEqualTo(2); + assertThat(Iterables.get(results, 0).getName()).isEqualTo("bar/"); + assertThat(Iterables.get(results, 0).getType()).isEqualTo(StorageType.RELATIVE_PATH); + assertThat(Iterables.get(results, 1).getName()).isEqualTo("foo/"); + assertThat(Iterables.get(results, 1).getType()).isEqualTo(StorageType.FOLDER); + } + public void testDirectoryListing() { String containerName = "testDirectoryListing"; blobStore.createContainerInLocation(null, containerName);
