Fix crash in node when mixing sync/async resolvers (backport of #3529)

[chrskrchr]

This is a backport of #3529 #3651 to the 15.x branch that fixes #3528.

Full credit to @asztal for identifying the fix and submitting the original PR.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: chrskrchr / name: Chris Karcher (651fc3f, b9dc10c, cd9b7b5)

[chrskrchr]

Working through the CLA authorization now.

[chrskrchr]

Backported and stripped type information from this module on main:

https://github.com/graphql/graphql-js/blob/main/src/__testUtils__/expectJSON.ts

[IvanGoncharov]

In v15 you don't need this standard expect should work fine.
See other test cases in the same file.

[chrskrchr]

thanks! fixed in the latest commit

[chrskrchr]

Moving this PR back to draft while I sort out the typing issues.

[IvanGoncharov]

Also please backport it to v16.
It would be strange to have this fix in v15 and v17 but not v16 :)

[IvanGoncharov]

In v15 you don't need this standard expect should work fine.
See other test cases in the same file.

[chrskrchr]

Also please backport it to v16. It would be strange to have this fix in v15 and v17 but not v16 :)

Thanks for the heads up! I wasn't aware that main had moved on to v17. Submitted a similar PR for 16.x.x here:

#3576

[chrskrchr]

@IvanGoncharov - I finally worked through the EasyCLA issues, would you mind taking another look at this PR?

[chrskrchr]

Closing this and re-submitting as a new PR per the guidance in #3651.