feat(scan): update serializing warnings

src/Commands/RunScan.spec.ts

@@ -67,4 +67,26 @@ describe('RunScan', () => {
       expect(act).toThrowError(SyntaxError);
     });
   });
+
+  describe('serializeWarnings', () => {
+    it('should print warnings correctly', () => {
+      // arrange
+      const input = [
+        {
+          message: 'Notice: Warning 1',
+          code: '1'
+        },
+        {
+          message: 'Notice: Warning 2',
+          code: '2'
+        }
+      ];
+
+      // act
+      const result = RunScan.serializeWarnings(input);
+
+      // assert
+      expect(result).toEqual('Notice: Warning 1\nNotice: Warning 2');
+    });
+  });
 });

src/Commands/RunScan.ts

@@ -7,7 +7,8 @@ import {
   ScanConfig,
   Scans,
   TestType,
-  ATTACK_PARAM_LOCATIONS_DEFAULT
+  ATTACK_PARAM_LOCATIONS_DEFAULT,
+  ScanWarning
 } from '../Scan';
 import { Helpers, logger } from '../Utils';
 import { Arguments, Argv, CommandModule } from 'yargs';
@@ -37,6 +38,10 @@ export class RunScan implements CommandModule {
       });
   }
 
+  public static serializeWarnings(warnings: ScanWarning[]): string {
+    return `${warnings.map((warning) => warning.message).join('\n')}`;
+  }
+
   public builder(argv: Argv): Argv {
     return argv
       .option('token', {
@@ -199,9 +204,7 @@ export class RunScan implements CommandModule {
       console.log(scanId);
 
       if (warnings.length) {
-        logger.warn(
-          `Scan has been started with warnings: ${warnings.join(', ')}`
-        );
+        logger.warn(RunScan.serializeWarnings(warnings));
       }
 
       process.exit(0);

src/Scan/Scans.ts

@@ -195,12 +195,14 @@ export interface StorageFile {
   type: SourceType;
 }
 
+export interface ScanWarning {
+  code: string;
+  message: string;
+}
+
 export interface ScanCreateResponse {
   id: string;
-  warnings?: {
-    code: string;
-    message: string;
-  }[];
+  warnings: ScanWarning[];
 }
 
 export interface Scans {