Merge pull request #409 from Nick-1234531/DOT-6254

snapshot name trim and encoded for snapshot status API

Author: parthlambdatest

package.json

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

src/lib/httpClient.ts

@@ -681,8 +681,13 @@ export default class httpClient {
 
     getSnapshotStatus(buildId: string, snapshotName: string, snapshotUuid: string, ctx: Context): Promise<Record<string, any>> {
         return this.request({
-            url: `/snapshot/status?buildId=${buildId}&snapshotName=${snapshotName}&snapshotUUID=${snapshotUuid}`,
+            url: `/snapshot/status`,
             method: 'GET',
+            params: {
+                buildId,
+                snapshotName,
+                snapshotUUID: snapshotUuid
+            },
             headers: {
                 'Content-Type': 'application/json',
             }

src/lib/server.ts

@@ -241,10 +241,12 @@ export default async (ctx: Context): Promise<FastifyInstance<Server, IncomingMes
 
 		try {
 			ctx.log.debug(`request.query : ${JSON.stringify(request.query)}`);
-			const { contextId, pollTimeout, snapshotName } = request.query as { contextId: string, pollTimeout: number, snapshotName: string };
+			const { contextId, pollTimeout, snapshotName: rawSnapshotName } = request.query as { contextId: string, pollTimeout: number, snapshotName: string };
+			const snapshotName = rawSnapshotName?.trim();
 			if (!contextId || !snapshotName) {
 				throw new Error('contextId and snapshotName are required parameters');
 			}
+			
 
 			const timeoutDuration = pollTimeout*1000 || 30000;