test: modified case in e2e tests to use undefined

Author: dreamorosi

packages/idempotency/tests/e2e/idempotentDecorator.test.FunctionCode.ts

@@ -29,12 +29,13 @@ class DefaultLambda implements LambdaInterface {
   public async handler(
     _event: Record<string, unknown>,
     _context: Context
-  ): Promise<string> {
+  ): Promise<void> {
     logger.info(`Got test event: ${JSON.stringify(_event)}`);
     // sleep to enforce error with parallel execution
     await new Promise((resolve) => setTimeout(resolve, 1000));
 
-    return 'Hello World';
+    // We return void to test that the utility handles it correctly
+    return;
   }
 
   @idempotent({

packages/idempotency/tests/e2e/idempotentDecorator.test.ts

@@ -193,7 +193,7 @@ describe('Idempotency e2e test decorator, default settings', () => {
       expect(idempotencyRecord.Items?.[0].id).toEqual(
         `${functionNameDefault}#${payloadHash}`
       );
-      expect(idempotencyRecord.Items?.[0].data).toEqual('Hello World');
+      expect(idempotencyRecord.Items?.[0].data).toBeUndefined();
       expect(idempotencyRecord.Items?.[0].status).toEqual('COMPLETED');
       // During the first invocation the handler should be called, so the logs should contain 1 log
       expect(functionLogs[0]).toHaveLength(1);