> @@ -189,7 +188,7 @@ public void testNoDescription() {
> @Test
> public void testId() {
> ProductItemPrice price = ProductItemPrice.builder().id(1234).build();
Actually, as far as I can see from [the
code](https://github.com/jclouds/jclouds/blob/master/providers/softlayer/src/main/java/org/jclouds/softlayer/compute/functions/ProductItemToImage.java#L182),
the ID that is returned is that of _the ProductItemPrice_, **not** the
_ProductItem_. If that is the intention, this test should remain unchanged or
even be something like
```
ProductItemPrice price = ProductItemPrice.builder().id(1234).build();
ProductItem item = ProductItem.builder().id(2345).prices(price).build();
assertEquals("1234", imageId().apply(item), "Expected the ID of the
ProductItemPrice to be returned");
```
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/77/files#r5494191
