Instrumentation for sync cli (#1514)

percy · Feb 6, 2024 · 413f06b · 413f06b
1 parent 07bd7ce
commit 413f06b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/packages/webdriver-utils/src/providers/automateProvider.js b/packages/webdriver-utils/src/providers/automateProvider.js
@@ -114,7 +114,8 @@ export default class AutomateProvider extends GenericProvider {
           percyScreenshotUrl: this.buildInfo?.url,
           status: error ? 'failure' : 'success',
           statusMessage: error ? `${error}` : '',
-          state: 'end'
+          state: 'end',
+          sync: this.options?.sync
         });
       } catch (e) {
         log.debug(`[${name}] : Could not execute percyScreenshot command for Automate`);

diff --git a/packages/webdriver-utils/test/providers/automateProvider.test.js b/packages/webdriver-utils/test/providers/automateProvider.test.js
@@ -237,6 +237,19 @@ describe('AutomateProvider', () => {
           args: []
         });
     });
+
+    it('passes sync options value to', async () => {
+      const automateProvider = new AutomateProvider('1234', 'https://localhost/command-executor', { platform: 'win' }, {}, 'client', 'environment', { sync: true }, percyBuildInfo);
+      await automateProvider.createDriver();
+      automateProvider.driver.executeScript = jasmine.createSpy().and.resolveTo('success');
+      await automateProvider.percyScreenshotEnd('abc', undefined);
+
+      expect(automateProvider.driver.executeScript).toHaveBeenCalledWith(
+        {
+          script: `browserstack_executor: {"action":"percyScreenshot","arguments":{"name":"abc","percyScreenshotUrl":"${percyBuildInfo.url}","status":"success","statusMessage":"","state":"end","sync":true}}`,
+          args: []
+        });
+    });
   });
 
   function tilesErrorResponseCheck(automateProvider) {