refactor: apply DRY principle to Elasticsearch configuration

- Move default environment variables to ElasticsearchController.DefaultEnvVars
- Remove duplicate environment variable configuration from test code
- Use ping() method for connection test which works correctly with Elasticsearch 9.1.3
- Format code with scalafmt

Author: j5ik2o

build.sbt

@@ -236,8 +236,8 @@ val `docker-controller-scala-elasticsearch` = (project in file("docker-controlle
     libraryDependencies ++= Seq(
       scalatest.scalatest          % Test,
       logback.classic              % Test,
-      "co.elastic.clients"         % "elasticsearch-java"        % "9.1.4" % Test,
-      "org.elasticsearch.client"   % "elasticsearch-rest-client" % "9.1.3" % Test,
+      "co.elastic.clients"         % "elasticsearch-java"        % "9.1.4"  % Test,
+      "org.elasticsearch.client"   % "elasticsearch-rest-client" % "9.1.3"  % Test,
       "com.fasterxml.jackson.core" % "jackson-databind"          % "2.20.0" % Test,
       "org.apache.logging.log4j"   % "log4j-api"                 % "2.25.1" % Test,
       "org.apache.logging.log4j"   % "log4j-core"                % "2.25.1" % Test

docker-controller-scala-elasticsearch/src/main/scala/com/github/j5ik2o/dockerController/elasticsearch/ElasticsearchController.scala

@@ -14,13 +14,20 @@ object ElasticsearchController {
   final val DefaultImageTag: Option[String] = Some("9.1.3")
   final val DefaultContainerPorts: Seq[Int] = Seq(9200, 9300)
 
+  final val DefaultEnvVars: Map[String, String] = Map(
+    "discovery.type"                  -> "single-node",
+    "xpack.security.enabled"          -> "false",
+    "xpack.security.http.ssl.enabled" -> "false",
+    "ES_JAVA_OPTS"                    -> "-Xms512m -Xmx512m"
+  )
+
   def apply(
       dockerClient: DockerClient,
       isDockerClientAutoClose: Boolean = false,
       outputFrameInterval: FiniteDuration = 500.millis,
       imageName: String = DefaultImageName,
       imageTag: Option[String] = DefaultImageTag,
-      envVars: Map[String, String] = Map.empty
+      envVars: Map[String, String] = DefaultEnvVars
   )(
       hostPort1: Int,
       hostPort2: Int

docker-controller-scala-elasticsearch/src/test/scala/com/github/j5ik2o/dockerController/elasticsearch/ElasticsearchControllerSpec.scala

@@ -15,17 +15,9 @@ class ElasticsearchControllerSpec extends AnyFreeSpec with DockerControllerSpecS
   val testTimeFactor: Int = sys.env.getOrElse("TEST_TIME_FACTOR", "1").toInt
   logger.debug(s"testTimeFactor = $testTimeFactor")
 
-  val hostPort1: Int = temporaryServerPort()
-  val hostPort2: Int = temporaryServerPort()
-  val controller: ElasticsearchController = ElasticsearchController(
-    dockerClient,
-    envVars = Map(
-      "discovery.type"                  -> "single-node",
-      "xpack.security.enabled"          -> "false",
-      "xpack.security.http.ssl.enabled" -> "false",
-      "ES_JAVA_OPTS"                    -> "-Xms512m -Xmx512m"
-    )
-  )(hostPort1, hostPort2)
+  val hostPort1: Int                      = temporaryServerPort()
+  val hostPort2: Int                      = temporaryServerPort()
+  val controller: ElasticsearchController = ElasticsearchController(dockerClient)(hostPort1, hostPort2)
 
   override protected val dockerControllers: Vector[DockerController] = Vector(controller)
 
@@ -56,9 +48,9 @@ class ElasticsearchControllerSpec extends AnyFreeSpec with DockerControllerSpecS
           new JacksonJsonpMapper()
         )
         val esClient = new ElasticsearchClient(transport)
-        // Use info() instead of ping() for compatibility
-        val info = esClient.info()
-        assert(info.clusterName() != null)
+        // Test connection with ping
+        val pingResult = esClient.ping()
+        assert(pingResult.value())
       } finally {
         if (transport != null)
           transport.close()