Merge pull request #226 from LambdaTest/stage

Release version 4.1.1

Author: sushobhit-lt

package.json

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

src/commander/capture.ts

@@ -27,6 +27,7 @@ command
             process.exit(1);
         }
         let ctx: Context = ctxInit(command.optsWithGlobals());
+        ctx.isSnapshotCaptured = true
 
         if (!fs.existsSync(file)) {
             ctx.log.error(`Web Static Config file ${file} not found.`);

src/commander/upload.ts

@@ -34,6 +34,7 @@ command
             process.exit(1);
         }
         let ctx: Context = ctxInit(command.optsWithGlobals());
+        ctx.isSnapshotCaptured = true
 
         if (!fs.existsSync(directory)) {
             console.log(`Error: The provided directory ${directory} not found.`);

src/commander/uploadFigma.ts

@@ -24,6 +24,7 @@ uploadFigma
     .option('--buildName <buildName>', 'Name of the build')
     .action(async function (file, _, command) {
         let ctx: Context = ctxInit(command.optsWithGlobals());
+        ctx.isSnapshotCaptured = true;
 
         if (!fs.existsSync(file)) {
             console.log(`Error: Figma Config file ${file} not found.`);

src/lib/ctx.ts

@@ -127,6 +127,7 @@ export default (options: Record<string, string>): Context => {
         },
         cliVersion: version,
         totalSnapshots: -1,
-        isStartExec: false
+        isStartExec: false,
+        isSnapshotCaptured: false
     }
 }

src/lib/server.ts

@@ -38,6 +38,7 @@ export default async (ctx: Context): Promise<FastifyInstance<Server, IncomingMes
 			if (!validateSnapshot(snapshot)) throw new Error(validateSnapshot.errors[0].message);
 			ctx.testType = testType;
 			ctx.snapshotQueue?.enqueue(snapshot);
+			ctx.isSnapshotCaptured = true;
 			replyCode = 200;
 			replyBody = { data: { message: "success", warnings: [] }};
 		} catch (error: any) {

src/tasks/finalizeBuild.ts

@@ -28,8 +28,11 @@ export default (ctx: Context): ListrTask<Context, ListrRendererFactory, ListrRen
                 ctx.log.debug(`Closed browser`);
                 await ctx.server?.close();
                 ctx.log.debug(`Closed server`);
-                let resp = await ctx.client.getS3PreSignedURL(ctx);
-                await ctx.client.uploadLogs(ctx, resp.data.url);
+                if (ctx.isSnapshotCaptured) {
+                    ctx.log.debug(`Log file to be uploaded`)
+                    let resp = await ctx.client.getS3PreSignedURL(ctx);
+                    await ctx.client.uploadLogs(ctx, resp.data.url);
+                }
                 fs.unlinkSync(constants.LOG_FILE_PATH);
                 ctx.log.debug(`Log file deleted: ${constants.LOG_FILE_PATH}`);
             } catch (error: any) {

src/types.ts

@@ -53,6 +53,7 @@ export interface Context {
     figmaDesignConfig?: FigmaDesignConfig;
     testType?: string;
     isStartExec ?: boolean;
+    isSnapshotCaptured ?: boolean;
 }
 
 export interface Env {