From b3403dba5097d38581d10361874c346386669ae3 Mon Sep 17 00:00:00 2001
From: Cedric Beust <cedric@beust.com>
Date: Mon, 9 May 2016 21:56:18 -0800
Subject: [PATCH] Fix the "ping" response.

---
 .../kotlin/com/beust/kobalt/app/remote/KobaltServer.kt    | 8 ++++++++
 .../kotlin/com/beust/kobalt/app/remote/SparkServer.kt     | 4 ++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/main/kotlin/com/beust/kobalt/app/remote/KobaltServer.kt b/src/main/kotlin/com/beust/kobalt/app/remote/KobaltServer.kt
index f6b477ef..c0bcc22d 100644
--- a/src/main/kotlin/com/beust/kobalt/app/remote/KobaltServer.kt
+++ b/src/main/kotlin/com/beust/kobalt/app/remote/KobaltServer.kt
@@ -20,6 +20,14 @@ import java.util.concurrent.Callable
 class KobaltServer(val force: Boolean, val givenPort: Int? = null,
         val initCallback: (String) -> List<Project>,
         val cleanUpCallback: () -> Unit) : Callable<Int> {
+
+    companion object {
+        /**
+         * Default response sent for calls that don't return a payload.
+         */
+        val OK = "ok"
+    }
+
 //    var outgoing: PrintWriter? = null
 
     interface IServer {
diff --git a/src/main/kotlin/com/beust/kobalt/app/remote/SparkServer.kt b/src/main/kotlin/com/beust/kobalt/app/remote/SparkServer.kt
index 996d4296..57176d82 100644
--- a/src/main/kotlin/com/beust/kobalt/app/remote/SparkServer.kt
+++ b/src/main/kotlin/com/beust/kobalt/app/remote/SparkServer.kt
@@ -29,7 +29,7 @@ class SparkServer(val initCallback: (String) -> List<Project>, val cleanUpCallba
 
     override fun run(port: Int) {
         Spark.port(port)
-        Spark.get("/ping", { req, res -> "The Kobalt server is up and running" })
+        Spark.get("/ping", { req, res -> KobaltServer.OK })
         Spark.get("/quit", { req, res ->
             Executors.newFixedThreadPool(1).let { executor ->
                 executor.submit {
@@ -37,7 +37,7 @@ class SparkServer(val initCallback: (String) -> List<Project>, val cleanUpCallba
                     Spark.stop()
                     executor.shutdown()
                 }
-                "ok"
+                KobaltServer.OK
             }
         })
         Spark.get("/v0/getDependencies", "application/json", Route { request, response ->