k8s tests, style

Author: onursatici

resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesExecutorBuilder.scala

@@ -56,7 +56,8 @@ private[spark] class KubernetesExecutorBuilder(
       Seq(provideMountLocalFilesStep(kubernetesConf))
     } else Nil
 
-    val allFeatures = baseFeatures ++ secretFeature ++ secretEnvFeature ++ volumesFeature ++ localFilesFeature
+    val allFeatures = baseFeatures ++
+      secretFeature ++ secretEnvFeature ++ volumesFeature ++ localFilesFeature
 
     var executorPod = SparkPod.initialPod()
     for (feature <- allFeatures) {

resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/MountLocalFilesFeatureStepSuite.scala

@@ -64,6 +64,7 @@ class MountLocalFilesFeatureStepSuite extends SparkFunSuite with BeforeAndAfter
       Map.empty,
       Map.empty,
       Map.empty,
+      Seq.empty,
       sparkFiles)
     stepUnderTest = new MountLocalFilesFeatureStep(kubernetesConf)
   }

resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/MountVolumesFeatureStepSuite.scala

@@ -30,6 +30,7 @@ class MountVolumesFeatureStepSuite extends SparkFunSuite {
       Seq.empty),
     appResourceNamePrefix = "resource",
     appId = "app-id",
+    mountLocalFilesSecretName = None,
     roleLabels = Map.empty,
     roleAnnotations = Map.empty,
     roleSecretNamesToMountPaths = Map.empty,

resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/submit/KubernetesDriverBuilderSuite.scala

@@ -218,6 +218,7 @@ class KubernetesDriverBuilderSuite extends SparkFunSuite {
       Map.empty,
       Map.empty,
       Map.empty,
+      Seq.empty,
       allFiles)
     validateStepTypesApplied(
       builderUnderTest.buildFromFeatures(conf),
@@ -244,6 +245,7 @@ class KubernetesDriverBuilderSuite extends SparkFunSuite {
         Seq.empty),
       "prefix",
       "appId",
+      Some("secret"),
       Map.empty,
       Map.empty,
       Map.empty,

resource-managers/kubernetes/core/src/test/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesExecutorBuilderSuite.scala

@@ -106,6 +106,7 @@ class KubernetesExecutorBuilderSuite extends SparkFunSuite {
       Map.empty,
       Map.empty,
       Map.empty,
+      Seq.empty,
       Seq.empty[String])
     validateStepTypesApplied(
       builderUnderTest.buildFromFeatures(conf),
@@ -126,6 +127,7 @@ class KubernetesExecutorBuilderSuite extends SparkFunSuite {
         "executor-id", new PodBuilder().build()),
       "prefix",
       "appId",
+      Some("secret"),
       Map.empty,
       Map.empty,
       Map.empty,
@@ -137,6 +139,7 @@ class KubernetesExecutorBuilderSuite extends SparkFunSuite {
       builderUnderTest.buildFromFeatures(conf),
       BASIC_STEP_TYPE,
       LOCAL_DIRS_STEP_TYPE,
+      MOUNT_LOCAL_FILES_STEP_TYPE,
       MOUNT_VOLUMES_STEP_TYPE)
   }