jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject Re: [jclouds-labs] Profitbricks REST - Snapshot API (#244)
Date Fri, 04 Mar 2016 11:10:44 GMT
> +   }
> +   
> +   @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
Mime
View raw message