You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Miguel Munoz (Jira)" <ji...@apache.org> on 2020/06/10 11:55:00 UTC

[jira] [Commented] (LANG-1499) Equals transitivity is violated in EqualsBuilder

    [ https://issues.apache.org/jira/browse/LANG-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17130575#comment-17130575 ] 

Miguel Munoz commented on LANG-1499:
------------------------------------

This is a valid issue, but the test case has problems. It relies on an equals method that doesn't use EqualsBuilder. Here's a better test case:  [^EqualTransitivityTest2.java] 

> Equals transitivity is violated in EqualsBuilder
> ------------------------------------------------
>
>                 Key: LANG-1499
>                 URL: https://issues.apache.org/jira/browse/LANG-1499
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.builder.*
>    Affects Versions: 3.9
>         Environment: Ubuntu 18.04
> JDK 1.8.0_221
> Commons Lang 3.9-RC2
> JUnit 5.4
>            Reporter: Zhiqiang Zang
>            Priority: Major
>              Labels: Equals(), EqualsBuilder, transitivity
>         Attachments: EqualTransitivityTest2.java, EqualsTransitivityTest.java
>
>
> EqualsBuilder.reflectionEquals() does not hold transitivity when comparing two subclasses extending a common superclass. For example:
> Given that both class D and E are subclasses of class C, C == D and C == E should imply D == E. However EqualsBuilder.reflectionEquals(D, E) returns *false* when both EqualsBuilder.reflectionEquals(C, D) and EqualsBuilder.reflectionEquals(C, E) return true.
> A junit test is provided as attachment.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)