Fix options.method being undefined in normalized options (#671)

Author: kleinfreund

source/core/Ky.ts

@@ -135,7 +135,7 @@ export class Ky {
 				},
 				options.hooks,
 			),
-			method: normalizeRequestMethod(options.method ?? (this._input as Request).method),
+			method: normalizeRequestMethod(options.method ?? (this._input as Request).method ?? 'GET'),
 			// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
 			prefixUrl: String(options.prefixUrl || ''),
 			retry: normalizeRetryOptions(options.retry),

test/methods.ts

@@ -24,6 +24,29 @@ test('common method is normalized', async t => {
 	await server.close();
 });
 
+test('method defaults to "GET"', async t => {
+	const server = await createHttpTestServer();
+	server.all('/', (_request, response) => {
+		response.end();
+	});
+
+	t.plan(2);
+
+	await t.notThrowsAsync(
+		ky(server.url, {
+			hooks: {
+				beforeRequest: [
+					(_input, options) => {
+						t.is(options.method, 'GET');
+					},
+				],
+			},
+		}),
+	);
+
+	await server.close();
+});
+
 test.failing('custom method remains identical', async t => {
 	const server = await createHttpTestServer();
 	server.all('/', (_request, response) => {