> @@ -132,6 +132,45 @@ public void testCreateWithNetworkOptions() {
>        }
>     }
>  
> +   /**
> +    * This test creates a new server with a boot device from an image.
> +    *
> +    * This needs to be supported by the provider, and is usually not 
> supported.
> +    *
> +    * TODO: Configurable system properties for flavor/image refs.
> +    */
> +   @Test(enabled = false)
> +   public void testCreateWithBlockDeviceMapping() {
> +      String serverId = null;

I'd prefer to see this guarded with an extensionApi check on the namespace. 
disabled tests are much more likely to become debt. I understand that the 
hard-coded ids below might not work even with that in place, so at least put a 
TODO: guard with extensionApi on os-block-device-mapping-v2-boot or throw 
SkipException

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

Reply via email to