> @@ -150,7 +154,53 @@ public boolean apply(VolumeAttachmentApi
> volumeAttachmentApi) {
> if (server_id != null)
> api.getServerApiForZone(zone).delete(server_id);
> }
> + }
> + }
> +
> + @Test(dependsOnMethods = "testCreateVolume")
> + public void testAttachmentAtBoot() {
> + if (volumeApi.isPresent()) {
I notice this pattern in a lot of tests, and am in fact in favor of a general
review of where it's used and make sure to skip the tests instead of passing
them. Is it the right idea to do that here, or should I maintain consistency
(mostly to make sure that the general solution does not get missed in this
class as well)?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/326/files#r12566182
