feat: add ability to mock requests in record mode

[markgmilner]

Related issues:

• nock does not intercept requests when recorder is on #247
• Do not disable mocks during recording #1747

Description

Adds a new allow_mocked recorder option.

If true, the recorder will only record unmatched requests. Calls to matched requests will return the mocked response and will not be recorded.

This is useful when you want your tests to use specific responses for certain requests but to record the rest.

[HartS]

This sounds useful! Looking forward to it

[sdotson]

what needs to happen for this to get reviewed/merged?

My use case: Make it a lot more visible to engineers that un-mocked network requests are happening, even in files that have some mocked requests already. The goal would be to get that number down to zero in unit tests.

[irfan798]

The problem with this is its using matchOrigin which doesnt check body beforehand. It assumes that there is a mock but then throws an error when it can't match body later.

So it mistakenly forwards to mocked request instead of sending it to internet and recording it.

https://github.com/ava-labs/nock/blob/allow_mocked/lib/recorder.js#L246