jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Timur Alperovich <notificati...@github.com>
Subject [jclouds] Handle delimiter and prefix being set in LocalBlobStore (#812)
Date Tue, 14 Jul 2015 20:19:59 GMT
If the prefix is set, we need to also handle the delimiter option. One commit plumbs this through.
Another commit adds a test for the scenario. The last commit moves the ```testContainerListWithPrefix```
and ```testDelimiterList``` tests from ```BaseContainerLiveTest`` to ```BaseContainerIntegrationTest```.
The ```BaseContainerLiveTest``` is not run for the file system API and regressions, such as
the one above occur.

Lastly, the delimiter tests are changed from using a (possible) file separator character to
use "-", as creating a blob with a file separator character in its name will cause jclouds
to create a directory, which is visible when listing blobs.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/812

-- Commit Summary --

  * Add a test that sets delimiter and prefix.
  * Move tests to test the file system API.
  * Propagate delimiter changes to LocalBlobStore.

-- File Changes --

    M apis/atmos/src/test/java/org/jclouds/atmos/blobstore/integration/AtmosContainerIntegrationLiveTest.java
(10)
    M apis/atmos/src/test/java/org/jclouds/atmos/blobstore/integration/AtmosContainerLiveTest.java
(11)
    M apis/swift/src/test/java/org/jclouds/openstack/swift/blobstore/integration/SwiftContainerLiveTest.java
(11)
    M blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java (13)
    M blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseContainerIntegrationTest.java
(55)
    M blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseContainerLiveTest.java
(57)
    M blobstore/src/test/java/org/jclouds/blobstore/strategy/internal/ListContainerTest.java
(18)
    M providers/hpcloud-objectstorage/src/test/java/org/jclouds/hpcloud/objectstorage/blobstore/integration/HPCloudObjectStorageContainerIntegrationLiveTest.java
(5)
    M providers/hpcloud-objectstorage/src/test/java/org/jclouds/hpcloud/objectstorage/blobstore/integration/HPCloudObjectStorageContainerLiveTest.java
(6)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/812.patch
https://github.com/jclouds/jclouds/pull/812.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/812

Mime
View raw message