Merge pull request #341 from Nick-1234531/DOT-5815

changes to add pageUrl in capture flow

Author: sushobhit-lt

src/lib/httpClient.ts

@@ -398,7 +398,7 @@ export default class httpClient {
 
     uploadScreenshot(
         { id: buildId, name: buildName, baseline }: Build,
-        ssPath: string, ssName: string, browserName: string, viewport: string, log: Logger
+        ssPath: string, ssName: string, browserName: string, viewport: string, url: string = '', log: Logger
     ) {
         browserName = browserName === constants.SAFARI ? constants.WEBKIT : browserName;
         const file = fs.readFileSync(ssPath);
@@ -410,6 +410,7 @@ export default class httpClient {
         form.append('buildName', buildName);
         form.append('screenshotName', ssName);
         form.append('baseline', baseline.toString());
+        form.append('pageUrl',url)
 
         return this.axiosInstance.request({
             url: `/screenshot`,

src/lib/screenshot.ts

@@ -57,7 +57,7 @@ async function captureScreenshotsForConfig(
 
             await page?.screenshot({ path: ssPath, fullPage });
 
-            await ctx.client.uploadScreenshot(ctx.build, ssPath, name, browserName, viewportString, ctx.log);
+            await ctx.client.uploadScreenshot(ctx.build, ssPath, name, browserName, viewportString, url, ctx.log);
         }
     } catch (error) {
         throw new Error(`captureScreenshotsForConfig failed for browser ${browserName}; error: ${error}`);