> +               imageArchNull, Functions.<TemplateBuilderImpl>identity());
> +   }
> +
> +   public void testResolveImagesCustomSorterPreferringNonNull() {
> +      final Ordering<Image> sorterPreferringNonNullArch = new 
> Ordering<Image>() {
> +         @Override
> +         public int compare(Image left, Image right) {
> +            return ComparisonChain.start()
> +                   .compare(left.getOperatingSystem().getArch(), 
> right.getOperatingSystem().getArch(),
> +                       Ordering.<String> natural().nullsFirst())
> +                   .compare(left, right, 
> TemplateBuilderImpl.DEFAULT_IMAGE_ORDERING)
> +                   .result();
> +         }
> +      };
> +      
> Assert.assertTrue(TemplateBuilderImpl.DEFAULT_IMAGE_ORDERING.compare(image64bit,
>  imageArchNull) < 0);
> +      Assert.assertTrue(sorterPreferringNonNullArch.compare(image64bit, 
> imageArchNull) > 0);

Static import of `Assert.assertTrue`? And add a descriptive message?

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

Reply via email to