https://issues.apache.org/bugzilla/show_bug.cgi?id=51497
--- Comment #3 from Sebb <s...@apache.org> --- 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: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org