Add test case for tapir jsonitor max char size

ghostdogpr · Nov 19, 2024 · 68361a0 · 68361a0
1 parent f23fba0
commit 68361a0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/adapters/play/src/test/resources/application.conf b/adapters/play/src/test/resources/application.conf
@@ -0,0 +1,2 @@
+http.server.max-content-length = infinite
+play.http.parser.maxMemoryBuffer = 100M
diff --git a/adapters/quick/src/test/scala/caliban/QuickAdapterSpec.scala b/adapters/quick/src/test/scala/caliban/QuickAdapterSpec.scala
@@ -42,7 +42,7 @@ object QuickAdapterSpec extends ZIOSpecDefault {
     suite.provideShared(
       apiLayer,
       Scope.default,
-      Server.defaultWith(_.port(8090).responseCompression())
+      Server.defaultWith(_.port(8090).enableRequestStreaming.responseCompression())
     )
   }
 }
diff --git a/interop/tapir/src/test/scala/caliban/interop/tapir/TapirAdapterSpec.scala b/interop/tapir/src/test/scala/caliban/interop/tapir/TapirAdapterSpec.scala
@@ -3,7 +3,7 @@ package caliban.interop.tapir
 import caliban.InputValue.ObjectValue
 import caliban.Value.StringValue
 import caliban._
-import com.github.plokhotnyuk.jsoniter_scala.core.{ readFromString, writeToString, JsonValueCodec }
+import com.github.plokhotnyuk.jsoniter_scala.core.{ readFromString, writeToString, JsonValueCodec, ReaderConfig }
 import com.github.plokhotnyuk.jsoniter_scala.macros.JsonCodecMaker
 import sttp.capabilities.zio.ZioStreams
 import sttp.capabilities.{ Effect, WebSockets }
@@ -272,6 +272,13 @@ object TapirAdapterSpec {
               method = Method.GET.method,
               query = """mutation{ deleteCharacter(name: "Amos Burton") }"""
             ).map(r => assertTrue(r.code.code == 400))
+          },
+          test("very long field values in mutations") {
+            val name = "A".repeat(ReaderConfig.maxCharBufSize + 1)
+            runHttpRequest(
+              method = Method.POST.method,
+              query = s"""mutation { deleteCharacter(name: \"$name\") }"""
+            ).map(r => assertTrue(r.code.code == 200))
           }
         )
       ),