diff --git a/src/execution/__tests__/executor-test.ts b/src/execution/__tests__/executor-test.ts index 838eeedf07..a945b4d0b5 100644 --- a/src/execution/__tests__/executor-test.ts +++ b/src/execution/__tests__/executor-test.ts @@ -582,7 +582,7 @@ describe('Execute: Handles basic execution tasks', () => { }); it('handles sync errors combined with rejections', async () => { - let isAsyncResolverCalled = false; + let isAsyncResolverFinished = false; const schema = new GraphQLSchema({ query: new GraphQLObjectType({ @@ -598,8 +598,8 @@ describe('Execute: Handles basic execution tasks', () => { await resolveOnNextTick(); await resolveOnNextTick(); await resolveOnNextTick(); - isAsyncResolverCalled = true; - return Promise.resolve(null); + isAsyncResolverFinished = true; + return null; }, }, }, @@ -614,10 +614,10 @@ describe('Execute: Handles basic execution tasks', () => { } `); - const result = await execute({ schema, document }); + const result = execute({ schema, document }); - expect(isAsyncResolverCalled).to.equal(true); - expectJSON(result).toDeepEqual({ + expect(isAsyncResolverFinished).to.equal(false); + expectJSON(await result).toDeepEqual({ data: null, errors: [ { @@ -628,6 +628,7 @@ describe('Execute: Handles basic execution tasks', () => { }, ], }); + expect(isAsyncResolverFinished).to.equal(true); }); it('Full response path is included for non-nullable fields', () => {