test: response.url after redirect is set to target url (#2716)

test/fetch/fetch-url-after-redirect.js

@@ -0,0 +1,40 @@
+'use strict'
+
+const assert = require('node:assert')
+const { test } = require('node:test')
+const { createServer } = require('node:http')
+const { fetch } = require('../..')
+const { closeServerAsPromise } = require('../utils/node-http')
+const { promisify } = require('node:util')
+
+test('after redirecting the url of the response is set to the target url', async (t) => {
+  // redirect-1 -> redirect-2 -> target
+  const server = createServer((req, res) => {
+    switch (res.req.url) {
+      case '/redirect-1':
+        res.writeHead(302, undefined, { Location: '/redirect-2' })
+        res.end()
+        break
+      case '/redirect-2':
+        res.writeHead(302, undefined, { Location: '/redirect-3' })
+        res.end()
+        break
+      case '/redirect-3':
+        res.writeHead(302, undefined, { Location: '/target' })
+        res.end()
+        break
+      case '/target':
+        res.writeHead(200, 'dummy', { 'Content-Type': 'text/plain' })
+        res.end()
+        break
+    }
+  })
+  t.after(closeServerAsPromise(server))
+
+  const listenAsync = promisify(server.listen.bind(server))
+  await listenAsync(0)
+  const { port } = server.address()
+  const response = await fetch(`http://127.0.0.1:${port}/redirect-1`)
+
+  assert.strictEqual(response.url, `http://127.0.0.1:${port}/target`)
+})