Adding CLI Metrics to measure client duration (#1350)

percy · Aug 30, 2023 · 68e43af · 68e43af
1 parent bac46d4
commit 68e43af
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/packages/core/src/config.js b/packages/core/src/config.js
@@ -445,7 +445,9 @@ export const comparisonSchema = {
         windowHeight: {
           type: 'integer',
           minimum: 0
-        }
+        },
+        cliScreenshotStartTime: { type: 'integer', default: 0 },
+        cliScreenshotEndTime: { type: 'integer', default: 0 }
       }
     },
     tag: {

diff --git a/packages/webdriver-utils/src/index.js b/packages/webdriver-utils/src/index.js
@@ -34,12 +34,16 @@ export default class WebdriverUtils {
 
   async automateScreenshot() {
     try {
+      const startTime = Date.now();
       this.log.info(`[${this.snapshotName}] : Starting automate screenshot ...`);
       const automate = ProviderResolver.resolve(this.sessionId, this.commandExecutorUrl, this.capabilities, this.sessionCapabilites, this.clientInfo, this.environmentInfo, this.options, this.buildInfo);
       this.log.debug(`[${this.snapshotName}] : Resolved provider ...`);
       await automate.createDriver();
       this.log.debug(`[${this.snapshotName}] : Created driver ...`);
-      return await automate.screenshot(this.snapshotName, this.options);
+      const comparisonData = await automate.screenshot(this.snapshotName, this.options);
+      comparisonData.metadata.cliScreenshotStartTime = startTime;
+      comparisonData.metadata.cliScreenshotEndTime = Date.now();
+      return comparisonData;
     } catch (e) {
       this.log.error(`[${this.snapshotName}] : Error - ${e.message}`);
       this.log.error(`[${this.snapshotName}] : Error Log - ${e.toString()}`);