Skip to content

Commit ce89362

Browse files
authored
chore: enable tests for array and single-value parameters
1 parent 169f94f commit ce89362

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

packages/event-handler/tests/e2e/httpRouter.test.ts

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -227,14 +227,8 @@ describe('REST Event Handler E2E tests', () => {
227227
expect(data.limit).toBe(limit);
228228
});
229229

230-
it.skip('handles array query parameters', async () => {
231-
// TODO: Bug in proxyEventV1ToWebRequest - duplicates multi-value query parameters
232-
// Tracked in: https://github.com/aws-powertools/powertools-lambda-typescript/issues/4750
233-
// API Gateway V1 puts same param in both queryStringParameters (last value) and
234-
// multiValueQueryStringParameters (all values), causing duplication
235-
// Expected: ['active', 'published']
236-
// Actual: ['published', 'active', 'published']
237-
230+
it('handles array query parameters', async () => {
231+
// Prepare
238232
const searchQuery = 'test';
239233
const filters = ['active', 'published'];
240234

@@ -299,10 +293,7 @@ describe('REST Event Handler E2E tests', () => {
299293
expect(data.limit).toBe(limit);
300294
});
301295

302-
it.skip('handles single-value array parameter', async () => {
303-
// TODO: Related to bug in proxyEventV1ToWebRequest - duplicates single-value query parameters
304-
// Tracked in: https://github.com/aws-powertools/powertools-lambda-typescript/issues/4750
305-
296+
it('handles single-value array parameter', async () => {
306297
// Act
307298
const response = await fetch(`${apiUrl}/params/search?filter=active`);
308299
const data = await response.json();

0 commit comments

Comments
 (0)