https://issues.apache.org/bugzilla/show_bug.cgi?id=51497

--- Comment #3 from Sebb <[email protected]> ---
Minor nit: the tests use "assertTrue(result.equals(expected))" throughout.
This will detect errors, but won't show what any detail if anything goes wrong.

It would be better to use "assertEquals(expected, result)" as that shows the
actual result if it differs from expected. For checking against null, use
assertNull().

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to