> +   }
> +   
> +   @Test(dependsOnMethods = "testUpdateSnapshot")
> +   public void testDeleteSnapshot() {
> +      api.volumeApi().deleteVolume(testVolume.dataCenterId(), 
> testVolume.id());
> +      assertVolumeRemoved(testVolume);
> +      snapshotApi().deleteSnapshot(testSnapshot.id());
> +      assertSnapshotRemoved(testSnapshot);
> +   }
> +   
> +   private SnapshotApi snapshotApi() {
> +      return api.snapshotApi();
> +   }
> +   
> +   private void assertVolumeAvailable(Volume volume) {
> +      assertRandom(new Predicate<VolumeRef>() {

Just seen this... Would it be more convenient for this method to be named 
`assertPredicate`? The current name seems to mean that it will perform 
arbitrary assertions...

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

Reply via email to