[
https://issues.apache.org/jira/browse/HADOOP-12693?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16905364#comment-16905364
]
Adam Antal commented on HADOOP-12693:
-------------------------------------
[~templedf] Oh indeed, I thought those were already done.
As I checked some of the occurrences in the just-rough-approx.txt some of those
entries are valid, and still present in hadoop-common, hadoop-hdfs and
hadoop-tools projects. Filed HADOOP-16510, HADOOP-16511 and HADOOP-16512. Also
collected them under this jira as sub-tasks for better trackability. Hope you
don't mind.
> Many misusages of assertEquals(expected, actual)
> ------------------------------------------------
>
> Key: HADOOP-12693
> URL: https://issues.apache.org/jira/browse/HADOOP-12693
> Project: Hadoop Common
> Issue Type: Test
> Reporter: Akihiro Suda
> Priority: Trivial
> Attachments: just-rough-approx.txt
>
>
> The first arg of {{org.JUnit.Assert.assertEquals()}} should be an
> {{expected}} value, and the second one should be an {{actual}} value.
> {code}
> void assertEquals(T expected, T actual);
> {code}
> http://junit.org/apidocs/org/junit/Assert.html#assertEquals(java.lang.Object,
> java.lang.Object)
> However, there are so many violations in Hadoop, which can make a misleading
> message like this:
> {code}
> AssertionError: expected:<actual> but was:<expected>
> {code}.
> Please refer to {{just-rough-approx.txt}}.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
