Fix response details getting lost when using onDownloadProgress (#441)

Author: promer94

source/core/Ky.ts

@@ -167,7 +167,7 @@ export class Ky {
 			// To provide correct form boundary, Content-Type header should be deleted each time when new Request instantiated from another one
 			if (
 				((supportsFormData && this._options.body instanceof globalThis.FormData)
-				|| this._options.body instanceof URLSearchParams) && !(this._options.headers && (this._options.headers as Record<string, string>)['content-type'])
+					|| this._options.body instanceof URLSearchParams) && !(this._options.headers && (this._options.headers as Record<string, string>)['content-type'])
 			) {
 				this.request.headers.delete('content-type');
 			}
@@ -314,6 +314,11 @@ export class Ky {
 					await read();
 				},
 			}),
+			{
+				status: response.status,
+				statusText: response.statusText,
+				headers: response.headers,
+			},
 		);
 	}
 }

test/browser.ts

@@ -91,6 +91,43 @@ test('aborting a request', withPage, async (t: ExecutionContext, page: Page) =>
 	await server.close();
 });
 
+test('should copy origin response info when use onDownloadProgress', withPage, async (t: ExecutionContext, page: Page) => {
+	const json = {hello: 'world'};
+	const status = 202;
+	const statusText = 'Accepted';
+	const server = await createEsmTestServer();
+	server.get('/', (_request, response) => {
+		response.end('meow');
+	});
+
+	server.get('/test', (_request, response) => {
+		setTimeout(() => {
+			response.statusMessage = statusText;
+			response.status(status).header('X-ky-Header', 'ky').json(json);
+		}, 500);
+	});
+	await page.goto(server.url);
+	await addKyScriptToPage(page);
+	const data = await page.evaluate(async url => {
+		// eslint-disable-next-line @typescript-eslint/no-empty-function
+		const request = window.ky.get(`${url}/test`, {onDownloadProgress() {}}).then(async v => ({
+			headers: v.headers.get('X-ky-Header'),
+			status: v.status,
+			statusText: v.statusText,
+			data: await v.json(),
+		}));
+		return request;
+	}, server.url);
+
+	t.deepEqual(data, {
+		status,
+		headers: 'ky',
+		statusText,
+		data: json,
+	});
+	await server.close();
+});
+
 test('aborting a request with onDonwloadProgress', withPage, async (t: ExecutionContext, page: Page) => {
 	const server = await createEsmTestServer();