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

Reply via email to