Synchronised head methods about dart_nock

[julescmay]

I have an idea to patch Registry.match method:

That was exactly my thinking.

I'm on it!

[cah4a]

Hey @julescmay!

Thanks for the interesting question.

You are absolutely right that nock should support basic HTTP protocol features.
I'm sure that it shouldn't be done under the hood. Tests should always fully describe app behavior.

It could be done by adding Interceptor.withHead() method like so:

nock.get('/path')
    ..reply(200, "body", headers: {})
    ..withHead();

Currently, I don't have much time to do it by myself (I'm Ukrainian, don't have much free time for now).
If you have some time, I would really appreciate your PR.

[julescmay]

I'm Ukrainian, don't have much free time for now

I understand. My heartfelt sympathies. Stay safe.

It could be done by adding Interceptor.withHead() method ...

IIUC http spec says that head should respond identically to get except for the lack of body. May I suggest: head response should be automatic unless:

• there's a noHead() method on the interceptor;
• there's another head interceptor on the same path.

If you have some time, I would really appreciate your PR...

Absolutely. Let me know your thoughts, and I'll get on it.

[cah4a]

My heartfelt sympathies. Stay safe.

Huge thanks for the kind words!

noHead() method on the interceptor;

It's clear that it will be more intuitive to use.
I still have a feeling that .withHead() is more controlled and like it more, but I'm ok with reverse ideology too.

there's another head interceptor on the same path.

Because there is no "priority" for the Interceptors, I have an idea to patch Registry.match method: If no interceptor is found and the requested method is HEAD try to find corresponding GET request and create a new HEAD Interceptor.