fix DOT-4106 add debug logs

Author: sushobhit-lt

src/commander/capture.ts

@@ -16,7 +16,7 @@ command
     .name('capture')
     .description('Capture screenshots of static sites')
     .argument('<file>', 'Web static config file')
-    .option('-P, --parallel [number]', 'Specify the number of instances per browser', parseInt)
+    .option('-C, --parallel [number]', 'Specify the number of instances per browser', parseInt)
     .option('-F, --force', 'forcefully apply the specified parallel instances per browser')
     .option('--fetch-results [filename]', 'Fetch results and optionally specify an output file, e.g., <filename>.json')
     .action(async function(file, _, command) {

src/lib/screenshot.ts

@@ -259,7 +259,11 @@ export async function captureScreenshotsConcurrent(ctx: Context): Promise<Record
     let browserInstances = ctx.options.parallel || 1;
     let optimizeBrowserInstances : number = 0
     optimizeBrowserInstances = Math.floor(Math.log2(totalSnapshots));
-    if (browserInstances != -1 && optimizeBrowserInstances > browserInstances) {
+    if (optimizeBrowserInstances < 1) {
+        optimizeBrowserInstances = 1;
+    }
+
+    if (optimizeBrowserInstances > browserInstances) {
         optimizeBrowserInstances = browserInstances;
     }
 
@@ -274,9 +278,9 @@ export async function captureScreenshotsConcurrent(ctx: Context): Promise<Record
     } else {
         urlsPerInstance = Math.ceil(totalSnapshots / optimizeBrowserInstances);
     }
-    console.log(`*** browserInstances requested ${ctx.options.parallel} `);
-    console.log(`*** optimizeBrowserInstances  ${optimizeBrowserInstances} `);
-    console.log(`*** urlsPerInstance  ${urlsPerInstance}`);
+    ctx.log.debug(`*** browserInstances requested ${ctx.options.parallel} `);
+    ctx.log.debug(`*** optimizeBrowserInstances  ${optimizeBrowserInstances} `);
+    ctx.log.debug(`*** urlsPerInstance  ${urlsPerInstance}`);
     ctx.task.output = `URLs : ${totalSnapshots} || Parallel Browser Instances: ${optimizeBrowserInstances}\n`;
     //Divide the URLs into chunks
     let staticURLChunks = splitURLs(ctx.webStaticConfig, urlsPerInstance);
@@ -292,7 +296,6 @@ export async function captureScreenshotsConcurrent(ctx: Context): Promise<Record
         totalCapturedScreenshots += response.capturedScreenshots;
         output += response.finalOutput;
     });
-    console.log(`*** totalCapturedScreenshots ${totalCapturedScreenshots}`);
 
     utils.delDir('screenshots');
 

src/tasks/captureScreenshots.ts

@@ -18,7 +18,6 @@ export default (ctx: Context): ListrTask<Context, ListrRendererFactory, ListrRen
 
                 if (ctx.options.parallel) {
                     let { totalCapturedScreenshots, output } = await captureScreenshotsConcurrent(ctx);
-                    console.log(`#### captureScreenshotsConcurrent  ${totalCapturedScreenshots}`);
                     if (totalCapturedScreenshots != ctx.webStaticConfig.length) {
                         throw new Error(output)
                     }