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 2017/11/11 15:46:32 UTC

jclouds git commit: Tighten up test expectations

Repository: jclouds
Updated Branches:
  refs/heads/master 26570db68 -> e473d7df6


Tighten up test expectations

Follow on to 8b94febfeb7b55664fc6fca58bbc7ef20255c71c.


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

Branch: refs/heads/master
Commit: e473d7df6a28e8f98a424f53cd250e753730c866
Parents: 26570db
Author: Andrew Gaul <ga...@apache.org>
Authored: Sat Nov 11 07:45:23 2017 -0800
Committer: Andrew Gaul <ga...@apache.org>
Committed: Sat Nov 11 07:45:23 2017 -0800

----------------------------------------------------------------------
 .../blobstore/integration/internal/BaseBlobIntegrationTest.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/e473d7df/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 3eb2778..431dce7 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
@@ -16,6 +16,7 @@
  */
 package org.jclouds.blobstore.integration.internal;
 
+import static org.assertj.core.api.Fail.failBecauseExceptionWasNotThrown;
 import static com.google.common.base.Charsets.UTF_8;
 import static com.google.common.hash.Hashing.md5;
 import static org.assertj.core.api.Assertions.assertThat;
@@ -387,11 +388,9 @@ public class BaseBlobIntegrationTest extends BaseBlobStoreIntegrationTest {
          addObjectAndValidateContent(container, name);
          try {
             view.getBlobStore().getBlob(container, name, range(TEST_STRING.length(), TEST_STRING.length() + 1));
-            throw new AssertionError("Invalid range not caught");
+            failBecauseExceptionWasNotThrown(HttpResponseException.class);
          } catch (HttpResponseException e) {
             assertThat(e.getResponse().getStatusCode()).isEqualTo(416);
-         } catch (IllegalArgumentException e) {
-            // expected
          }
       } finally {
          returnContainer(container);