Updated tests to use Equality in asserts, removed unnecessary string …

…interpolation
apache · Aug 20, 2019 · c8b9aaf · c8b9aaf
1 parent dc07e09
commit c8b9aaf
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/...n/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala b/...n/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
@@ -270,7 +270,7 @@ private[spark] class MesosCoarseGrainedSchedulerBackend(
         }
       val executable = new File(executorSparkHome, "./bin/spark-class").getPath
       val runScript = s"$prefixEnv $executable " +
-        s"org.apache.spark.executor.CoarseGrainedExecutorBackend"
+        "org.apache.spark.executor.CoarseGrainedExecutorBackend"
 
       command.setValue(buildExecutorCommand(runScript, taskId, numCores, offer))
     } else {

diff --git a/.../test/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtilSuite.scala b/.../test/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtilSuite.scala
@@ -33,7 +33,7 @@ class MesosSchedulerBackendUtilSuite extends SparkFunSuite {
       conf)
     val params = containerInfo.getDocker.getParametersList
 
-    assert(params.size() == 0)
+    assert(params.size() === 0)
   }
 
   test("ContainerInfo parses docker parameters") {
@@ -44,13 +44,13 @@ class MesosSchedulerBackendUtilSuite extends SparkFunSuite {
     val containerInfo = MesosSchedulerBackendUtil.buildContainerInfo(
       conf)
     val params = containerInfo.getDocker.getParametersList
-    assert(params.size() == 3)
-    assert(params.get(0).getKey == "a")
-    assert(params.get(0).getValue == "1")
-    assert(params.get(1).getKey == "b")
-    assert(params.get(1).getValue == "2")
-    assert(params.get(2).getKey == "c")
-    assert(params.get(2).getValue == "3")
+    assert(params.size() === 3)
+    assert(params.get(0).getKey === "a")
+    assert(params.get(0).getValue === "1")
+    assert(params.get(1).getKey === "b")
+    assert(params.get(1).getValue === "2")
+    assert(params.get(2).getKey === "c")
+    assert(params.get(2).getValue === "3")
   }
 
   test("ContainerInfo respects Docker network configuration") {
@@ -62,10 +62,10 @@ class MesosSchedulerBackendUtilSuite extends SparkFunSuite {
 
     val containerInfo = MesosSchedulerBackendUtil.buildContainerInfo(conf)
 
-    assert(containerInfo.getDocker.getNetwork == DockerInfo.Network.USER)
+    assert(containerInfo.getDocker.getNetwork === DockerInfo.Network.USER)
     val params = containerInfo.getDocker.getParametersList
-    assert(params.size() == 1)
-    assert(params.get(0).getKey == "net")
-    assert(params.get(0).getValue == networkName)
+    assert(params.size() === 1)
+    assert(params.get(0).getKey === "net")
+    assert(params.get(0).getValue === networkName)
   }
 }