> +             public Set<String> getIso3166Codes() {
> +                     return ImmutableSet.<String>of();
> +             }
> +             
> +             @Override
> +             public String getId() {
> +                     return "id";
> +             }
> +             
> +             @Override
> +             public String getDescription() {
> +                     return "description";
> +             }
> +     };
> +     
> +     public void testInstanceWithNullGroup() {

Wow...do we need all this setup for a single test? Is there some other test to 
which we could add something? Otherwise, perhaps also test the non-null case 
here - then at least we're hopefully getting a bit more "value for our code" 
out of all the setup you've had to create? ;-)

Formatting-wise, jclouds uses 3-space indents. Please see the [Coding 
Standards](https://wiki.apache.org/jclouds/Coding%20Standards) Wiki page.

Thanks, @najtmar!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/26/files#r13318498

Reply via email to