Merge pull request #529 from michludw/scalars-without-apostrophes

support for scalar argument without apostrophes
typelevel · Dec 12, 2023 · d5ebe44 · d5ebe44
2 parents e0b85de + 16175f3
commit d5ebe44
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 0 deletions.
diff --git a/modules/core/src/main/scala/schema.scala b/modules/core/src/main/scala/schema.scala
@@ -1011,6 +1011,8 @@ object Value {
         value.success
       case (s: ScalarType, Some(value: BooleanValue)) if !s.isBuiltIn =>
         value.success
+      case (s: ScalarType, Some(enumValue: EnumValue)) if !s.isBuiltIn =>
+        StringValue(enumValue.name).success
 
       case (IDType, Some(value: IDValue)) =>
         value.success

diff --git a/modules/core/src/test/scala/compiler/ScalarsSuite.scala b/modules/core/src/test/scala/compiler/ScalarsSuite.scala
@@ -388,6 +388,39 @@ final class ScalarsSuite extends CatsEffectSuite {
     assertIO(res, expected)
   }
 
+  test("query with scalar argument without apostrophes") {
+    val query = """
+      query {
+        moviesLongerThan(duration: PT3H) {
+          title
+          duration
+        }
+      }
+    """
+
+    val expected = json"""
+      {
+        "data" : {
+          "moviesLongerThan" : [
+            {
+              "title" : "Celine et Julie Vont en Bateau",
+              "duration" : "PT3H25M"
+            },
+            {
+              "title" : "L'Amour fou",
+              "duration" : "PT4H12M"
+            }
+          ]
+        }
+      }
+    """
+
+    val res = MovieMapping.compileAndRun(query)
+
+    assertIO(res, expected)
+  }
+
+
   test("query with LocalTime argument") {
     val query = """
       query {