@testim/testim-cli 3.266.0 → 3.268.0

package/workers/WorkerSelenium.js

@@ -17,31 +17,39 @@ const { preloadTests } = require('../commons/preloadTests');
 const NO_NAVIGATION_TIME_LIMIT = 1e9;
 
 class WorkerSelenium extends BaseWorker {
-    constructor(...args) {
-        super(...args);
-        this.getBrowserOnce = Promise.method(this.getBrowserOnce);
-    }
-
+    /**
+     * @override
+     * @param {import('../testRunHandler')} testRunHandler
+     */
     initPlayer(testRunHandler) {
-        return new SeleniumTestPlayer(this.id,
-            testRunHandler.getRunParams(),
+        return new SeleniumTestPlayer(
+            this.id,
+            testRunHandler.runParams,
             this.options.shouldMonitorPerformance,
-            testRunHandler.getAutomationMode(),
+            testRunHandler.automationMode,
             undefined,
-            testRunHandler.getRetryCount(),
-            testRunHandler.getPreviousTestResultId());
+            testRunHandler.retryCount,
+            testRunHandler.previousTestResultId,
+        );
     }
 
+    /**
+     * @override
+     * @param {import('../testRunHandler')} testRunHandler
+     * @param {string=} customExtensionLocalLocation
+     * @param {SeleniumTestPlayer} seleniumTestPlayer
+     * @param {Awaited<ReturnType<import('../services/gridService')['handleHybridOrVendorIfNeeded']>>} gridInfo
+     */
     async getBrowserOnce(testRunHandler, customExtensionLocalLocation, seleniumTestPlayer, gridInfo) {
         perf.log('in WorkerSelenium getBrowserOnce');
-        reporter.onGetSession(this.id, this.testName, testRunHandler.getRunMode());
+        reporter.onGetSession(this.id, this.testName, testRunHandler.runMode);
         const { driver } = seleniumTestPlayer;
 
         this.windowUtils = new WindowUtils(this.id, driver);
         seleniumTestPlayer.clearSessionTabs();
 
         const { browserValue } = this.testRunConfig;
-        const baseUrl = testRunHandler.getBaseUrl();
+        const baseUrl = testRunHandler.baseUrl;
 
         try {
             const fastInit = this.options.useLocalChromeDriver;
@@ -81,6 +89,7 @@ class WorkerSelenium extends BaseWorker {
     }
 
     /**
+     * @override
      * @param {import('../testRunHandler')} testRunHandler
      * @param {import('../player/seleniumTestPlayer')} seleniumTestPlayer
      */
@@ -92,12 +101,12 @@ class WorkerSelenium extends BaseWorker {
 
         setupCliPerformanceMonitoring(sessionPlayer);
 
-        sessionPlayer.playbackManager.executionId = testRunHandler.getExecutionId();
-        sessionPlayer.playbackManager.executionName = testRunHandler.getExecutionName();
+        sessionPlayer.playbackManager.executionId = testRunHandler.executionId;
+        sessionPlayer.playbackManager.executionName = testRunHandler.executionName;
 
         sessionPlayer.setLightweightMode(this.options.lightweightMode);
-        if (testRunHandler.getSfdcCredential()) {
-            sessionPlayer.setSfdcCredential(testRunHandler.getSfdcCredential());
+        if (testRunHandler.sfdcCredential) {
+            sessionPlayer.setSfdcCredential(testRunHandler.sfdcCredential);
         }
         if (sessionPlayerInit.localAssetService) {
             sessionPlayerInit.localAssetService.initialize({ serverUrl: this.options.localRCASaver });
@@ -110,7 +119,7 @@ class WorkerSelenium extends BaseWorker {
         }
 
         async function runSeleniumTest() {
-            if (testRunHandler.getAutomationMode() === 'codeful') {
+            if (testRunHandler.automationMode === 'codeful') {
                 // Testim Development Kit test;
                 if (!sessionPlayer.callOrderScheduler) { // old session player
                     await testRunHandler.waitForExecutionStartedFinished();
@@ -133,8 +142,8 @@ class WorkerSelenium extends BaseWorker {
                     false,
                     this.overrideTestConfigId,
                     this.branch,
-                    testRunHandler.getCode(),
-                    testRunHandler.getTestName()
+                    testRunHandler.code,
+                    testRunHandler.testName
                 ).catch(reject))
                     .log('right after playTestByCode')
                     .timeout(this.testRunTimeout, timeoutMessages.TEST_COMPLETE_TIMEOUT_MSG)
@@ -164,7 +173,7 @@ class WorkerSelenium extends BaseWorker {
                     this.overrideTestConfigId,
                     this.branch,
                     INCOGNITO,
-                    testRunHandler.getRemoteRunId(),
+                    testRunHandler.remoteRunId,
                     undefined,
                     undefined,
                     preloadedTest