#5 failure message doesn't match code.

CVS_code
closed
5
2003-05-01
2003-05-01
Eric Meyer
No

I believe that the second set of assertions should be
neq.equals(eq1) etc. to match the assertion failure
messages.

In EqualsHashCodeTestCase

public final void testEqualsAgainstUnequalObjects() {

Assert.assertFalse("1st vs. not-equal",
eq1.equals(neq));

Assert.assertFalse("2nd vs. not-equal",
eq2.equals(neq));

Assert.assertFalse("3rd vs. not-equal",
eq3.equals(neq));

Assert.assertFalse("not-equal vs. 1st",
eq1.equals(neq));

Assert.assertFalse("not-equal vs. 2nd",
eq2.equals(neq));

Assert.assertFalse("not-equal vs. 3rd",
eq3.equals(neq));
}

Discussion

  • Logged In: YES
    user_id=135259

    Could you provide a patch? I'll be happy to apply it.

     
    • assigned_to: nobody --> vbossica
    • status: open --> closed