> @@ -422,8 +425,9 @@ public void testDeleteObjectWithOptions() {
>  
>     @AfterClass
>     private void deleteBucket() {
> -      api.getBucketApi().deleteBucket(BUCKET_NAME);
> -      api.getBucketApi().deleteBucket(BUCKET_NAME2);
> +      assertTrue(api.getBucketApi().deleteBucket(BUCKET_NAME));
> +      assertTrue(api.getBucketApi().deleteBucket(BUCKET_NAME2));
> +      assertFalse(api.getBucketApi().deleteBucket(NONEXISTENT_BUCKET_NAME));

Should `@AfterClass` have asserts?  This method should just clean up after the 
test mehod.  Instead can you create a separate `testDeleteBucket` method?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/122/files#r27321367

Reply via email to