You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ga...@apache.org on 2016/01/19 23:42:20 UTC

jclouds git commit: Add BaseBlobIntegrationTest.testSetBlobAccess

Repository: jclouds
Updated Branches:
  refs/heads/master 40f31786c -> 52dc1a3cc


Add BaseBlobIntegrationTest.testSetBlobAccess

Similar to BaseContainerIntegrationTest.testSetContainerAccess.


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/52dc1a3c
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/52dc1a3c
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/52dc1a3c

Branch: refs/heads/master
Commit: 52dc1a3ccfc6413d3350bc8f1d092cda8d7a25d3
Parents: 40f3178
Author: Andrew Gaul <ga...@apache.org>
Authored: Tue Jan 19 14:38:38 2016 -0800
Committer: Andrew Gaul <ga...@apache.org>
Committed: Tue Jan 19 14:40:55 2016 -0800

----------------------------------------------------------------------
 .../integration/FilesystemContainerIntegrationTest.java      | 2 +-
 .../blobstore/integration/TransientBlobIntegrationTest.java  | 7 +++++++
 .../integration/TransientContainerIntegrationTest.java       | 2 +-
 .../integration/internal/BaseBlobIntegrationTest.java        | 8 ++++++++
 4 files changed, 17 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/52dc1a3c/apis/filesystem/src/test/java/org/jclouds/filesystem/integration/FilesystemContainerIntegrationTest.java
----------------------------------------------------------------------
diff --git a/apis/filesystem/src/test/java/org/jclouds/filesystem/integration/FilesystemContainerIntegrationTest.java b/apis/filesystem/src/test/java/org/jclouds/filesystem/integration/FilesystemContainerIntegrationTest.java
index b150b6a..7a701a0 100644
--- a/apis/filesystem/src/test/java/org/jclouds/filesystem/integration/FilesystemContainerIntegrationTest.java
+++ b/apis/filesystem/src/test/java/org/jclouds/filesystem/integration/FilesystemContainerIntegrationTest.java
@@ -189,6 +189,6 @@ public class FilesystemContainerIntegrationTest extends BaseContainerIntegration
    @Override
    @Test(groups = { "integration", "live" })
    public void testSetContainerAccess() throws Exception {
-      throw new SkipException("transient does not support anonymous access");
+      throw new SkipException("filesystem does not support anonymous access");
    }
 }

http://git-wip-us.apache.org/repos/asf/jclouds/blob/52dc1a3c/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientBlobIntegrationTest.java
----------------------------------------------------------------------
diff --git a/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientBlobIntegrationTest.java b/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientBlobIntegrationTest.java
index eeaaf96..e2b842e 100644
--- a/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientBlobIntegrationTest.java
+++ b/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientBlobIntegrationTest.java
@@ -18,10 +18,17 @@ package org.jclouds.blobstore.integration;
 
 import org.jclouds.blobstore.integration.internal.BaseBlobIntegrationTest;
 import org.testng.annotations.Test;
+import org.testng.SkipException;
 
 @Test(groups = { "integration" })
 public class TransientBlobIntegrationTest extends BaseBlobIntegrationTest {
    public TransientBlobIntegrationTest() {
       provider = "transient";
    }
+
+   @Override
+   @Test(groups = { "integration", "live" })
+   public void testSetBlobAccess() throws Exception {
+      throw new SkipException("transient does not support anonymous access");
+   }
 }

http://git-wip-us.apache.org/repos/asf/jclouds/blob/52dc1a3c/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientContainerIntegrationTest.java
----------------------------------------------------------------------
diff --git a/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientContainerIntegrationTest.java b/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientContainerIntegrationTest.java
index f12f273..c676b23 100644
--- a/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientContainerIntegrationTest.java
+++ b/blobstore/src/test/java/org/jclouds/blobstore/integration/TransientContainerIntegrationTest.java
@@ -85,6 +85,6 @@ public class TransientContainerIntegrationTest extends BaseContainerIntegrationT
    @Override
    @Test(groups = { "integration", "live" })
    public void testSetContainerAccess() throws Exception {
-      throw new SkipException("filesystem does not support anonymous access");
+      throw new SkipException("transient does not support anonymous access");
    }
 }

http://git-wip-us.apache.org/repos/asf/jclouds/blob/52dc1a3c/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java
----------------------------------------------------------------------
diff --git a/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java b/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java
index 479c174..8442377 100644
--- a/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java
+++ b/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java
@@ -65,6 +65,8 @@ import org.jclouds.blobstore.options.PutOptions;
 import org.jclouds.blobstore.strategy.internal.MultipartUploadSlicingAlgorithm;
 import org.jclouds.crypto.Crypto;
 import org.jclouds.encryption.internal.JCECrypto;
+import org.jclouds.http.HttpRequest;
+import org.jclouds.http.HttpResponse;
 import org.jclouds.http.HttpResponseException;
 import org.jclouds.io.ByteStreams2;
 import org.jclouds.io.ContentMetadataBuilder;
@@ -667,6 +669,12 @@ public class BaseBlobIntegrationTest extends BaseBlobStoreIntegrationTest {
          blobStore.setBlobAccess(containerName, blobName, BlobAccess.PUBLIC_READ);
          assertThat(blobStore.getBlobAccess(containerName, blobName)).isEqualTo(BlobAccess.PUBLIC_READ);
 
+         // test that blob is anonymously readable
+         HttpRequest request = view.getSigner().signGetBlob(containerName, blobName).toBuilder()
+                .replaceQueryParams(ImmutableMap.<String, String>of()).build();
+         HttpResponse response = view.utils().http().invoke(request);
+         assertThat(response.getStatusCode()).isEqualTo(200);
+
          blobStore.setBlobAccess(containerName, blobName, BlobAccess.PRIVATE);
          assertThat(blobStore.getBlobAccess(containerName, blobName)).isEqualTo(BlobAccess.PRIVATE);
       } finally {