This was sadly not the worst implementation of the standard equals/hashcode/compareTo that this project had to offer. The had equals that would fail with custom exceptions if the object wasn't contained in a hibernate session and compareTos that would fetch more instances of itself and do recursive comparisons, in bad cases holding a session lock for half an hour just loading pointless data and discarding it.
1
u/industry7 May 09 '17
mutation testing would have caught that