handle exceptions (#106)

Author: sargunv

.github/workflows/ci.yml

@@ -18,9 +18,8 @@ jobs:
             os: ubuntu
           - target: jsNode
             os: ubuntu
-          # TODO: wasmJs times out on bulk tests
-          # - target: wasmJsBrowser
-          #   os: ubuntu
+          - target: wasmJsBrowser
+            os: ubuntu
           - target: wasmJsNode
             os: ubuntu
           - target: linuxX64

justfile

@@ -3,8 +3,12 @@ set windows-shell := ["C:\\Program Files\\Git\\bin\\sh.exe", "-c"]
 _default:
     just --list
 
+test-server-cmd := "npx -y http-server src/commonTest/resources/data -e json -p 8080 --cors"
+
+# Serve PokeAPI data from static files
 test-server:
-    npx -y http-server src/commonTest/resources/data -e json -p 8080 --cors
+    {{ test-server-cmd }}
 
+# Spawn a background job serving PokeAPI data from static files
 test-server-background:
-    npx -y http-server src/commonTest/resources/data -e json -p 8080 --cors &
+    {{ test-server-cmd }} &

src/appleMain/kotlin/dev/sargunv/pokekotlin/util/getDefaultEngine.apple.kt renamed to src/appleMain/kotlin/dev/sargunv/pokekotlin/internal/getDefaultEngine.apple.kt

@@ -1,4 +1,4 @@
-package dev.sargunv.pokekotlin.util
+package dev.sargunv.pokekotlin.internal
 
 import io.ktor.client.engine.darwin.Darwin
 

src/commonMain/kotlin/dev/sargunv/pokekotlin/client/PokeApi.kt renamed to src/commonMain/kotlin/dev/sargunv/pokekotlin/PokeApi.kt

@@ -1,9 +1,14 @@
-package dev.sargunv.pokekotlin.client
+@file:JvmName("PokeApiClient")
+
+package dev.sargunv.pokekotlin
 
 import de.jensklingenberg.ktorfit.Ktorfit.Builder
 import de.jensklingenberg.ktorfit.http.GET
 import de.jensklingenberg.ktorfit.http.Path
 import de.jensklingenberg.ktorfit.http.Query
+import dev.sargunv.pokekotlin.internal.PokeApiConverter
+import dev.sargunv.pokekotlin.internal.PokeApiJson
+import dev.sargunv.pokekotlin.internal.getDefaultEngine
 import dev.sargunv.pokekotlin.model.Ability
 import dev.sargunv.pokekotlin.model.ApiResourceList
 import dev.sargunv.pokekotlin.model.Berry
@@ -55,14 +60,15 @@ import dev.sargunv.pokekotlin.model.SuperContestEffect
 import dev.sargunv.pokekotlin.model.Type
 import dev.sargunv.pokekotlin.model.Version
 import dev.sargunv.pokekotlin.model.VersionGroup
-import dev.sargunv.pokekotlin.util.getDefaultEngine
 import io.ktor.client.HttpClient
 import io.ktor.client.HttpClientConfig
 import io.ktor.client.engine.HttpClientEngine
 import io.ktor.client.plugins.cache.HttpCache
 import io.ktor.client.plugins.cache.storage.CacheStorage
 import io.ktor.client.plugins.contentnegotiation.ContentNegotiation
+import io.ktor.http.ContentType
 import io.ktor.serialization.kotlinx.json.json
+import kotlin.jvm.JvmName
 
 interface PokeApi {
 
@@ -557,6 +563,7 @@ interface PokeApi {
   companion object : PokeApi by PokeApi()
 }
 
+@JvmName("create")
 fun PokeApi(
   baseUrl: String = "https://pokeapi.co/api/v2/",
   engine: HttpClientEngine = getDefaultEngine(),
@@ -567,10 +574,11 @@ fun PokeApi(
     .baseUrl(baseUrl)
     .httpClient(
       HttpClient(engine) {
-        this.configure()
-        this.install(HttpCache) { cacheStorage?.let { privateStorage(it) } }
-        this.install(ContentNegotiation) { json(PokeApiJson) }
+        configure()
+        install(HttpCache) { cacheStorage?.let { privateStorage(it) } }
+        install(ContentNegotiation) { json(PokeApiJson, ContentType.Any) }
       }
     )
+    .converterFactories(PokeApiConverter.Factory)
     .build()
     .createPokeApi()

src/commonMain/kotlin/dev/sargunv/pokekotlin/PokeApiException.kt

@@ -0,0 +1,13 @@
+package dev.sargunv.pokekotlin
+
+import io.ktor.http.HttpStatusCode
+
+sealed class PokeApiException : Throwable {
+  constructor(message: String) : super(message)
+
+  constructor(cause: Throwable) : super(cause)
+
+  class HttpStatus(val status: HttpStatusCode) : PokeApiException(status.toString())
+
+  class UnknownException(cause: Throwable) : PokeApiException(cause)
+}

src/commonMain/kotlin/dev/sargunv/pokekotlin/client/PokeApiError.kt

@@ -1,4 +1,4 @@
-package dev.sargunv.pokekotlin.util
+package dev.sargunv.pokekotlin.internal
 
 import kotlinx.serialization.KSerializer
 import kotlinx.serialization.descriptors.SerialDescriptor

src/commonMain/kotlin/dev/sargunv/pokekotlin/util/DelegatingSerializer.kt renamed to src/commonMain/kotlin/dev/sargunv/pokekotlin/internal/DelegatingSerializer.kt

@@ -0,0 +1,33 @@
+package dev.sargunv.pokekotlin.internal
+
+import de.jensklingenberg.ktorfit.Ktorfit
+import de.jensklingenberg.ktorfit.converter.Converter
+import de.jensklingenberg.ktorfit.converter.KtorfitResult
+import de.jensklingenberg.ktorfit.converter.TypeData
+import dev.sargunv.pokekotlin.PokeApiException
+import io.ktor.client.call.body
+import io.ktor.client.statement.HttpResponse
+import io.ktor.http.isSuccess
+
+internal class PokeApiConverter(private val typeData: TypeData) :
+  Converter.SuspendResponseConverter<HttpResponse, Any> {
+  override suspend fun convert(result: KtorfitResult): Any {
+    return when (result) {
+      is KtorfitResult.Failure -> throw PokeApiException.UnknownException(result.throwable)
+      is KtorfitResult.Success -> {
+        val status = result.response.status
+        when {
+          status.isSuccess() -> result.response.body<Any>(typeData.typeInfo)
+          else -> throw PokeApiException.HttpStatus(status)
+        }
+      }
+    }
+  }
+
+  internal object Factory : Converter.Factory {
+    override fun suspendResponseConverter(
+      typeData: TypeData,
+      ktorfit: Ktorfit,
+    ): Converter.SuspendResponseConverter<HttpResponse, Any> = PokeApiConverter(typeData)
+  }
+}

src/commonMain/kotlin/dev/sargunv/pokekotlin/internal/PokeApiConverter.kt

@@ -1,11 +1,11 @@
-package dev.sargunv.pokekotlin.client
+package dev.sargunv.pokekotlin.internal
 
 import kotlinx.serialization.ExperimentalSerializationApi
 import kotlinx.serialization.json.Json
 import kotlinx.serialization.json.JsonNamingStrategy
 
 @OptIn(ExperimentalSerializationApi::class)
-val PokeApiJson = Json {
+internal val PokeApiJson = Json {
   prettyPrint = true
   ignoreUnknownKeys = true
   namingStrategy = JsonNamingStrategy.SnakeCase

src/commonMain/kotlin/dev/sargunv/pokekotlin/client/PokeApiJson.kt renamed to src/commonMain/kotlin/dev/sargunv/pokekotlin/internal/PokeApiJson.kt

@@ -1,4 +1,4 @@
-package dev.sargunv.pokekotlin.util
+package dev.sargunv.pokekotlin.internal
 
 import io.ktor.client.engine.HttpClientEngine