Michael Solomon
Michael Solomon
Oh, seems like fix wasn't back-ported to previous versions.
@gr2m Cool! thanks! Node.js did not backport the compression order fix to versions 18 and 20, so the test passes for version 22 (with the correct order) but fails for...
> Do you know if that might still be happening? From a (really) quick check I did, it should get backported soon. > Depends on how many Node versions we...
I set the coverage threshold to 100 so I'll remember to increase it back after I decrease it in this PR: https://github.com/nock/nock/pull/2834
Thanks for the report!. Can you please share minimum reproducible example? (no ts, no react, not 3rd packages as possible)
@lithin I run this example 1000 times (with mocha) and it worked for me: ```js import { randomUUID } from "node:crypto"; import nock from "nock"; import { expect } from...
@lithin, for the small subset (point 3), maybe you can ask GPT/Copilot to make a minimal reproduction from these tests? (like remove any proprietary-related code and such)?
@piotrjak is it a timeout error?
CC @goldbergyoni, You may find the above comment interesting☝️
@ekohilas WDYT about [this](https://github.com/goldbergyoni/nodejs-testing-best-practices/blob/master/example-application/test/createOrder.callingOtherServices.test.ts#L35-L38) approach to extract the request body? This is currently what I do. ```js let emailPayload; nock('http://mailer.com') .post('/send', (body) => { emailPayload = body return true })...