Skip to content

Commit d22e69d

Browse files
committed
fix: use warn as the default unhandled strategy in setupServer
1 parent d09d8cb commit d22e69d

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

src/node/setup-server.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ export class SetupServerApi implements SetupServer {
6060
handlers,
6161
handlersController: this.#handlersController,
6262
onUnhandledFrame: fromLegacyOnUnhandledRequest(() => {
63-
return this.#listenOptions?.onUnhandledRequest
63+
return this.#listenOptions?.onUnhandledRequest || 'warn'
6464
}),
6565
})
6666

test/node/msw-api/setup-server/scenarios/on-unhandled-request/default.node.test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
1-
/**
2-
* @vitest-environment node
3-
*/
1+
// @vitest-environment node
42
import { setupServer } from 'msw/node'
53
import { HttpResponse, http } from 'msw'
64

@@ -40,5 +38,3 @@ it('warns on unhandled requests by default', async () => {
4038
If you still wish to intercept this unhandled request, please create a request handler for it.
4139
Read more: https://mswjs.io/docs/http/intercepting-requests`)
4240
})
43-
44-
it.todo('does not warn on unhandled "file://" requests')

0 commit comments

Comments
 (0)