fix(param-decorator): @query take first value by default

.gitignore

@@ -8,3 +8,4 @@ out
 gen
 node_modules
 coverage
+deno.lock

spec/method-param-decorator.test.ts

@@ -32,6 +32,11 @@ class SimpleController {
 		return myvalue;
 	}
 
+	@Get('/query/myvalue/default')
+	simpleGetMyValueDefault(@Query('myvalue') myvalue: string[]) {
+		return myvalue;
+	}
+
 	@Get('/query/myvalue/last')
 	simpleGetMyValueLast(@Query('myvalue', { value: 'last' }) myvalue: string) {
 		return myvalue;
@@ -182,6 +187,22 @@ Deno.test(`@Query decorator with value 'first' to return the first value for a g
 	await app.close();
 });
 
+Deno.test(`@Query decorator with value and no option to return the first value for a given query parameter`, async () => {
+	await app.init(MyModule);
+	const listenEvent = await app.listen(0);
+
+	const res = await fetch(
+		`http://localhost:${listenEvent.port}/query/myvalue/default?myvalue=foo&myvalue=bar`,
+		{
+			method: 'GET',
+		},
+	);
+	const text = await res.text();
+	assertEquals(text, `foo`);
+
+	await app.close();
+});
+
 Deno.test(`@Query decorator with no key and value 'array' to return all values of all query parameters`, async () => {
 	await app.init(MyModule);
 	const listenEvent = await app.listen(0);

src/router/controller/params/decorators.ts

@@ -128,19 +128,19 @@ function formatQueryValue(
 	queryValue: string[] | undefined,
 	value: 'first' | 'last' | 'array' | undefined,
 ) {
-	if (!queryValue || !value) {
+	if (!queryValue) {
 		return undefined;
 	}
 
-	switch (value || 'last') {
+	switch (value) {
 		case 'first':
 			return queryValue[0];
 		case 'last':
 			return queryValue[queryValue.length - 1];
 		case 'array':
 			return queryValue;
 		default:
-			return undefined;
+			return queryValue[0];
 	}
 }