Method listing the contents of a bucket in FakeStorageRpc does not use bucket name

Hello,

The implementation of the method list(String bucket, Map<Option, ?> options) in FakeStorageRpc does not use bucket parameter to retrieve the files inside the bucket passed as parameter (see FakeStorageRpc.java). I’m using version 0.34.0, but it seems like in previous versions the behaviour was similar. Am I missing something ?

As a consequence, when using LocalStorageHelper (using FakeStorageRpc) for local testing purposes, the following code does not retrieve only the files in the bucket passed as parameter, but all files in all created buckets :

Storage storage = LocalStorageHelper.customOptions(true).getService()

[...] // creating dummy files in two separate buckets (b1 and b2)

storage.list("b1") // returns all files in b1 and b2

I double checked in the doc, listing the contents of a bucket is however defined as a supported operation.

[norbjd]

Thanks for your replies @hzyi-google @pongad

It’s all clear now, in fact I was just confused because the bucket parameter is not used in the implementation of list method in FakeStorageRpc.

I looked to tests in google-cloud-storage as you suggest (BucketTest.java), but I am not able to used these tests “as-is”, since many classes are package-private (for instance, I can’t initialize a Blob).

Storage can be mocked, and in this example, mocking also Blob works, but I think it will be a good idea to implement method to test storage locally (with LocalStorageHelper) without any mocking.

[yihanzhen]

The feature request is added to the backlog. Feel free to add comments here.