mswjs / interceptors

Low-level network interception library.
https://npm.im/@mswjs/interceptors
MIT License
539 stars 123 forks source link

test(http): add "connect" and "secureConnect" tests #576

Closed kettanaito closed 3 months ago

kettanaito commented 4 months ago
kettanaito commented 4 months ago

Hi, @mikicho. Could you please take a brief look at these tests? They confirm that the Socket interceptor correctly emits the connect and secureConnect events for both mocked and bypassed scenarios. Thanks!

mikicho commented 3 months ago

Looks great! The tests in Nock fail due to https://github.com/mswjs/interceptors/pull/515#issuecomment-2067702330, which we can tackle later. my bad