From 84b1982060422760702eca6f1ef515c6ad3e85a5 Mon Sep 17 00:00:00 2001 From: Akira Ajisaka Date: Fri, 23 Aug 2019 14:38:50 +0900 Subject: [PATCH] YARN-9774. Fix order of arguments for assertEquals in TestSLSUtils. Contributed by Nikhil Navadiya. --- .../org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java b/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java index 74907202b1..423d6b2c7f 100644 --- a/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java +++ b/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java @@ -37,13 +37,13 @@ public class TestSLSUtils { public void testGetRackHostname() { String str = "/rack1/node1"; String rackHostname[] = SLSUtils.getRackHostName(str); - Assert.assertEquals(rackHostname[0], "rack1"); - Assert.assertEquals(rackHostname[1], "node1"); + Assert.assertEquals("rack1", rackHostname[0]); + Assert.assertEquals("node1", rackHostname[1]); str = "/rackA/rackB/node1"; rackHostname = SLSUtils.getRackHostName(str); - Assert.assertEquals(rackHostname[0], "rackA/rackB"); - Assert.assertEquals(rackHostname[1], "node1"); + Assert.assertEquals("rackA/rackB", rackHostname[0]); + Assert.assertEquals("node1", rackHostname[1]); } @Test