feat: omit query string parameters from builder functions

src/lib/builder_functions.js

@@ -22,9 +22,16 @@ const wrapHandler = (handler) => (event, context, callback) => {
     })
   }
 
+  // Removing query string parameters from the builder function.
+  const modifiedEvent = {
+    ...event,
+    multiValueQueryStringParameters: {},
+    queryStringParameters: {},
+  }
+
   // eslint-disable-next-line promise/prefer-await-to-callbacks
   const wrappedCallback = (error, response) => callback(error, augmentResponse(response))
-  const execution = handler(event, context, wrappedCallback)
+  const execution = handler(modifiedEvent, context, wrappedCallback)
 
   if (isPromise(execution)) {
     // eslint-disable-next-line promise/prefer-await-to-then

test/builder_functions.js

@@ -150,3 +150,25 @@ test('Preserves errors thrown inside the wrapped handler', async (t) => {
 
   await t.throwsAsync(invokeLambda(builderFunction(myHandler)), { is: error })
 })
+
+test('Does not pass query parameters to the wrapped handler', async (t) => {
+  const originalResponse = {
+    body: ':thumbsup:',
+    statusCode: 200,
+  }
+  // eslint-disable-next-line require-await
+  const myHandler = async (event) => {
+    t.deepEqual(event.multiValueQueryStringParameters, {})
+    t.deepEqual(event.queryStringParameters, {})
+
+    return originalResponse
+  }
+  const multiValueQueryStringParameters = { foo: ['bar'], bar: ['baz'] }
+  const queryStringParameters = { foo: 'bar', bar: 'baz' }
+  const response = await invokeLambda(builderFunction(myHandler), {
+    multiValueQueryStringParameters,
+    queryStringParameters,
+  })
+
+  t.deepEqual(response, { ...originalResponse, ...METADATA_OBJECT })
+})

test/helpers/main.js

@@ -1,5 +1,6 @@
-const invokeLambda = (handler, { method = 'GET' } = {}) => {
+const invokeLambda = (handler, { method = 'GET', ...options } = {}) => {
   const event = {
+    ...options,
     httpMethod: method,
   }