Tighten up test expectations

Follow on to 8b94febfeb.
This commit is contained in:
Andrew Gaul 2017-11-11 07:45:23 -08:00
parent 26570db680
commit e473d7df6a
1 changed files with 2 additions and 3 deletions

View File

@ -16,6 +16,7 @@
*/ */
package org.jclouds.blobstore.integration.internal; 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.base.Charsets.UTF_8;
import static com.google.common.hash.Hashing.md5; import static com.google.common.hash.Hashing.md5;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
@ -387,11 +388,9 @@ public class BaseBlobIntegrationTest extends BaseBlobStoreIntegrationTest {
addObjectAndValidateContent(container, name); addObjectAndValidateContent(container, name);
try { try {
view.getBlobStore().getBlob(container, name, range(TEST_STRING.length(), TEST_STRING.length() + 1)); 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) { } catch (HttpResponseException e) {
assertThat(e.getResponse().getStatusCode()).isEqualTo(416); assertThat(e.getResponse().getStatusCode()).isEqualTo(416);
} catch (IllegalArgumentException e) {
// expected
} }
} finally { } finally {
returnContainer(container); returnContainer(container);