This uncovers a new test failure `BaseBlobIntegrationTest#testGetIfMatch`

testGetIfMatch was not actually testing the intended behavior. Neither request 
was attaching a If-Match header and the test was passing just fine. 

Now it is attaching an If-Match header but with quotes around the etag causing 
the request to be rejected. 


You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-labs-google/pull/151

-- Commit Summary --

  * Update ObjectApi and getBlob, fix testGetRangeOutOfRange

-- File Changes --

    M 
google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/blobstore/GoogleCloudStorageBlobStore.java
 (9)
    M 
google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/features/ObjectApi.java
 (5)
    M 
google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/options/GetObjectOptions.java
 (4)
    M 
google-cloud-storage/src/test/java/org/jclouds/googlecloudstorage/features/ObjectApiMockTest.java
 (4)

-- Patch Links --

https://github.com/jclouds/jclouds-labs-google/pull/151.patch
https://github.com/jclouds/jclouds-labs-google/pull/151.diff

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

Reply via email to