Skip to content

Commit

Permalink
Move httpclient configuration into the config class and add (ignored)…
Browse files Browse the repository at this point in the history
… tests
  • Loading branch information
Bdegraaf1234 committed Jun 5, 2024
1 parent 5fd8819 commit 1653bb3
Show file tree
Hide file tree
Showing 6 changed files with 195 additions and 28 deletions.
12 changes: 6 additions & 6 deletions buildSrc/src/main/kotlin/Versions.kt
Original file line number Diff line number Diff line change
Expand Up @@ -14,17 +14,17 @@ object Versions {
}

const val java = 17
const val slf4j = "2.0.9"
const val confluent = "7.5.0"
const val kafka = "7.5.0-ce"
const val slf4j = "2.0.13"
const val confluent = "7.6.0"
const val kafka = "${confluent}-ce"
const val avro = "1.11.3"
const val jackson = "2.15.2"
const val okhttp = "4.11.0"
const val jackson = "2.15.3"
const val okhttp = "4.12.0"
const val junit = "5.10.0"
const val mockito = "5.5.0"
const val mockitoKotlin = "5.1.0"
const val hamcrest = "2.2"
const val radarSchemas = "0.8.4"
const val radarSchemas = "0.8.8"
const val opencsv = "5.8"
const val ktor = "2.3.4"
const val coroutines = "1.7.3"
Expand Down
15 changes: 7 additions & 8 deletions radar-commons-gradle/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -190,17 +190,17 @@ object Versions {
}

const val java = 17
const val slf4j = "2.0.9"
const val confluent = "7.5.0"
const val kafka = "7.5.0-ce"
const val slf4j = "2.0.13"
const val confluent = "7.6.0"
const val kafka = "${confluent}-ce"
const val avro = "1.11.3"
const val jackson = "2.15.2"
const val okhttp = "4.11.0"
const val junit = "5.10.0"
const val jackson = "2.15.3"
const val okhttp = "4.12.0"
const val junit = "5.10.3"
const val mockito = "5.5.0"
const val mockitoKotlin = "5.1.0"
const val hamcrest = "2.2"
const val radarSchemas = "0.8.4"
const val radarSchemas = "0.8.8"
const val opencsv = "5.8"
const val ktor = "2.3.4"
const val coroutines = "1.7.3"
Expand All @@ -210,4 +210,3 @@ object Versions {
const val gradleVersionsPlugin = "0.50.0"
const val ktlint = "12.0.3"
}

Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,10 @@ package org.radarbase.producer.schema

import io.ktor.client.HttpClient
import io.ktor.client.call.body
import io.ktor.client.plugins.auth.*
import io.ktor.client.plugins.auth.providers.*
import io.ktor.client.plugins.contentnegotiation.ContentNegotiation
import io.ktor.client.plugins.defaultRequest
import io.ktor.client.request.HttpRequestBuilder
import io.ktor.client.request.accept
import io.ktor.client.request.request
import io.ktor.client.request.setBody
import io.ktor.client.request.url
import io.ktor.client.request.*
import io.ktor.http.ContentType
import io.ktor.http.HttpMethod
import io.ktor.http.contentType
Expand All @@ -21,13 +18,15 @@ import kotlinx.coroutines.withContext
import kotlinx.serialization.json.Json
import org.apache.avro.Schema
import org.radarbase.producer.rest.RestException.Companion.toRestException
import org.slf4j.LoggerFactory
import java.io.IOException
import java.net.URI
import kotlin.coroutines.CoroutineContext

/** REST client for Confluent schema registry. */
class SchemaRestClient(
httpClient: HttpClient,
baseUrl: String,
private val baseUrl: String,
private val ioContext: CoroutineContext = Dispatchers.IO,
) {
private val httpClient: HttpClient = httpClient.config {
Expand All @@ -39,10 +38,6 @@ class SchemaRestClient(
},
)
}
defaultRequest {
url(baseUrl)
accept(ContentType.Application.Json)
}
}

