Merge pull request #98 from LambdaTest/stage

Release v3.0.11

Author: arushsaxena1998

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lambdatest/smartui-cli",
-  "version": "3.0.10",
+  "version": "3.0.11",
   "description": "A command line interface (CLI) to run SmartUI tests on LambdaTest",
   "files": [
     "dist/**/*"

src/commander/exec.ts

@@ -16,6 +16,7 @@ command
     .name('exec')
     .description('Run test commands around SmartUI')
     .argument('<command...>', 'Command supplied for running tests')
+    .option('-P, --port <number>', 'Port number for the server')
     .action(async function(execCommand, _, command) {
         let ctx: Context = ctxInit(command.optsWithGlobals());
 

src/lib/ctx.ts

@@ -12,6 +12,7 @@ export default (options: Record<string, string>): Context => {
     let webConfig: WebConfig;
     let mobileConfig: MobileConfig;
     let config = constants.DEFAULT_CONFIG;
+    let port: number;
 
     try {
         if (options.config) {
@@ -28,6 +29,10 @@ export default (options: Record<string, string>): Context => {
                 throw new Error(validateConfig.errors[0].message);
             }
         }
+        port = parseInt(options.port || '49152', 10);
+        if (isNaN(port) || port < 1 || port > 65535) {
+            throw new Error('Invalid port number. Port number must be an integer between 1 and 65535.');
+        }
     } catch (error: any) {
         console.log(`[smartui] Error: ${error.message}`);
         process.exit();
@@ -75,7 +80,8 @@ export default (options: Record<string, string>): Context => {
         options: {
             parallel: options.parallel ? true : false,
             markBaseline: options.markBaseline ? true : false,
-            buildName: options.buildName || ''
+            buildName: options.buildName || '',
+            port: port
         },
         cliVersion: version,
         totalSnapshots: -1

src/lib/processSnapshot.ts

@@ -17,7 +17,7 @@ export default async (snapshot: Snapshot, ctx: Context): Promise<Record<string,
         javaScriptEnabled: ctx.config.enableJavaScript,
         userAgent: constants.CHROME_USER_AGENT,
     }
-    if (!ctx.browser) {
+    if (!ctx.browser?.isConnected()) {
         if (ctx.env.HTTP_PROXY || ctx.env.HTTPS_PROXY) launchOptions.proxy = { server: ctx.env.HTTP_PROXY || ctx.env.HTTPS_PROXY };
         ctx.browser = await chromium.launch(launchOptions);
         ctx.log.debug(`Chromium launched with options ${JSON.stringify(launchOptions)}`);

src/lib/server.ts

@@ -57,8 +57,7 @@ export default async (ctx: Context): Promise<FastifyInstance<Server, IncomingMes
 		return reply.code(replyCode).send(replyBody);
 	});
 
-
-	await server.listen({ port: 49152 });
+	await server.listen({ port: ctx.options.port });
 	// store server's address for SDK
 	let { port } = server.addresses()[0];
 	process.env.SMARTUI_SERVER_ADDRESS = `http://localhost:${port}`;

src/types.ts

@@ -30,6 +30,7 @@ export interface Context {
         parallel?: boolean,
         markBaseline?: boolean,
         buildName?: string,
+        port?: number,
     }
     cliVersion: string;
     totalSnapshots: number;