This looks good (once the `listEnvironmentNodes`) test is fixed.

Also, would it make sense to add `listEnvironmentNodes` a method to the 
[ChefService](https://github.com/jclouds/jclouds-chef/blob/master/core/src/main/java/org/jclouds/chef/ChefService.java)
 to get the entire node objects (not only the name) for a given environment? 
Something similar to what the 
[ChefService#listNodes](https://github.com/jclouds/jclouds-chef/blob/master/core/src/main/java/org/jclouds/chef/ChefService.java#L150-L154)
 methods do.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/12#issuecomment-23772619

Reply via email to