[HDFS-14897] [hadoop-hdfs] Fix order of actual and expected expression in assert statements

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 3.2.0
• Target Version/s: 3.5.0
• Assignee: Unassigned
• Reporter: Adam Antal
• Created: 12/Aug/19 16:39
• Updated: 04/Jan/24 08:13

Description

Fix order of actual and expected expression in assert statements which gives misleading message when test case fails. Attached file has some of the places where it is placed wrongly.

[ERROR] testNodeRemovalGracefully(org.apache.hadoop.yarn.server.resourcemanager.TestResourceTrackerService)  Time elapsed: 3.385 s  <<< FAILURE!
java.lang.AssertionError: Shutdown nodes should be 0 now expected:<1> but was:<0>

For long term, AssertJ can be used for new test cases which avoids such mistakes.

This is a follow-up jira for the hadoop-hdfs project.

Issue Links

• is a child of: HADOOP-12693 Many misusages of assertEquals(expected, actual)
• relates to: HADOOP-12693 Many misusages of assertEquals(expected, actual)
• links to: GitHub Pull Request #1539