Repository: jclouds Updated Branches: refs/heads/master 775921cd5 -> 90fac3b68
JCLOUDS-1271: Remove BlobStore.signRemoveBlob Most providers never supported this functionality and the portable abstraction should not have included it. Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/90fac3b6 Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/90fac3b6 Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/90fac3b6 Branch: refs/heads/master Commit: 90fac3b683ad92f298251b47d8b8572d0bea3f97 Parents: 775921c Author: Andrew Gaul <[email protected]> Authored: Fri Mar 9 20:14:00 2018 -0800 Committer: Andrew Gaul <[email protected]> Committed: Fri Mar 9 20:52:12 2018 -0800 ---------------------------------------------------------------------- .../atmos/blobstore/AtmosBlobRequestSigner.java | 9 --------- .../blobstore/AtmosBlobRequestSignerTest.java | 14 -------------- .../filesystem/FilesystemBlobStoreTest.java | 8 -------- .../RegionScopedTemporaryUrlBlobSigner.java | 6 ------ .../s3/blobstore/S3BlobRequestSigner.java | 8 -------- .../jclouds/blobstore/BlobRequestSigner.java | 14 -------------- .../blobstore/LocalBlobRequestSigner.java | 8 -------- .../internal/RequestSigningUnsupported.java | 6 ------ .../TransientBlobRequestSignerTest.java | 11 ----------- .../internal/BaseBlobSignerLiveTest.java | 20 -------------------- .../internal/BaseBlobSignerExpectTest.java | 6 ------ .../blobstore/AWSS3BlobSignerV4ExpectTest.java | 5 ----- .../blobstore/AzureBlobRequestSigner.java | 2 -- .../blobstore/AzureBlobRequestSignerTest.java | 4 ++-- .../integration/B2BlobSignerLiveTest.java | 10 ---------- .../GoogleCloudStorageBlobRequestSigner.java | 6 ------ .../GoogleCloudStorageBlobSignerLiveTest.java | 9 --------- 17 files changed, 2 insertions(+), 144 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/apis/atmos/src/main/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSigner.java ---------------------------------------------------------------------- diff --git a/apis/atmos/src/main/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSigner.java b/apis/atmos/src/main/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSigner.java index d8b23e1..0c58d4c 100644 --- a/apis/atmos/src/main/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSigner.java +++ b/apis/atmos/src/main/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSigner.java @@ -111,15 +111,6 @@ public class AtmosBlobRequestSigner implements BlobRequestSigner { throw new UnsupportedOperationException(); } - @Deprecated - @Override - public HttpRequest signRemoveBlob(String container, String name) { - checkNotNull(container, "container"); - checkNotNull(name, "name"); - return cleanRequest(processor.apply(Invocation.create(deleteMethod, - ImmutableList.<Object> of(getPath(container, name))))); - } - private String getPath(String container, String name) { return checkNotNull(container, "container") + "/" + checkNotNull(name, "name"); } http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/apis/atmos/src/test/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSignerTest.java ---------------------------------------------------------------------- diff --git a/apis/atmos/src/test/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSignerTest.java b/apis/atmos/src/test/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSignerTest.java index 883873a..7e3f246 100644 --- a/apis/atmos/src/test/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSignerTest.java +++ b/apis/atmos/src/test/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSignerTest.java @@ -66,20 +66,6 @@ public class AtmosBlobRequestSignerTest extends BaseRestAnnotationProcessingTest assertEquals(request.getFilters().size(), 0); } - public void testSignRemoveBlob() throws ArrayIndexOutOfBoundsException, SecurityException, IllegalArgumentException, - NoSuchMethodException, IOException { - HttpRequest request = signer.signRemoveBlob("container", "name"); - - assertRequestLineEquals(request, - "DELETE https://accesspoint.atmosonline.com/rest/namespace/container/name HTTP/1.1"); - assertNonPayloadHeadersEqual( - request, - "Accept: */*\nDate: Thu, 05 Jun 2008 16:38:19 GMT\nx-emc-signature: cPnxwSdWfIjChx8sox+43U9oo20=\nx-emc-uid: identity\n"); - assertPayloadEquals(request, null, null, false); - - assertEquals(request.getFilters().size(), 0); - } - public void testSignPutBlob() throws ArrayIndexOutOfBoundsException, SecurityException, IllegalArgumentException, NoSuchMethodException, IOException { HashCode hashCode = HashCode.fromBytes(new byte[16]); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/apis/filesystem/src/test/java/org/jclouds/filesystem/FilesystemBlobStoreTest.java ---------------------------------------------------------------------- diff --git a/apis/filesystem/src/test/java/org/jclouds/filesystem/FilesystemBlobStoreTest.java b/apis/filesystem/src/test/java/org/jclouds/filesystem/FilesystemBlobStoreTest.java index e23bebd..71dd4e7 100644 --- a/apis/filesystem/src/test/java/org/jclouds/filesystem/FilesystemBlobStoreTest.java +++ b/apis/filesystem/src/test/java/org/jclouds/filesystem/FilesystemBlobStoreTest.java @@ -806,14 +806,6 @@ public class FilesystemBlobStoreTest { .build(); assertEquals(expected, request); - request = signer.signRemoveBlob(containerName, blobName); - expected = HttpRequest.builder() - .method("DELETE") - .endpoint(endPoint) - .headers(request.getHeaders()) - .build(); - assertEquals(expected, request); - Blob blob = blobStore.blobBuilder(blobName).forSigning().build(); request = signer.signPutBlob(containerName, blob); expected = HttpRequest.builder() http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedTemporaryUrlBlobSigner.java ---------------------------------------------------------------------- diff --git a/apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedTemporaryUrlBlobSigner.java b/apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedTemporaryUrlBlobSigner.java index d4182d7..cd966fc 100644 --- a/apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedTemporaryUrlBlobSigner.java +++ b/apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedTemporaryUrlBlobSigner.java @@ -88,12 +88,6 @@ public class RegionScopedTemporaryUrlBlobSigner implements BlobRequestSigner { return sign("PUT", container, blob.getMetadata().getName(), GetOptions.NONE, timestamp.get() + timeInSeconds); } - @Deprecated - @Override - public HttpRequest signRemoveBlob(String container, String name) { - return sign("DELETE", container, name, GetOptions.NONE, timestamp.get() + DEFAULT_SIGNING_TIMEOUT); - } - private HttpRequest sign(String method, String container, String name, GetOptions options, long expires) { checkNotNull(container, "container"); checkNotNull(name, "name"); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/apis/s3/src/main/java/org/jclouds/s3/blobstore/S3BlobRequestSigner.java ---------------------------------------------------------------------- diff --git a/apis/s3/src/main/java/org/jclouds/s3/blobstore/S3BlobRequestSigner.java b/apis/s3/src/main/java/org/jclouds/s3/blobstore/S3BlobRequestSigner.java index 015b5e6..9707f4f 100644 --- a/apis/s3/src/main/java/org/jclouds/s3/blobstore/S3BlobRequestSigner.java +++ b/apis/s3/src/main/java/org/jclouds/s3/blobstore/S3BlobRequestSigner.java @@ -95,14 +95,6 @@ public class S3BlobRequestSigner<T extends S3Client> implements BlobRequestSigne return cleanRequest(authSigner.signForTemporaryAccess(request, timeInSeconds)); } - @Deprecated - @Override - public HttpRequest signRemoveBlob(String container, String name) { - checkNotNull(container, "container"); - checkNotNull(name, "name"); - return cleanRequest(processor.apply(Invocation.create(deleteMethod, ImmutableList.<Object> of(container, name)))); - } - @Override public HttpRequest signGetBlob(String container, String name, org.jclouds.blobstore.options.GetOptions options) { checkNotNull(container, "container"); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/blobstore/src/main/java/org/jclouds/blobstore/BlobRequestSigner.java ---------------------------------------------------------------------- diff --git a/blobstore/src/main/java/org/jclouds/blobstore/BlobRequestSigner.java b/blobstore/src/main/java/org/jclouds/blobstore/BlobRequestSigner.java index a86a84a..d8c62c1 100644 --- a/blobstore/src/main/java/org/jclouds/blobstore/BlobRequestSigner.java +++ b/blobstore/src/main/java/org/jclouds/blobstore/BlobRequestSigner.java @@ -61,20 +61,6 @@ public interface BlobRequestSigner { HttpRequest signGetBlob(String container, String name, GetOptions options); /** - * gets a signed request, including headers as necessary, to delete a blob from an external - * client. - * - * @param container - * container where the blob resides - * @param directory - * full path to the blob - * @throws UnsupportedOperationException - * if not supported by the provider - */ - @Deprecated - HttpRequest signRemoveBlob(String container, String name); - - /** * gets a signed request, including headers as necessary, to upload a blob from an external * client. * http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/blobstore/src/main/java/org/jclouds/blobstore/LocalBlobRequestSigner.java ---------------------------------------------------------------------- diff --git a/blobstore/src/main/java/org/jclouds/blobstore/LocalBlobRequestSigner.java b/blobstore/src/main/java/org/jclouds/blobstore/LocalBlobRequestSigner.java index 13f8520..c9ba42d 100644 --- a/blobstore/src/main/java/org/jclouds/blobstore/LocalBlobRequestSigner.java +++ b/blobstore/src/main/java/org/jclouds/blobstore/LocalBlobRequestSigner.java @@ -75,14 +75,6 @@ public class LocalBlobRequestSigner implements BlobRequestSigner { throw new UnsupportedOperationException(); } - @Deprecated - @Override - public HttpRequest signRemoveBlob(String container, String name) { - HttpRequest request = HttpRequest.builder().method("DELETE").endpoint(String.format("%s/%s/%s", endpoint.get(), container, - name)).build(); - return basicAuth.filter(request); - } - @Override public HttpRequest signGetBlob(String container, String name, GetOptions options) { HttpRequest request = HttpRequest.builder().method("GET").endpoint( http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/blobstore/src/main/java/org/jclouds/blobstore/internal/RequestSigningUnsupported.java ---------------------------------------------------------------------- diff --git a/blobstore/src/main/java/org/jclouds/blobstore/internal/RequestSigningUnsupported.java b/blobstore/src/main/java/org/jclouds/blobstore/internal/RequestSigningUnsupported.java index b3de332..298b044 100644 --- a/blobstore/src/main/java/org/jclouds/blobstore/internal/RequestSigningUnsupported.java +++ b/blobstore/src/main/java/org/jclouds/blobstore/internal/RequestSigningUnsupported.java @@ -41,12 +41,6 @@ public class RequestSigningUnsupported implements BlobRequestSigner { throw new UnsupportedOperationException(); } - @Deprecated - @Override - public HttpRequest signRemoveBlob(String container, String name) { - throw new UnsupportedOperationException(); - } - @Override public HttpRequest signPutBlob(String container, Blob blob) { throw new UnsupportedOperationException(); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java ---------------------------------------------------------------------- diff --git a/blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java b/blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java index 633b737..028f569 100644 --- a/blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java +++ b/blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java @@ -61,17 +61,6 @@ public class TransientBlobRequestSignerTest extends BaseRestAnnotationProcessing assertEquals(request.getFilters().size(), 0); } - public void testSignRemoveBlob() throws ArrayIndexOutOfBoundsException, SecurityException, IllegalArgumentException, - NoSuchMethodException, IOException { - HttpRequest request = signer.signRemoveBlob(containerName, blobName); - - assertRequestLineEquals(request, "DELETE " + fullUrl + " HTTP/1.1"); - assertNonPayloadHeadersEqual(request, "Authorization: Basic aWRlbnRpdHk6Y3JlZGVudGlhbA==\n"); - assertPayloadEquals(request, null, null, false); - - assertEquals(request.getFilters().size(), 0); - } - public void testSignPutBlob() throws ArrayIndexOutOfBoundsException, SecurityException, IllegalArgumentException, NoSuchMethodException, IOException { HashCode hashCode = HashCode.fromBytes(new byte[16]); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobSignerLiveTest.java ---------------------------------------------------------------------- diff --git a/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobSignerLiveTest.java b/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobSignerLiveTest.java index 8ec0434..56c1167 100644 --- a/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobSignerLiveTest.java +++ b/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobSignerLiveTest.java @@ -193,26 +193,6 @@ public class BaseBlobSignerLiveTest extends BaseBlobStoreIntegrationTest { testSignPutUrlWithTime(-getSignedUrlTimeout()); } - @Test - public void testSignRemoveUrl() throws Exception { - String name = "hello"; - String text = "fooooooooooooooooooooooo"; - - Blob blob = view.getBlobStore().blobBuilder(name).payload(text).contentType("text/plain").build(); - String container = getContainerName(); - try { - view.getBlobStore().putBlob(container, blob); - awaitConsistency(); - assertConsistencyAwareContainerSize(container, 1); - HttpRequest request = view.getSigner().signRemoveBlob(container, name); - assertEquals(request.getFilters().size(), 0); - view.utils().http().invoke(request); - assert !view.getBlobStore().blobExists(container, name); - } finally { - returnContainer(container); - } - } - protected void awaitConsistency() { if (view.getConsistencyModel() == ConsistencyModel.EVENTUAL) { Uninterruptibles.sleepUninterruptibly(AWAIT_CONSISTENCY_TIMEOUT_SECONDS, TimeUnit.SECONDS); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/blobstore/src/test/java/org/jclouds/blobstore/internal/BaseBlobSignerExpectTest.java ---------------------------------------------------------------------- diff --git a/blobstore/src/test/java/org/jclouds/blobstore/internal/BaseBlobSignerExpectTest.java b/blobstore/src/test/java/org/jclouds/blobstore/internal/BaseBlobSignerExpectTest.java index e623422..39c49b9 100644 --- a/blobstore/src/test/java/org/jclouds/blobstore/internal/BaseBlobSignerExpectTest.java +++ b/blobstore/src/test/java/org/jclouds/blobstore/internal/BaseBlobSignerExpectTest.java @@ -79,12 +79,6 @@ public abstract class BaseBlobSignerExpectTest extends BaseRestApiExpectTest<Blo protected abstract HttpRequest getBlobWithOptions(); @Test - public void testSignRemoveBlob() { - BlobStore removeBlob = requestsSendResponses(init()); - assertEquals(removeBlob.getContext().getSigner().signRemoveBlob(container, name), removeBlob()); - } - - @Test public void testSignPutBlob() throws Exception { HashCode hashCode = HashCode.fromBytes(new byte[16]); BlobStore signPutBlob = requestsSendResponses(init()); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/providers/aws-s3/src/test/java/org/jclouds/aws/s3/blobstore/AWSS3BlobSignerV4ExpectTest.java ---------------------------------------------------------------------- diff --git a/providers/aws-s3/src/test/java/org/jclouds/aws/s3/blobstore/AWSS3BlobSignerV4ExpectTest.java b/providers/aws-s3/src/test/java/org/jclouds/aws/s3/blobstore/AWSS3BlobSignerV4ExpectTest.java index 8b2af5e..80dc438 100644 --- a/providers/aws-s3/src/test/java/org/jclouds/aws/s3/blobstore/AWSS3BlobSignerV4ExpectTest.java +++ b/providers/aws-s3/src/test/java/org/jclouds/aws/s3/blobstore/AWSS3BlobSignerV4ExpectTest.java @@ -136,11 +136,6 @@ public class AWSS3BlobSignerV4ExpectTest extends S3BlobSignerExpectTest { } @Override - public void testSignRemoveBlob() { - throw new SkipException("skip testSignRemoveBlob"); - } - - @Override protected Module createModule() { return new TestAWSS3SignerV4HttpApiModule(); } http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSigner.java ---------------------------------------------------------------------- diff --git a/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSigner.java b/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSigner.java index 1db3547..0053cf6 100644 --- a/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSigner.java +++ b/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSigner.java @@ -90,8 +90,6 @@ public class AzureBlobRequestSigner implements BlobRequestSigner { blob.getMetadata().getContentMetadata().getContentLength()); } - @Deprecated - @Override public HttpRequest signRemoveBlob(String container, String name) { checkNotNull(container, "container"); checkNotNull(name, "name"); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java ---------------------------------------------------------------------- diff --git a/providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java b/providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java index 165dccc..45aefa5 100644 --- a/providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java +++ b/providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java @@ -50,7 +50,7 @@ public class AzureBlobRequestSignerTest extends BaseRestAnnotationProcessingTest credential = "aaaabbbb"; } - private BlobRequestSigner signer; + private AzureBlobRequestSigner signer; private Factory blobFactory; public void testSignGetBlob() throws ArrayIndexOutOfBoundsException, SecurityException, IllegalArgumentException, @@ -102,7 +102,7 @@ public class AzureBlobRequestSignerTest extends BaseRestAnnotationProcessingTest protected void setupFactory() throws IOException { super.setupFactory(); this.blobFactory = injector.getInstance(Blob.Factory.class); - this.signer = injector.getInstance(BlobRequestSigner.class); + this.signer = (AzureBlobRequestSigner) injector.getInstance(BlobRequestSigner.class); } @Override http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/providers/b2/src/test/java/org/jclouds/b2/blobstore/integration/B2BlobSignerLiveTest.java ---------------------------------------------------------------------- diff --git a/providers/b2/src/test/java/org/jclouds/b2/blobstore/integration/B2BlobSignerLiveTest.java b/providers/b2/src/test/java/org/jclouds/b2/blobstore/integration/B2BlobSignerLiveTest.java index 7a027f0..83f7a75 100644 --- a/providers/b2/src/test/java/org/jclouds/b2/blobstore/integration/B2BlobSignerLiveTest.java +++ b/providers/b2/src/test/java/org/jclouds/b2/blobstore/integration/B2BlobSignerLiveTest.java @@ -99,14 +99,4 @@ public final class B2BlobSignerLiveTest extends BaseBlobSignerLiveTest { throw new SkipException("unsupported by B2", uoe); } } - - @Test - public void testSignRemoveUrl() throws Exception { - try { - super.testSignRemoveUrl(); - failBecauseExceptionWasNotThrown(UnsupportedOperationException.class); - } catch (UnsupportedOperationException uoe) { - throw new SkipException("unsupported by B2", uoe); - } - } } http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/providers/google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/blobstore/GoogleCloudStorageBlobRequestSigner.java ---------------------------------------------------------------------- diff --git a/providers/google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/blobstore/GoogleCloudStorageBlobRequestSigner.java b/providers/google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/blobstore/GoogleCloudStorageBlobRequestSigner.java index 390ada1..d41de7d 100644 --- a/providers/google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/blobstore/GoogleCloudStorageBlobRequestSigner.java +++ b/providers/google-cloud-storage/src/main/java/org/jclouds/googlecloudstorage/blobstore/GoogleCloudStorageBlobRequestSigner.java @@ -98,12 +98,6 @@ public final class GoogleCloudStorageBlobRequestSigner implements BlobRequestSig return sign("PUT", container, blob.getMetadata().getName(), GetOptions.NONE, timestamp.get() + timeInSeconds, null); } - @Deprecated - @Override - public HttpRequest signRemoveBlob(String container, String name) { - throw new UnsupportedOperationException(); - } - private HttpRequest sign(String method, String container, String name, GetOptions options, long expires, String contentType) { checkNotNull(container, "container"); checkNotNull(name, "name"); http://git-wip-us.apache.org/repos/asf/jclouds/blob/90fac3b6/providers/google-cloud-storage/src/test/java/org/jclouds/googlecloudstorage/blobstore/integration/GoogleCloudStorageBlobSignerLiveTest.java ---------------------------------------------------------------------- diff --git a/providers/google-cloud-storage/src/test/java/org/jclouds/googlecloudstorage/blobstore/integration/GoogleCloudStorageBlobSignerLiveTest.java b/providers/google-cloud-storage/src/test/java/org/jclouds/googlecloudstorage/blobstore/integration/GoogleCloudStorageBlobSignerLiveTest.java index 2f76490..e2834b7 100644 --- a/providers/google-cloud-storage/src/test/java/org/jclouds/googlecloudstorage/blobstore/integration/GoogleCloudStorageBlobSignerLiveTest.java +++ b/providers/google-cloud-storage/src/test/java/org/jclouds/googlecloudstorage/blobstore/integration/GoogleCloudStorageBlobSignerLiveTest.java @@ -17,7 +17,6 @@ package org.jclouds.googlecloudstorage.blobstore.integration; import org.jclouds.blobstore.integration.internal.BaseBlobSignerLiveTest; -import org.testng.SkipException; import org.testng.annotations.Test; @Test(groups = { "live" }) @@ -25,12 +24,4 @@ public class GoogleCloudStorageBlobSignerLiveTest extends BaseBlobSignerLiveTest public GoogleCloudStorageBlobSignerLiveTest() { provider = "google-cloud-storage"; } - - public void testSignRemoveUrl() throws Exception { - try { - super.testSignRemoveUrl(); - } catch (UnsupportedOperationException uoe) { - throw new SkipException("not yet implemented in GCS", uoe); - } - } }
