igluctl: fix false failure message in push command (close #313)

snowplow · Jan 29, 2018 · 30434c2 · 30434c2
1 parent a9de8f8
commit 30434c2
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/0-common/igluctl/src/main/scala/com.snowplowanalytics.iglu/ctl/PushCommand.scala b/0-common/igluctl/src/main/scala/com.snowplowanalytics.iglu/ctl/PushCommand.scala
@@ -231,8 +231,9 @@ object PushCommand {
    * @param message human-readable message
    * @param location optional URI available for successful upload
    */
-  case class ServerMessage(status: Int, message: String, location: Option[String]) {
-    def asString: String =
+  case class ServerMessage(status: Int, message: String, location: Option[String])
+  object ServerMessage {
+    def asString(status: Int, message: String, location: Option[String]): String =
       s"$message ${location.map("at " + _ + " ").getOrElse("")} ($status)"
   }
 
@@ -256,7 +257,7 @@ object PushCommand {
   case class Result(serverMessage: Either[String, ServerMessage], status: Status) {
     def asString: String =
       serverMessage match {
-        case Right(message) => message.asString
+        case Right(message) => ServerMessage.asString(message.status, message.message, message.location)
         case Left(responseBody) => responseBody
       }
   }