Merge pull request #280 from LambdaTest/prod

Prod

Author: parthlambdatest

package.json

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

src/lib/ctx.ts

@@ -102,7 +102,7 @@ export default (options: Record<string, string>): Context => {
             skipBuildCreation: config.skipBuildCreation ?? false,
             tunnel: config.tunnel ?? false,
             tunnelName: config.tunnelName || '',
-            userAgent: config.userAgent || '',
+            userAgent: config.userAgent || ''
         },
         uploadFilePath: '',
         webStaticConfig: [],

src/lib/processSnapshot.ts

@@ -180,7 +180,7 @@ export default async function processSnapshot(snapshot: Snapshot, ctx: Context):
                     ctx.log.debug(`Resource had a disallowed status for retry as well  ${requestUrl} disallowed status [${responseOfRetry.status()}]`);
                     if (responseOfRetry && responseOfRetry.headers()) {
                         const responseHeadersRetry = responseOfRetry.headers();
-                        ctx.log.debug(`Response headers for ${requestUrl}: ${JSON.stringify(responseHeadersRetry, null, 2)}`);
+                        ctx.log.debug(`Response headers for retry ${requestUrl}: ${JSON.stringify(responseHeadersRetry, null, 2)}`);
                     }
 
                     let data = {
@@ -203,8 +203,6 @@ export default async function processSnapshot(snapshot: Snapshot, ctx: Context):
                         discoveryErrors.browsers[globalBrowser][globalViewport]?.push(data);
                     }
                 }
-
-
             } else {
                 ctx.log.debug(`Handling request ${requestUrl}\n - content-type ${response.headers()['content-type']}`);
 

src/lib/schemaValidation.ts

@@ -198,7 +198,7 @@ const ConfigSchema = {
         },
         userAgent: {
             type: "string",
-            errorMessage: "Invalid config; userAgent must be string"
+            errorMessage: "User Agent value must be a valid string"
         },
     },
     anyOf: [

src/lib/screenshot.ts

@@ -35,7 +35,7 @@ async function captureScreenshotsForConfig(
         if(ctx.config.userAgent !== ""){
         contextOptions.userAgent = ctx.config.userAgent;
         }
-        if (urlConfig.userAgent !== "" && urlConfig.userAgent !== undefined) {
+        if (userAgent && userAgent !== "") {
             contextOptions.userAgent = userAgent;
         }
     }