@testim/testim-cli 3.255.0 → 3.256.0

package/runners/ParallelWorkerManager.js

@@ -54,10 +54,11 @@ class ParallelWorkerManager {
 
         let stoppedOnError = false;
         let runningTests = 0;
-        const runAndWaitToComplete = token => new Promise((resolve, reject) => {
+        const runAndWaitToComplete = token => new Promise((resolve) => {
             const projectId = options.project;
             const executionQueue = new ExecutionQueue(executionId, executionName, testList, options, branchToUse, testStatus);
 
+            /** @type {{ [testResultId: string]: any; }} */
             const combinedTestResults = {};
             const testCount = testList.length;
 
@@ -177,7 +178,7 @@ class ParallelWorkerManager {
                 }
             };
 
-            const onGridSlot = (executionId, resultId, gridInfo) => testStatus.onGridSlot(executionId, resultId, gridInfo);
+            const onGridSlot = (_executionId, resultId, gridInfo) => testStatus.onGridSlot(_executionId, resultId, gridInfo);
 
             options.userData = {
                 loginData: Object.assign({}, testimCustomToken.getTokenV3UserData(), {

package/runners/TestPlanRunner.js

@@ -39,12 +39,13 @@ class TestPlanRunner {
      * @param {any[]} tests
      * @param {any[]} afterTests
      * @param {string} branchToUse
-     * @param {*} tpOptions
+     * @param {import('../runOptions').RunnerOptions} tpOptions
      * @param {string} executionId
      * @param {string} executionName
      * @param {TestRunStatus} testStatus
      */
     async runTestAllPhases(beforeTests, tests, afterTests, branchToUse, tpOptions, executionId, executionName, testStatus) {
+        /** @type {{ [testResultId: string]: any; }} */
         const executionResults = {};
         const authData = testimCustomToken.getTokenV3UserData();
 
@@ -145,6 +146,7 @@ class TestPlanRunner {
         const childTestResults = {};
         realDataService.joinToTestResultsByRunId(runId, projectId);
         let isPromisePending = true;
+        /** @type {Promise<any[]>} */
         const promise = new Promise(_resolve => {
             const resolve = (val) => {
                 isPromisePending = false;
@@ -226,17 +228,18 @@ class TestPlanRunner {
         Logger.setExecutionId(executionId);
         beforeTests.forEach(test => { test.isBeforeTestPlan = true; });
         afterTests.forEach(test => { test.isAfterTestPlan = true; });
-        const testStatus = new TestRunStatus(_.concat(beforeTests, tests, afterTests), tpOptions, testPlanId, branch);
+        const testStatus = new TestRunStatus([].concat(beforeTests, tests, afterTests), tpOptions, testPlanId, branch);
 
-        const configs = _.chain(_.concat(beforeTests, tests, afterTests))
+        const configs = _.chain([].concat(beforeTests, tests, afterTests))
             .map(test => test.overrideTestConfig?.name || '')
             .uniq()
             .compact()
             .value();
+        /** @type {string | null} */
         const configName = configs?.length === 1 ? configs[0] : null;
 
         const isCodeMode = tpOptions.files.length > 0;
-        const testNames = tpOptions.lightweightMode?.onlyTestIdsNoSuite ? [] : _.concat(beforeTests, tests, afterTests).map(test => test.name);
+        const testNames = tpOptions.lightweightMode?.onlyTestIdsNoSuite ? [] : [].concat(beforeTests, tests, afterTests).map(test => test.name);
 
         const testListInfoPromise = tpOptions.lightweightMode?.onlyTestIdsNoSuite ?
             { beforeTests, tests, afterTests } :
@@ -375,13 +378,13 @@ class TestPlanRunner {
                 suiteResult.runName = `rerun-${options.rerunFailedByRunId}`;
             }
         }
-        const testPlanName = options.overrideExecutionName || suiteResult.runName || _.concat(options.label, options.name, options.suites).join(' ');
+        const testPlanName = options.overrideExecutionName || suiteResult.runName || [].concat(options.label, options.name, options.suites).join(' ');
         const isAnonymous = true;
         perf.log('Right before validateConfig + runAnonymousTestPlan tests map');
         return await utils.promiseMap(suiteResult.tests, async suiteTests => { // array of results per execution
             //override result id for remote run mode and run only the first test data
             if (options.resultId) {
-                const firstTest = _.first(suiteTests);
+                const firstTest = suiteTests[0];
                 firstTest.resultId = options.resultId;
                 suiteTests = [firstTest];
             }

package/runners/buildCodeTests.js

@@ -122,6 +122,7 @@ async function buildCodeTests(files, webpackConfig = { mode: 'development' }, ru
         throw new ArgError(`Compilation Webpack Error in tests: ${e.message}`);
     }
 
+    /** @type {{ code: string; name: string; }[]} */
     const fileResults = files.map((file, i) => ({ code: mfs.readFileSync(path.resolve('./dist', `${fileHashes[i]}.bundle.js`)), name: file })); // read all files
 
     suite.tests = [fileResults.map(({ code, name }) => ({

package/runners/runnerUtils.js

@@ -7,9 +7,14 @@ const analytics = require('../commons/testimAnalytics');
 const { ArgError } = require('../errors');
 
 
+/**
+ * @param {import('../runOptions').RunnerOptions} options
+ * @param {string} branchToUse
+ * @returns
+ */
 async function getSuite(options, branchToUse) {
     if (options.lightweightMode?.onlyTestIdsNoSuite && options.testId) {
-        return { tests: [options.testId.map(testId => ({ testId, testConfig: { }, resultId: utils.guid() }))] };
+        return { tests: [options.testId.map(testId => ({ testId, testConfig: {}, resultId: utils.guid() }))] };
     }
     // local code test
     if (options.files.length > 0) {
@@ -47,7 +52,11 @@ function calcTestResultStatus(tests) {
 }
 
 
-
+/**
+ * @template T
+ * @param {import('../runOptions').RunnerOptions} options
+ * @param {T[]} testList
+ */
 async function validateConfig(options, testList) {
     const supportedBrowsers = options.mode === 'extension' ? [
         'edge-chromium', 'chrome',

package/services/branchService.js

@@ -1,12 +1,18 @@
-"use strict";
+'use strict';
 
+/** @type {string | undefined} */
 let currentBranch;
 
 function getCurrentBranch() {
-    return currentBranch || "master";
+    return currentBranch || 'master';
 }
-function setCurrentBranch(branchData = "master", acknowledgeAutoDetect = "false") {
-    if (branchData && branchData.branch && branchData.branch === 'master') {
+
+/**
+ * @param {Awaited<ReturnType<import('../commons/initializeUserWithAuth')['initializeUserWithAuth']>>['branchName']} branchData
+ * @param {boolean | string} acknowledgeAutoDetect
+ */
+function setCurrentBranch(branchData = 'master', acknowledgeAutoDetect = 'false') {
+    if (branchData?.branch && branchData.branch === 'master') {
         currentBranch = 'master';
         return;
     }
@@ -19,5 +25,5 @@ function setCurrentBranch(branchData = "master", acknowledgeAutoDetect = "false"
 
 module.exports = {
     getCurrentBranch,
-    setCurrentBranch
+    setCurrentBranch,
 };

package/services/localRCASaver.js

@@ -37,6 +37,10 @@ function mapFilesToLocalDrive(test, logger) {
     }
 }
 
+/**
+ * @param {Partial<Pick<import('../runOptions').AgentModeOptions, 'agentPort' | 'agentBind'> & Pick<import('../runOptions').RunnerOptions, 'saveRCALocally'>>} param0
+ * @returns {Promise<import('net').AddressInfo>}
+ */
 async function initServer({ agentPort, agentBind, saveRCALocally }) {
     const multer = await lazyRequire('multer');
     saveRCALocally = typeof saveRCALocally === 'string' ? saveRCALocally : DEFUALT_PATH;