Commit553006b

authored

fix: await response middleware (#810)

1 parentd368c16 commit553006bCopy full SHA for 553006b

File tree

+23

-5

lines changed

+23

-5

lines changed

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ export class GraphQLClient {`
`75`	`75`	`})`
`76`	`76`
`77`	`77`	`if(responseMiddleware){`
`78`		`-responseMiddleware(response,{`
	`78`	`+awaitresponseMiddleware(response,{`
`79`	`79`	`operationName:document.operationName,`
`80`	`80`	`variables,`
`81`	`81`	`url:this.url,`
`@@ -146,7 +146,7 @@ export class GraphQLClient {`
`146`	`146`	`})`
`147`	`147`
`148`	`148`	`if(responseMiddleware){`
`149`		`-responseMiddleware(response,{`
	`149`	`+awaitresponseMiddleware(response,{`
`150`	`150`	`operationName:analyzedDocument.operationName,`
`151`	`151`	`variables:requestOptions.variables,`
`152`	`152`	`url:this.url,`
`@@ -222,7 +222,7 @@ export class GraphQLClient {`
`222`	`222`	`})`
`223`	`223`
`224`	`224`	`if(this.requestConfig.responseMiddleware){`
`225`		`-this.requestConfig.responseMiddleware(response,{`
	`225`	`+awaitthis.requestConfig.responseMiddleware(response,{`
`226`	`226`	`operationName:undefined,`
`227`	`227`	`variables,`
`228`	`228`	`url:this.url,`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`importtype{TypedDocumentNode}from'@graphql-typed-document-node/core'`
`2`	`2`	`importtype{GraphQLError}from'graphql/error/GraphQLError.js'`
`3`	`3`	`importtype{DocumentNode}from'graphql/language/ast.js'`
`4`		`-importtype{MaybeLazy,RemoveIndex}from'../../lib/prelude.js'`
	`4`	`+importtype{MaybeLazy,MaybePromise,RemoveIndex}from'../../lib/prelude.js'`
`5`	`5`	`importtype{ClientError}from'../classes/ClientError.js'`
`6`	`6`
`7`	`7`	`exporttypeFetch=typeoffetch`
`@@ -93,7 +93,7 @@ export type RequestOptions<V extends Variables = Variables, T = unknown> =`
`93`	`93`	`exporttypeResponseMiddleware=(`
`94`	`94`	`response:GraphQLClientResponse<unknown>\|ClientError\|Error,`
`95`	`95`	`request:RequestExtendedInit,`
`96`		`-)=>void`
	`96`	`+)=>MaybePromise<void>`
`97`	`97`
`98`	`98`	`exporttypeRequestMiddleware<VextendsVariables=Variables>=(`
`99`	`99`	`request:RequestExtendedInit<V>,`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -247,3 +247,5 @@ export function assertObject(v: unknown): asserts v is object {`
`247`	`247`	`}`
`248`	`248`
`249`	`249`	`exporttypeStringKeyof<T>=keyofT&string`
	`250`	`+`
	`251`	`+exporttypeMaybePromise<T>=T\|Promise<T>`

Lines changed: 16 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,16 @@`
	`1`	`+import{expect,test}from'vitest'`
	`2`	`+import{GraphQLClient}from'../../src/entrypoints/main.js'`
	`3`	`+import{setupMockServer}from'./__helpers.js'`
	`4`	`+`
	`5`	`+constctx=setupMockServer()`
	`6`	`+`
	`7`	+test(`throwing an error in response middleware is not swalled`,async()=>{
	`8`	`+constclient=newGraphQLClient(ctx.url,{`
	`9`	`+// eslint-disable-next-line`
	`10`	`+responseMiddleware:async()=>{`
	`11`	+thrownewError(`TEST ERROR`)
	`12`	`+},`
	`13`	`+})`
	`14`	`+ctx.res()`
	`15`	+awaitexpect(client.request(`{ me { id } }`)).rejects.toMatchInlineSnapshot(`[Error: TEST ERROR]`)
	`16`	`+})`

Comments

(0)