[scala-akka-client] Scala Akka client does not support arbitrary quer…

…y string parameters (#17386) * Add check for input if is Map type * Update Pet samples * Add PetApi support for filter by Pet property * Clean up * update samples * remove manual test --------- Co-authored-by: ameenhaq <[email protected]>
OpenAPITools · Dec 13, 2023 · 734b922 · 734b922
1 parent 42f7219
commit 734b922
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 2 deletions.
diff --git a/modules/openapi-generator/src/main/resources/scala-akka-client/apiRequest.mustache b/modules/openapi-generator/src/main/resources/scala-akka-client/apiRequest.mustache
@@ -49,6 +49,8 @@ case class ApiRequest[U](
 
   def withPathParam(name: String, value: Any): ApiRequest[U] = copy[U](pathParams = pathParams + (name -> value))
 
+  def withQueryParam(values: Map[String, Any]): ApiRequest[U] = copy[U](queryParams = queryParams ++ values)
+
   def withQueryParam(name: String, value: Any): ApiRequest[U] = copy[U](queryParams = queryParams + (name -> value))
 
   def withHeaderParam(name: String, value: Any): ApiRequest[U] = copy[U](headerParams = headerParams + (name -> value))

diff --git a/modules/openapi-generator/src/main/resources/scala-akka-client/paramCreation.mustache b/modules/openapi-generator/src/main/resources/scala-akka-client/paramCreation.mustache
@@ -1 +1 @@
-"{{baseName}}", {{#isContainer}}ArrayValues({{{paramName}}}{{#collectionFormat}}, {{collectionFormat.toUpperCase}}{{/collectionFormat}}){{/isContainer}}{{^isContainer}}{{{paramName}}}{{/isContainer}}
+{{#isMap}}{{baseName}}{{/isMap}}{{^isMap}}"{{baseName}}", {{#isContainer}}ArrayValues({{{paramName}}}{{#collectionFormat}}, {{collectionFormat.toUpperCase}}{{/collectionFormat}}){{/isContainer}}{{^isContainer}}{{{paramName}}}{{/isContainer}}{{/isMap}}
diff --git a/...s/client/petstore/scala-akka/src/main/scala/org/openapitools/client/core/ApiRequest.scala b/...s/client/petstore/scala-akka/src/main/scala/org/openapitools/client/core/ApiRequest.scala
@@ -59,6 +59,8 @@ case class ApiRequest[U](
 
   def withPathParam(name: String, value: Any): ApiRequest[U] = copy[U](pathParams = pathParams + (name -> value))
 
+  def withQueryParam(values: Map[String, Any]): ApiRequest[U] = copy[U](queryParams = queryParams ++ values)
+
   def withQueryParam(name: String, value: Any): ApiRequest[U] = copy[U](queryParams = queryParams + (name -> value))
 
   def withHeaderParam(name: String, value: Any): ApiRequest[U] = copy[U](headerParams = headerParams + (name -> value))

diff --git a/samples/client/petstore/scala-akka/src/test/scala/PetApiTest.scala b/samples/client/petstore/scala-akka/src/test/scala/PetApiTest.scala
@@ -87,7 +87,6 @@ class PetApiTest extends AsyncFlatSpec with Matchers {
 
   it should "find pets by status" in {
     val request = api.findPetsByStatus(List("available"))
-
     invoker
       .execute(request)
       .map { apiResponse =>