fix: MockResponseCallbackOptions type

[merojosa]

This relates to...

#1583

Rationale

The issue is self-explanatory. It's kinda complicated to get what the actual types are. So I decided to print the optsargument:

undici/lib/mock/mock-interceptor.js

Line 124 in 0b5a451

const wrappedDefaultsCallback = (opts) => {

This is what I got:

From what I understand, only path and method are sure to exist.

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

• I have read and agreed to the Developer's Certificate of Origin
• Tested
• Benchmarked (optional)
• Documented
• Review ready
• In review
• Merge ready

[metcoder95]

Can you add some testing to it?

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.17%. Comparing base (0b5a451) to head (9249b4b).
Report is 163 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2951      +/-   ##
==========================================
+ Coverage   93.68%   94.17%   +0.49%     
==========================================
  Files          87       90       +3     
  Lines       23973    24432     +459     
==========================================
+ Hits        22458    23009     +551     
+ Misses       1515     1423      -92     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mcollina]

Thanks for opening a PR! Can you please add a unit test? We use tsd for types

[merojosa]

@mcollina @metcoder95 I think this existing test already covers the callback options types:

expectAssignable<MockScope>(mockInterceptor.reply((options) => {
    expectAssignable<MockInterceptor.MockResponseCallbackOptions>(options);
    return { statusCode: 200, data: { foo: 'bar'}
  }}))

Even so, I saw that that there was a different test for options.header, so I tested the rest of possible values in this same test.

[mcollina]

lgtm