nodejs/undici

Allow asynchronous mocking replies

Closed

#1,348 opened on Apr 19, 2022

 (10 comments) (6 reactions) (0 assignees)JavaScript (444 forks)batch import
enhancementgood first issue

Repository metrics

Stars
 (5,319 stars)
PR merge metrics
 (Avg merge 2d 10h) (82 merged PRs in 30d)

Description

This would solve...

When trying to create a Undici adapter for Polly.JS I tried to use the mocking capabilities from Undici to record network requests and replay them later using stored HAR files. However, the MockInterceptor.reply method doesn't support asynchronous callbacks. So receiving the request, loading a file from the filesystem (asynchronously) (or passing through the request) and resolving with a promise doesn't work.

Sidenote: nock supports asynchronous callbacks: https://github.com/nock/nock#specifying-replies

The implementation should look like...

The MockInterceptor.reply method could accept asynchronous callbacks:

mockPool
  .intercept({
    path: () => true
  })
  .reply(async (options) => {
    const response = await fetchFromFileSystem(options)
    return mapping(response)
  })

I have also considered...

Creating my own version of MockAgent that supports async replies, but that just seemed too much work for a possibly temporary problem.

Additional context

Feature request for Undici adapter for Polly.JS Feature request for Undici support for Nock

Contributor guide