---

Ajay Kannan · Ajay Kannan · commit 0b42f8873bdb · 2016-02-11T09:22:02.000-08:00
yaml --- r: 1679 b: refs/heads/master c: bc66860 h: refs/heads/master i: 1677: ac00bc3 1675: bbe4bd1 1671: 4a2a73e 1663: 95b4bf0
diff --git a/[refs] b/[refs]
@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 9d201a05bcbb7e9f0270ddf8d4cc78bba3f80b1e
+refs/heads/master: bc66860a5283ae6e596245907739af56d904b590
 refs/heads/travis: e21ee7b88a5edc3f3d8c71f90c3fc32abf7e8dd6
 refs/heads/gh-pages: d1b373c30c176edc08692348167bec3a244bb823
 refs/heads/bigquery: 762fa5830e6c398c0396177e3e7fd243bd62cfc3
diff --git a/trunk/gcloud-java-bigquery/src/test/java/com/google/gcloud/bigquery/it/ITBigQueryTest.java b/trunk/gcloud-java-bigquery/src/test/java/com/google/gcloud/bigquery/it/ITBigQueryTest.java
@@ -197,9 +197,11 @@ public static void afterClass() throws ExecutionException, InterruptedException
     if (bigquery != null) {
       RemoteBigQueryHelper.forceDelete(bigquery, DATASET);
     }
-    if (storage != null && !RemoteGcsHelper.forceDelete(storage, BUCKET, 10, TimeUnit.SECONDS)
-        && LOG.isLoggable(Level.WARNING)) {
-      LOG.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);
+    if (storage != null) {
+      boolean wasDeleted = RemoteGcsHelper.forceDelete(storage, BUCKET, 10, TimeUnit.SECONDS);
+      if (!wasDeleted && LOG.isLoggable(Level.WARNING)) {
+        LOG.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);
+      }
     }
   }
 
diff --git a/trunk/gcloud-java-storage/src/test/java/com/google/gcloud/storage/it/ITStorageTest.java b/trunk/gcloud-java-storage/src/test/java/com/google/gcloud/storage/it/ITStorageTest.java
@@ -88,9 +88,11 @@ public static void beforeClass() {
 
   @AfterClass
   public static void afterClass() throws ExecutionException, InterruptedException {
-    if (storage != null && !RemoteGcsHelper.forceDelete(storage, BUCKET, 5, TimeUnit.SECONDS)
-        && log.isLoggable(Level.WARNING)) {
-      log.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);
+    if (storage != null) {
+      boolean wasDeleted = RemoteGcsHelper.forceDelete(storage, BUCKET, 5, TimeUnit.SECONDS);
+      if (!wasDeleted && log.isLoggable(Level.WARNING)) {
+        log.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);
+      }
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -197,9 +197,11 @@ public static void afterClass() throws ExecutionException, InterruptedException`
`197`	`197`	`if (bigquery != null) {`
`198`	`198`	`RemoteBigQueryHelper.forceDelete(bigquery, DATASET);`
`199`	`199`	`}`
`200`		`- if (storage != null && !RemoteGcsHelper.forceDelete(storage, BUCKET, 10, TimeUnit.SECONDS)`
`201`		`- && LOG.isLoggable(Level.WARNING)) {`
`202`		`- LOG.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);`
	`200`	`+ if (storage != null) {`
	`201`	`+ boolean wasDeleted = RemoteGcsHelper.forceDelete(storage, BUCKET, 10, TimeUnit.SECONDS);`
	`202`	`+ if (!wasDeleted && LOG.isLoggable(Level.WARNING)) {`
	`203`	`+ LOG.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);`
	`204`	`+ }`
`203`	`205`	`}`
`204`	`206`	`}`
`205`	`207`
Original file line number	Diff line number	Diff line change
`@@ -88,9 +88,11 @@ public static void beforeClass() {`
`88`	`88`
`89`	`89`	`@AfterClass`
`90`	`90`	`public static void afterClass() throws ExecutionException, InterruptedException {`
`91`		`- if (storage != null && !RemoteGcsHelper.forceDelete(storage, BUCKET, 5, TimeUnit.SECONDS)`
`92`		`- && log.isLoggable(Level.WARNING)) {`
`93`		`- log.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);`
	`91`	`+ if (storage != null) {`
	`92`	`+ boolean wasDeleted = RemoteGcsHelper.forceDelete(storage, BUCKET, 5, TimeUnit.SECONDS);`
	`93`	`+ if (!wasDeleted && log.isLoggable(Level.WARNING)) {`
	`94`	`+ log.log(Level.WARNING, "Deletion of bucket {0} timed out, bucket is not empty", BUCKET);`
	`95`	`+ }`
`94`	`96`	`}`
`95`	`97`	`}`
`96`	`98`