fix: check if property errors from response is an empty array (#454)

src/index.ts

@@ -482,7 +482,10 @@ async function makeRequest<T = any, V extends Variables = Variables>({
     isBatchingQuery && Array.isArray(result) ? !result.some(({ data }) => !data) : !!result.data
 
   const successfullyPassedErrorPolicy =
-    !result.errors || fetchOptions.errorPolicy === 'all' || fetchOptions.errorPolicy === 'ignore'
+    !result.errors ||
+    (Array.isArray(result.errors) && !result.errors.length) ||
+    fetchOptions.errorPolicy === 'all' ||
+    fetchOptions.errorPolicy === 'ignore'
 
   if (response.ok && successfullyPassedErrorPolicy && successfullyReceivedData) {
     const { headers, status } = response

tests/__helpers.ts

@@ -3,7 +3,7 @@ import body from 'body-parser'
 import express, { Application, Request } from 'express'
 import getPort from 'get-port'
 import { createServer, Server } from 'http'
-import { JsonObject } from 'type-fest'
+import { JsonArray, JsonObject } from 'type-fest'
 import { beforeAll, afterEach, afterAll, beforeEach } from 'vitest'
 
 type CapturedRequest = Pick<Request, 'headers' | 'method' | 'body'>
@@ -21,7 +21,7 @@ type Context<S extends MockSpec | MockSpecBatch = MockSpec> = {
 type MockSpecBody = {
   data?: JsonObject
   extensions?: JsonObject
-  errors?: JsonObject
+  errors?: JsonObject | JsonArray
 }
 
 type MockSpec = {

tests/general.test.ts

@@ -396,3 +396,16 @@ describe('operationName parsing', () => {
     expect(requestBody.operationName).toEqual('myStringOperation')
   })
 })
+
+test('should not throw error when errors property is an empty array (occured when using UltraGraphQL)', async () => {
+  ctx.res({
+    body: {
+      data: { test: 'test' },
+      errors: [],
+    },
+  })
+
+  const res = await new GraphQLClient(ctx.url).request(`{ test }`)
+
+  expect(res).toEqual(expect.objectContaining({ test: 'test' }))
+})