suspend inline fun <reified T> request(
Expand Down Expand Up @@ -88,7 +83,7 @@ class SchemaRestClient(
@Throws(IOException::class)
suspend fun schemaGet(path: String): SchemaMetadata = request {
method = HttpMethod.Get
url(path)
url(URI(baseUrl).resolve(path).toString())
}

@Throws(IOException::class)
Expand All @@ -97,7 +92,7 @@ class SchemaRestClient(
schema: Schema,
): SchemaMetadata = request {
method = HttpMethod.Post
url(path)
url(URI(baseUrl).resolve(path).toString())
contentType(ContentType.Application.Json)
setBody(SchemaMetadata(schema = schema.toString()))
}
Expand Down Expand Up @@ -132,4 +127,8 @@ class SchemaRestClient(
schemaGet("/schemas/ids/$id")
.toParsedSchemaMetadata(id)
.schema

companion object {
private val logger = LoggerFactory.getLogger(SchemaRestClient::class.java)
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 +205,8 @@ open class SchemaRetriever(config: Config) {
@RadarProducerDsl
class Config(
val baseUrl: String,
var httpClient: HttpClient? = null,
) {
var httpClient: HttpClient? = null
var schemaTimeout: CacheConfig = DEFAULT_SCHEMA_TIMEOUT_CONFIG
var ioContext: CoroutineContext = Dispatchers.IO
fun httpClient(config: HttpClientConfig<*>.() -> Unit) {
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,83 @@
// /*
// * Copyright 2017 The Hyve and King's College London
// *
// * Licensed under the Apache License, Version 2.0 (the "License");
// * you may not use this file except in compliance with the License.
// * You may obtain a copy of the License at
// *
// * http://www.apache.org/licenses/LICENSE-2.0
// *
// * Unless required by applicable law or agreed to in writing, software
// * distributed under the License is distributed on an "AS IS" BASIS,
// * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// * See the License for the specific language governing permissions and
// * limitations under the License.
// */
// package org.radarbase.producer.schema
//
// import io.ktor.client.*
// import io.ktor.client.engine.cio.*
// import io.ktor.client.plugins.auth.*
// import io.ktor.client.plugins.auth.providers.*
// import io.ktor.client.request.*
// import io.ktor.client.statement.*
// import io.ktor.http.*
// import kotlinx.coroutines.runBlocking
// import org.apache.avro.Schema
// import org.junit.Ignore
// import org.junit.jupiter.api.AfterEach
// import org.junit.jupiter.api.BeforeEach
// import org.junit.jupiter.api.Test
// import org.radarbase.producer.io.timeout
// import java.io.IOException
// import kotlin.time.Duration.Companion.seconds
//
// class SchemaRestClientIntegrationTest {
// private lateinit var retriever: SchemaRestClient
//
// // This testclass requires a functioning confluent cloud provider, therefore all tests here are ignored
//
// @BeforeEach
// fun setUp() {
// val apiSecret = "exampleSecret"
// val apiKey = "exampleKey"
// retriever = SchemaRestClient(
// httpClient = HttpClient {
// timeout(30.seconds)
// install(Auth) {
// basic {
// sendWithoutRequest { true }
// credentials {
// BasicAuthCredentials(username = apiKey, password = apiSecret)
// }
// }
// }
// },
// baseUrl = "https://SOME_EXAMPLE.westeurope.azure.confluent.cloud",
// )
// }
//
// @AfterEach
// @Throws(IOException::class)
// fun tearDown() {
// }
//
// @Test
// fun testSchemaGet() = runBlocking {
// val actualSchemaMetadata = retriever.schemaGet("/schemas/ids/100042")
// println(actualSchemaMetadata)
// }
//
// @Test
// fun testSchemaPost() = runBlocking {
// val postResult = retriever.schemaPost("subjects/d90c8b88-5793-438a-b27d-6c87580cc3d9", Schema.create(Schema.Type.STRING))
//
// println(postResult)
// }
//
// @Test
// fun testRetrieveSchemaById() = runBlocking {
// val res = retriever.retrieveSchemaById(100086)
// println(res)
// }
// }
Original file line number Diff line number Diff line change
@@ -0,0 +1,86 @@
// /*
// * Copyright 2017 The Hyve and King's College London
// *
// * Licensed under the Apache License, Version 2.0 (the "License");
// * you may not use this file except in compliance with the License.
// * You may obtain a copy of the License at
// *
// * http://www.apache.org/licenses/LICENSE-2.0
// *
// * Unless required by applicable law or agreed to in writing, software
// * distributed under the License is distributed on an "AS IS" BASIS,
// * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// * See the License for the specific language governing permissions and
// * limitations under the License.
// */
// package org.radarbase.producer.schema
//
// import io.ktor.client.*
// import io.ktor.client.engine.cio.*
// import io.ktor.client.plugins.auth.*
// import io.ktor.client.plugins.auth.providers.*
// import io.ktor.client.request.*
// import io.ktor.client.statement.*
// import io.ktor.http.*
// import kotlinx.coroutines.runBlocking
// import org.junit.Ignore
// import org.junit.jupiter.api.AfterEach
// import org.junit.jupiter.api.BeforeEach
// import org.junit.jupiter.api.Test
// import org.radarbase.producer.io.timeout
// import java.io.IOException
// import kotlin.time.Duration.Companion.seconds
//
// class SchemaRetrieverIntegrationTest {
// private lateinit var retriever: SchemaRetriever
//
// // This testclass requires a functioning confluent cloud provider, therefore all tests here are ignored
//
// @BeforeEach
// fun setUp() {
// val apiSecret = "exampleSecret"
// val apiKey = "exampleKey"
// retriever = SchemaRetriever.schemaRetriever(baseUrl = "https://SOME_EXAMPLE.westeurope.azure.confluent.cloud") {
// httpClient = HttpClient(CIO) {
// timeout(30.seconds)
// install(Auth) {
// basic {
// sendWithoutRequest { true }
// credentials {
// BasicAuthCredentials(username = apiKey, password = apiSecret)
// }
// }
// }
// }
// }
//
// val restClient = SchemaRestClient(
// httpClient = HttpClient {
// timeout(30.seconds)
// install(Auth) {
// basic {
// sendWithoutRequest { true }
// credentials {
// BasicAuthCredentials(username = apiKey, password = apiSecret)
// }
// }
// }
// },
// baseUrl = "https://SOME_EXAMPLE.westeurope.azure.confluent.cloud",
// )
// }
//
// @AfterEach
// @Throws(IOException::class)
// fun tearDown() {
// }
//
// @Test
// fun testGetById() = runBlocking {
// val topic = "android_phone_acceleration"
// val id = 100042
// val ofValue = true
// val res = retriever.getById(topic, ofValue, id)
// println(res)
// }
// }

0 comments on commit 1653bb3

Please sign in to comment.