Adds tests

Author: ArnyminerZ

app/src/androidTest/java/at/bitfire/icsdroid/MockServer.kt

@@ -21,8 +21,14 @@ object MockServer {
 
     private val queue = mutableListOf<Response>()
 
+    var lastRequestHeaders: Headers? = null
+        private set
+
     val createMockEngine: (CustomCertManager, SSLContext) -> HttpClientEngine = { _, _ ->
-        MockEngine {
+        MockEngine { request ->
+            // record headers from the incoming request
+            lastRequestHeaders = request.headers
+
             if (queue.isNotEmpty()) {
                 val response = lock.withLock { queue.removeAt(0) }
                 respond(response.content, response.status, response.headers)
@@ -34,6 +40,7 @@ object MockServer {
 
     fun clear() {
         queue.clear()
+        lastRequestHeaders = null
     }
 
     private fun enqueue(response: Response) {

app/src/androidTest/java/at/bitfire/icsdroid/net/TestAppHttpClient.kt

@@ -0,0 +1,59 @@
+package at.bitfire.icsdroid.net
+
+import android.content.Context
+import androidx.test.core.app.ApplicationProvider
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import at.bitfire.icsdroid.AppHttpClient
+import at.bitfire.icsdroid.MockServer
+import io.ktor.client.request.get
+import io.ktor.client.statement.HttpResponse
+import io.ktor.http.HttpHeaders
+import io.ktor.http.HttpStatusCode
+import kotlinx.coroutines.runBlocking
+import org.junit.After
+import org.junit.Before
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.junit.Assert.assertEquals
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertNotNull
+
+@RunWith(AndroidJUnit4::class)
+class TestAppHttpClient {
+
+    private val context = ApplicationProvider.getApplicationContext<Context>()
+    private lateinit var client: AppHttpClient
+
+    @Before
+    fun setUp() {
+        MockServer.clear()
+        client = MockServer.httpClient(context)
+    }
+
+    @After
+    fun tearDown() {
+        MockServer.clear()
+    }
+
+    // Verifies that no Accept-Charset header is sent by default
+    @Test
+    fun request_doesNotContainAcceptCharsetHeader() = runBlocking {
+        // enqueue a simple 200 response
+        MockServer.enqueue(content = "ok", status = HttpStatusCode.OK)
+
+        // perform a GET request to the mock server
+        val uri = MockServer.uri("test")
+        val response: HttpResponse = client.httpClient.get(uri.toString())
+
+        assertEquals(HttpStatusCode.OK, response.status)
+
+        // retrieve the headers recorded by the mock server
+        val headers = MockServer.lastRequestHeaders
+
+        // Ensure headers were recorded
+        assertNotNull(headers)
+
+        // Assert that Accept-Charset header is not present
+        assertFalse(headers!!.contains(HttpHeaders.AcceptCharset))
+    }
+}