jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Phillips <notificati...@github.com>
Subject Re: [jclouds/jclouds] Missing cleanup was causing testReplaceManifest to fail (#1038)
Date Tue, 15 Nov 2016 01:53:36 GMT
demobox commented on this pull request.



> @@ -117,6 +118,7 @@ public void testDeleteSinglePartObjectWithMultiPartDelete() throws
Exception {
          assertThat(resp.deleted()).isZero();
          assertThat(resp.notFound()).isZero();
          assertThat(resp.errors()).hasSize(1);
+         api.getObjectApi(regionId, containerName).delete(objectName);

@zack-shoylev Is it the intention that this only be called if the assertions succeed?

-- 
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/pull/1038#pullrequestreview-8524607
Mime
View raw message