In message <aanlktimrq6wh546m+kxvusvgswrux4mu5eqhqgp5a...@mail.gmail.com>,
Jesse Wilson writes:
>
> [snip]
> 
> It comes up less frequently, but I have some other preferences related
> to better detail messages. When comparing arrays, equals() does the
> wrong thing. To work around, I compare the string form for primitives:
> 
> byte[] actual = ...
> byte[] expected = ...
> assertEquals(Arrays.toString(actual), Arrays.toString(expected));
> 
> And the list form for reference types:
> 
> Object[] actual = ...
> Object[] expected = ...
> assertEquals(Arrays.asList(actual), Arrays.asList(expected));

+1 - expected comes first though.

> I'm not actually planning on doing massive scrubbing to change styles
> or anything. But when I investigate specific problematic tests, it's
> helpful to fix the test to tell me everything that it can.

+1.  I tend to fix them if I see any obvious ones on the commit list but
sometimes I look at the file and decide it's just too much work for the
time available.

Regards,
 Mark.


Reply via email to