Add one more test case

andimarek · Feb 13, 2024 · Dec 20, 2023 · Dec 20, 2023 · Dec 20, 2023 · Jan 2, 2024
commit 276c8723dcd24b8c2290d3ad4716bad760cb753d
diff --git a/src/test/groovy/graphql/execution/incremental/DeferExecutionSupportIntegrationTest.groovy b/src/test/groovy/graphql/execution/incremental/DeferExecutionSupportIntegrationTest.groovy
@@ -1178,7 +1178,7 @@ class DeferExecutionSupportIntegrationTest extends Specification {
         ]
     }
 
-    def "can handle non nullable error"() {
+    def "can handle non nullable error in one of the defer calls"() {
         def query = """
             query {
                 post {
@@ -1237,6 +1237,51 @@ class DeferExecutionSupportIntegrationTest extends Specification {
         ]
     }
 
+    def "can handle non nullable error in the initial result"() {
+        def query = """
+            query {
+                post {
+                    id
+                    nonNullableError
+                    ... @defer {
+                        summary
+                    }
+                }
+            }
+        """
+
+        when:
+        def initialResult = executeQuery(query)
+
+        then:
+        initialResult.toSpecification() == [
+                data   : [post: null],
+                errors : [
+                        [message   : "The field at path '/post/nonNullableError' was declared as a non null type, but the code involved in retrieving data has wrongly returned a null value.  The graphql specification requires that the parent field be set to null, or if that is non nullable that it bubble up null to its parent and so on. The non-nullable type is 'String' within parent type 'Post'",
+                         path      : ["post", "nonNullableError"],
+                         extensions: [classification: "NullValueInNonNullableField"]
+                        ]
+                ],
+                hasNext: true
+        ]
+
+        when:
+        def incrementalResults = getIncrementalResults(initialResult)
+
+        then:
+        incrementalResults == [
+                [
+                        hasNext    : false,
+                        incremental: [
+                                [
+                                        path: ["post"],
+                                        data: [summary: "A summary"],
+                                ]
+                        ]
+                ]
+        ]
+    }
+
     private ExecutionResult executeQuery(String query) {
         return this.executeQuery(query, true, [:])
     }