fix(ClientRequest): spread object args to support "follow-redirects" (#…

…541)

Co-authored-by: Michael Solomon <[email protected]>

src/interceptors/ClientRequest/utils/normalizeClientRequestArgs.test.ts

@@ -274,7 +274,7 @@ it('handles [RequestOptions, callback] input', () => {
   expect(url.href).toEqual('https://mswjs.io/resource')
 
   // Request options must be preserved.
-  expect(options).toEqual(initialOptions)
+  expect(options).toMatchObject(initialOptions)
 
   // Callback must be preserved.
   expect(callback).toBeTypeOf('function')
@@ -320,7 +320,7 @@ it('handles [PartialRequestOptions, callback] input', () => {
   )
 
   // Request options must be preserved.
-  expect(options).toEqual(initialOptions)
+  expect(options).toMatchObject(initialOptions)
 
   // Options protocol must be inferred from the request issuing module.
   expect(options.protocol).toEqual('https:')

src/interceptors/ClientRequest/utils/normalizeClientRequestArgs.ts

@@ -210,7 +210,7 @@ export function normalizeClientRequestArgs(
   // Handle a given "RequestOptions" object as-is
   // and derive the URL instance from it.
   else if (isObject(args[0])) {
-    options = args[0] as any
+    options = { ... args[0] as any }
     logger.info('first argument is RequestOptions:', options)
 
     // When handling a "RequestOptions" object without an explicit "protocol",

test/third-party/follow-redirect-http.test.ts

@@ -40,7 +40,7 @@ afterAll(async () => {
   await server.close()
 })
 
-it.skip('intercepts a POST request issued by "follow-redirects"', async () => {
+it('intercepts a POST request issued by "follow-redirects"', async () => {
   const { address } = server.https
   const payload = JSON.stringify({ todo: 'Buy the milk' })