MOL-357/PICT-241: update test cases

Author: tdang1-shopmacher

processor/src/controllers/connector.controller.ts

@@ -37,7 +37,7 @@ export const install = async (request: Request, response: Response) => {
   const protocol = request.secure ? 'https' : 'http';
   const extensionUrl = `${protocol}://${request.hostname}/processor`;
 
-  if (!extensionUrl) {
+  if (!request.hostname) {
     logger.debug('SCTM - install - Missing body parameters {extensionUrl}.');
     return apiError(response, formatErrorResponse(new CustomError(400, 'Missing body parameters.')).errors);
   }

processor/tests/controllers/connector.controller.spec.ts

@@ -39,7 +39,9 @@ describe('Test connector.controller.ts', () => {
 
   it('should return status code 200 with a successful install response', async () => {
     req = {
-      body: { extensionUrl: 'https://example.com/extensionUrl' },
+      hostname: 'test.com',
+      secure: true,
+      protocol: 'https',
     };
     await install(req as Request, res as Response);
     expect(logger.debug).toBeCalledTimes(1);

processor/tests/routes/processor.route.spec.ts

@@ -95,7 +95,7 @@ describe('Test src/route/processor.route.ts', () => {
   });
 
   describe('POST /install/', () => {
-    it('should return 200 if extensionUrl is provided', async () => {
+    it('should return 200 if hostname is provided', async () => {
       const layer = webhookRouter.stack.find(
         //@ts-expect-error route should be always available
         (layer) => layer.route.path === '/install' && layer.route.methods.post,
@@ -105,23 +105,23 @@ describe('Test src/route/processor.route.ts', () => {
       //@ts-expect-error handler should be always available
       const handler = layer.route.stack[0].handle;
 
-      req = {
-        body: {
-          extensionUrl: 'https://example.com',
-        },
-      };
-
       (createPaymentExtension as jest.Mock).mockReturnValueOnce(Promise.resolve());
       (createCustomPaymentType as jest.Mock).mockReturnValueOnce(Promise.resolve());
       (createCustomPaymentInterfaceInteractionType as jest.Mock).mockReturnValueOnce(Promise.resolve());
       (createCustomPaymentTransactionCancelReasonType as jest.Mock).mockReturnValueOnce(Promise.resolve());
 
+      req = {
+        hostname: 'test.com',
+        secure: true,
+        protocol: 'https',
+      };
+
       await handler(req as Request, res as Response, next);
 
       expect(res.status).toHaveBeenCalledWith(200);
     });
 
-    it('should return 400 if extensionUrl is not provided', async () => {
+    it('should return 400 if hostname is not provided', async () => {
       const layer = webhookRouter.stack.find(
         //@ts-expect-error route should be always available
         (layer) => layer.route.path === '/install' && layer.route.methods.post,
@@ -133,6 +133,12 @@ describe('Test src/route/processor.route.ts', () => {
 
       expect(handler).toBeDefined();
 
+      req = {
+        hostname: '',
+        secure: true,
+        protocol: 'https',
+      };
+
       await handler(req as Request, res as Response, next);
 
       expect(logger.debug).toBeCalledTimes(1);