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)