Merge pull request #13 from linked-planet/bug/ZITIS-887_improve_error_messages

improve error messages

Author: HighKo

kotlin-insight-client/kotlin-insight-client-api/src/main/kotlin/com/linkedplanet/kotlininsightclient/api/error/InsightClientError.kt

@@ -44,19 +44,17 @@ import io.swagger.v3.oas.annotations.media.Schema
 @Suppress("unused")
 sealed class InsightClientError(
     val error: String,
-    val message: String
+    val message: String,
+    val stacktrace: String = ""
 ) {
-    val stacktrace: String = Exception(message).stackTraceToString()
 
     companion object {
         private const val internalErrorString = "Jira/Insight hat ein internes Problem festgestellt"
-        fun fromException(e: Exception): InsightClientError =
-            ExceptionInsightClientError(e.message ?: internalErrorString, e.stackTraceToString())
         fun fromException(e: Throwable): InsightClientError =
-            ExceptionInsightClientError(e.message ?: "Interner Fehler", e.stackTraceToString())
+            ExceptionInsightClientError("Insight-Fehler", e.message ?: internalErrorString, e.stackTraceToString())
 
         fun internalError(message: String): Either<InsightClientError, InsightAttribute> =
-            InternalInsightClientError("Interner Fehler", message).asEither()
+            InternalInsightClientError("Interner Insight-Fehler", message).asEither()
 
     }
 }
@@ -65,15 +63,15 @@ fun <T> InsightClientError.asEither(): Either<InsightClientError, T> = Either.Le
 class InvalidArgumentInsightClientError(message: String) : InsightClientError("Unerwarteter Parameter", message)
 
 class InternalInsightClientError(error: String, message: String) : InsightClientError(error, message)
-class ExceptionInsightClientError(error: String, message: String) : InsightClientError(error, message)
+class ExceptionInsightClientError(error: String, message: String, stacktrace: String) : InsightClientError(error, message, stacktrace)
 
 class AuthenticationError(message: String) : InsightClientError("Authentifizierung fehlgeschlagen", message)
 
 class ObjectNotFoundError(val objectId: InsightObjectId):
-    InsightClientError("Objekt nicht gefunden", "Das Objekt mit der InsightObjectId=$objectId wurde nicht gefunden.")
+    InsightClientError("Insight Objekt nicht gefunden", "Das Objekt mit der InsightObjectId=$objectId wurde nicht gefunden.")
 
 class ObjectTypeNotFoundError(val rootObjectTypeId: InsightObjectTypeId) :
-    InsightClientError("Objekttyp unbekannt", "Der Objekttyp mit der angegebenen InsightObjectTypeId=$rootObjectTypeId wurde nicht gefunden.")
+    InsightClientError("Insight Objekttyp unbekannt", "Der Objekttyp mit der angegebenen InsightObjectTypeId=$rootObjectTypeId wurde nicht gefunden.")
 
 class OtherNotFoundError(message: String) : InsightClientError("Nicht gefunden.", message)
 

kotlin-jira-client/kotlin-jira-client-api/src/main/kotlin/com/linkedplanet/kotlinjiraclient/api/error/JiraClientError.kt

@@ -22,9 +22,9 @@ package com.linkedplanet.kotlinjiraclient.api.error
 @Suppress("unused")
 open class JiraClientError(
     val error: String,
-    val message: String
+    val message: String,
+    val stacktrace: String = ""
 ) {
-    val stacktrace: String = Exception(message).stackTraceToString()
 
     companion object
 }

kotlin-jira-client/kotlin-jira-client-http/src/main/kotlin/com/linkedplanet/kotlinjiraclient/http/util/JiraClientErrorExtension.kt

@@ -23,4 +23,4 @@ import com.linkedplanet.kotlinhttpclient.error.HttpDomainError
 import com.linkedplanet.kotlinjiraclient.api.error.JiraClientError
 
 fun JiraClientError.Companion.fromHttpDomainError(e: HttpDomainError): JiraClientError =
-    JiraClientError(e.error, e.message)
+    JiraClientError(e.error, e.message, "HttpStatusCode-${e.statusCode}")

kotlin-jira-client/kotlin-jira-client-sdk/src/main/kotlin/com/linkedplanet/kotlinjiraclient/sdk/util/EitherExtension.kt

@@ -27,5 +27,9 @@ inline fun <B> Either.Companion.catchJiraClientError(
     message: String? = null,
     f: () -> B
 ): Either<JiraClientError, B> = catch(f).mapLeft {
-    JiraClientError(error ?: it.message.toString(), message ?: it.stackTraceToString())
+    JiraClientError(
+        error = error ?: "Jira-Fehler",
+        message = message ?: it.message ?: "-",
+        stacktrace = it.stackTraceToString()
+    )
 }