Add Content-Length header tests

Author: lauzadis

runtime/protocol/http-client-engines/http-client-engine-crt/jvm/test/aws/smithy/kotlin/runtime/http/engine/crt/RequestUtilTest.kt

@@ -4,9 +4,17 @@
  */
 package aws.smithy.kotlin.runtime.http.engine.crt
 
+import aws.smithy.kotlin.runtime.content.ByteStream
+import aws.smithy.kotlin.runtime.http.Headers
+import aws.smithy.kotlin.runtime.http.request.HttpRequest
+import aws.smithy.kotlin.runtime.http.HttpMethod
+import aws.smithy.kotlin.runtime.http.toHttpBody
+import aws.smithy.kotlin.runtime.net.url.Url
+import kotlinx.coroutines.test.runTest
 import kotlin.test.Test
 import kotlin.test.assertFalse
 import kotlin.test.assertTrue
+import kotlin.test.assertEquals
 
 class RequestUtilTest {
     @Test
@@ -42,4 +50,19 @@ class RequestUtilTest {
             assertFalse(isRetryable(code, name), "Expected $name to not be retryable!")
         }
     }
+
+    @Test
+    fun testContentLengthHeader() = runTest {
+        val data = "a".repeat(100)
+
+        val request = HttpRequest(
+            HttpMethod.POST,
+            url = Url.parse("https://notarealurl.com"),
+            headers = Headers { set("Content-Length", data.length.toString()) },
+            body = ByteStream.fromString(data).toHttpBody()
+        )
+
+        val crtRequest = request.toCrtRequest(coroutineContext)
+        assertEquals(listOf(data.length.toString()), crtRequest.headers.getAll("Content-Length"))
+    }
 }

runtime/protocol/http-client-engines/http-client-engine-okhttp/jvm/test/aws/smithy/kotlin/runtime/http/engine/okhttp/OkHttpRequestTest.kt

@@ -5,6 +5,7 @@
 
 package aws.smithy.kotlin.runtime.http.engine.okhttp
 
+import aws.smithy.kotlin.runtime.content.ByteStream
 import aws.smithy.kotlin.runtime.http.*
 import aws.smithy.kotlin.runtime.http.engine.internal.HttpClientMetrics
 import aws.smithy.kotlin.runtime.http.request.HttpRequest
@@ -71,6 +72,22 @@ class OkHttpRequestTest {
         assertEquals(listOf("bar", "baz"), actual.headers("FoO"))
     }
 
+    @Test
+    fun itConvertsContentLengthHeader() {
+        val data = "a".repeat(100)
+        val url = Url.parse("https://aws.amazon.com")
+        val headers = Headers {
+            append("Content-Length", data.length.toString())
+        }
+        val request = HttpRequest(HttpMethod.POST, url, headers, ByteStream.fromString(data).toHttpBody())
+
+        val execContext = ExecutionContext()
+        val actual = request.toOkHttpRequest(execContext, EmptyCoroutineContext, testMetrics)
+
+        assertTrue(actual.headers.size >= 1)
+        assertEquals(listOf(data.length.toString()), actual.headers("Content-Length"))
+    }
+
     @Test
     fun itSupportsNonAsciiHeaderValues() {
         val url = Url.parse("https://aws.amazon.com")