Merge pull request #171 from disneystreaming/http4s-empty-body

http4s make body empty when metadata is total

Author: Baccata

modules/http4s/src/smithy4s/http4s/EntityCompiler.scala

@@ -25,6 +25,7 @@ import org.http4s._
 import org.http4s.headers.`Content-Type`
 import smithy4s.http.BodyPartial
 import smithy4s.http.CodecAPI
+import smithy4s.http.Metadata
 
 trait EntityCompiler[F[_]] {
 
@@ -67,13 +68,20 @@ object EntityCompiler {
       def compileEntityEncoder[A](schema: Schema[A]): EntityEncoder[F, A] = {
         val codecA: codecAPI.Codec[A] = codecAPI.compileCodec(schema)
         val mediaType = MediaType.unsafeParse(codecAPI.mediaType(codecA).value)
-
-        EntityEncoder
-          .byteArrayEncoder[F]
-          .withContentType(
-            `Content-Type`(mediaType)
-          )
-          .contramap[A]((a: A) => codecAPI.writeToArray(codecA, a))
+        val expectBody = Metadata.PartialDecoder
+          .fromSchema(schema)
+          .total
+          .isEmpty // expect body if metadata decoder is not total
+        if (expectBody) {
+          EntityEncoder
+            .byteArrayEncoder[F]
+            .withContentType(
+              `Content-Type`(mediaType)
+            )
+            .contramap[A]((a: A) => codecAPI.writeToArray(codecA, a))
+        } else {
+          EntityEncoder.emptyEncoder
+        }
       }
 
       def compileEntityDecoder[A](

modules/tests/src/smithy4s/tests/PizzaSpec.scala

@@ -324,9 +324,10 @@ abstract class PizzaSpec
           log
         )
       } yield {
-        val (code, headers, _) = res
+        val (code, headers, body) = res
 
         expect(code == 200) &&
+        expect(body.isEmpty()) &&
         expect(headers.get("x-uppercase-header") == Some(List("header-1"))) &&
         expect(headers.get("x-capitalized-header") == Some(List("header-2"))) &&
         expect(headers.get("x-lowercase-header") == Some(List("header-3"))) &&