fixed test and sonar finding

Author: sdangol

packages/event-handler/src/rest/middleware/cors.ts

@@ -105,21 +105,21 @@ export const cors = (options?: CorsOptions): Middleware => {
           config.maxAge.toString()
         );
       }
-      allowedMethods.forEach((method) => {
+      for (const method of allowedMethods) {
         reqCtx.res.headers.append('access-control-allow-methods', method);
-      });
-      allowedHeaders.forEach((header) => {
+      }
+      for (const header of allowedHeaders) {
         reqCtx.res.headers.append('access-control-allow-headers', header);
-      });
+      }
       return new Response(null, {
         status: HttpErrorCodes.NO_CONTENT,
         headers: reqCtx.res.headers,
       });
     }
 
-    config.exposeHeaders.forEach((header) => {
+    for (const header of config.exposeHeaders) {
       reqCtx.res.headers.append('access-control-expose-headers', header);
-    });
+    }
 
     await next();
   };

packages/event-handler/tests/unit/rest/middleware/cors.test.ts

@@ -197,7 +197,7 @@ describe('CORS Middleware', () => {
       corsConfig.allowMethods
     );
     expect(result.multiValueHeaders?.['access-control-allow-headers']).toEqual(
-      corsConfig.allowHeaders
+      corsConfig.allowHeaders.map((header) => header.toLowerCase())
     );
     expect(result.headers?.['access-control-max-age']).toEqual(
       corsConfig.maxAge.toString()