> + expected = byteSource.openStream();
> + assertThat(actual).hasContentEqualTo(expected);
> + } finally {
> + Closeables2.closeQuietly(expected);
> + Closeables2.closeQuietly(actual);
> + }
> + } finally {
> + if (uploadFile != null) {
> + objectApi.deleteFileVersion(uploadFile.fileName(),
> uploadFile.fileId());
> + }
> + bucketApi.deleteBucket(response.bucketId());
> + }
> + }
> +
> + @Test(groups = "live")
> + public void testDownloadFileByNameNonExistent() throws Exception {
In general, I don't think we need to "live" test stuff like this. We are
creating unnecessary resources just to validate a jclouds behavior that does
not give us any value of "how aligned with the provider api we are". The
purpose of this test is to verify that we propagate and exception, and that is
alreaydy covered in the mock tests.
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/276/files/d27eef19e1ef39d560170786aa7a9ef1f711b71b#r65691650
