In general, make all domain model classes immutable by setting the fields to 
final. Also add the proper null and other checks for all parameters in the 
constructor.

There are also no unit tests, and there should be added for:

* All custom binders.
* The custom json parser.
* Mock tests for all RemoteApi methods.
* Tests for all transformation Functions.

This is enough for a first-round review. Nice work @andreaturli! Look forward 
to seeing progress on this!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/57#issuecomment-38287947

Reply via email to