@mablhq/mabl-cli 2.79.1 → 2.79.9

package/commands/mcp/mcp_cmds/utils/testUtils.js

@@ -7,6 +7,7 @@ exports.getTestOutputUrl = getTestOutputUrl;
 exports.getCreateTestOutputText = getCreateTestOutputText;
 const zod_1 = require("zod");
 const env_1 = require("../../../../env/env");
+const openUtils_1 = require("../../../../core/trainer/openUtils");
 exports.testOutputSchema = zod_1.z.object({
     id: zod_1.z.string(),
     name: zod_1.z.string(),
@@ -53,7 +54,7 @@ function getTestOutputUrl(workspaceId, testRunId) {
     return `${env_1.BASE_APP_URL}/workspaces/${workspaceId}/output/test-runs/${testRunId}`;
 }
 function getCreateTestOutputText(trainerUrl) {
-    return `
-    A mabl trainer was launched. If you don't see the trainer running, open the following URL: ${trainerUrl} to open the trainer manually (Show the URL to the user using and explain to the user how to open it. If you have a tool that can open a link use it.).
-  `;
+    return process.platform === openUtils_1.WIN32_PLATFORM
+        ? `Tell the user to open the following URL to launch the mabl trainer: ${trainerUrl}. Show it using a link, trying to hide the full URL. If you have a tool that can open a link use it.`
+        : `A mabl trainer was launched. If you don't see the trainer running, open the following URL: ${trainerUrl} to open the trainer manually (Show the URL to the user using and explain to the user how to open it. If you have a tool that can open a link use it.).`;
 }

package/commands/tests/testsUtil.js

@@ -408,44 +408,42 @@ function validateRunCommandWithLabels(testId, suppliedLabelsInclude, suppliedLab
     }
     return true;
 }
-async function pullDownTestRunConfig(testRunId, apiClient) {
-    const journeyRun = await apiClient.getTestRun(testRunId);
-    const planRun = await apiClient.getPlanRun(journeyRun.parent_execution);
-    const testDatatablevariables = journeyRun?.journey_parameters?.user_variables &&
-        (0, utils_1.variableRowAsScenario)(journeyRun?.journey_parameters?.user_variables);
-    const dataTableId = testDatatablevariables?.table_id;
-    if (dataTableId) {
-        await apiClient.getDataTable(dataTableId);
-    }
+function buildTestRunConfig(testRun, filterHttpRequests, planRun) {
+    const testDatatablevariables = testRun.journey_parameters?.user_variables &&
+        (0, utils_1.variableRowAsScenario)(testRun.journey_parameters?.user_variables);
     return {
-        basicAuthCredentialsId: planRun?.run_policy?.http_auth_credentials_required
-            ? planRun?.run_policy?.http_auth_credentials_id
-            : undefined,
-        branchName: journeyRun.journey_parameters?.source_control_tag,
-        credentialsId: planRun.run_policy?.credentials_required
-            ? planRun.run_policy?.credentials_id
-            : undefined,
+        basicAuthCredentialsId: testRun.journey_parameters?.http_auth_credentials_id ??
+            (planRun?.run_policy?.http_auth_credentials_required
+                ? planRun?.run_policy?.http_auth_credentials_id
+                : undefined),
+        branchName: testRun.journey_parameters?.source_control_tag,
+        credentialsId: testRun.journey_parameters?.credentials_id ??
+            (planRun?.run_policy?.credentials_required
+                ? planRun?.run_policy?.credentials_id
+                : undefined),
         dataTableVariables: testDatatablevariables,
-        deviceEmulation: journeyRun.journey_parameters?.device_emulation,
-        environmentId: journeyRun.journey_parameters?.deployment?.environment_id,
+        deviceEmulation: testRun.journey_parameters?.device_emulation,
+        environmentId: testRun.journey_parameters?.deployment?.environment_id,
         extraHttpHeaders: planRun?.run_policy?.http_headers_required === true
-            ? planRun?.run_policy?.http_headers?.reduce((headers, header) => ({
-                ...headers,
-                [header.name]: header.value ?? '',
-            }), {})
+            ? headerArrayToRecord(planRun?.run_policy?.http_headers)
             : undefined,
-        filterHttpRequests: false,
-        importedVariables: journeyRun.journey_parameters?.imported_variables,
-        linkCrawlerStartingUrlOverride: getLinkCrawlerUrlOverride(journeyRun),
-        localizationOptions: journeyRun.localization_options,
-        pageLoadWait: journeyRun.journey_parameters?.page_load_wait,
-        runId: journeyRun.id,
-        testId: journeyRun.journey?.invariant_id,
-        url: journeyRun.journey_parameters?.deployment?.uri,
+        filterHttpRequests,
+        importedVariables: testRun.journey_parameters?.imported_variables,
+        linkCrawlerStartingUrlOverride: getLinkCrawlerUrlOverride(testRun),
+        localizationOptions: testRun.localization_options,
+        pageLoadWait: testRun.journey_parameters?.page_load_wait,
+        runId: testRun.id,
+        testId: testRun.journey?.invariant_id,
+        url: testRun.journey_parameters?.deployment?.uri,
     };
 }
+async function pullDownTestRunConfig(testRunId, apiClient) {
+    const testRun = await apiClient.getTestRun(testRunId);
+    const planRun = await apiClient.getPlanRun(testRun.parent_execution);
+    return buildTestRunConfig(testRun, false, planRun);
+}
 async function extractTestRunConfig(executionMessage, apiClient) {
-    const journeyRun = executionMessage.journey_run ??
+    const testRun = executionMessage.journey_run ??
         (await apiClient.getTestRun(executionMessage.journey_run_id));
     const planRun = executionMessage.plan_run;
     const maybeRunnerType = planRun?.run_policy
@@ -453,31 +451,8 @@ async function extractTestRunConfig(executionMessage, apiClient) {
         ?
             planRun?.run_policy?.nodejs_runtime_variant
         : undefined;
-    const config = {
-        basicAuthCredentialsId: planRun?.run_policy?.http_auth_credentials_required
-            ? planRun?.run_policy?.http_auth_credentials_id
-            : undefined,
-        branchName: journeyRun?.journey_parameters?.source_control_tag,
-        credentialsId: planRun?.run_policy?.credentials_required
-            ? planRun?.run_policy?.credentials_id
-            : undefined,
-        dataTableVariables: journeyRun?.journey_parameters?.user_variables &&
-            (0, utils_1.variableRowAsScenario)(journeyRun?.journey_parameters?.user_variables),
-        deviceEmulation: journeyRun?.journey_parameters?.device_emulation,
-        environmentId: journeyRun?.journey_parameters?.deployment?.environment_id,
-        extraHttpHeaders: planRun?.run_policy?.http_headers_required === true
-            ? headerArrayToRecord(planRun?.run_policy?.http_headers)
-            : undefined,
-        filterHttpRequests: true,
-        importedVariables: journeyRun.journey_parameters?.imported_variables,
-        linkCrawlerStartingUrlOverride: getLinkCrawlerUrlOverride(journeyRun),
-        localizationOptions: journeyRun.localization_options,
-        pageLoadWait: journeyRun.journey_parameters?.page_load_wait,
-        runId: journeyRun.id,
-        runnerType: maybeRunnerType,
-        testId: journeyRun?.journey?.invariant_id,
-        url: journeyRun?.journey_parameters?.deployment?.uri,
-    };
+    const config = buildTestRunConfig(testRun, true, planRun);
+    config.runnerType = maybeRunnerType;
     if (executionMessage.test_type === mablApi_1.TestTypeEnum.Mobile) {
         const mobileMessage = executionMessage;
         config.mobileConfig = {

package/core/trainer/openUtils.js

@@ -63,12 +63,17 @@ async function createConfigServer(config) {
     return new Promise((resolve, reject) => {
         server.listen(0, () => {
             const port = server.address().port;
-            const requestPromise = new Promise((resolveRequest, rejectRequest) => {
-                const timeout = (0, timers_1.setTimeout)(() => {
-                    rejectRequest(new Error('Failed to open the trainer automatically. User will open the trainer manually.'));
-                }, 10_000);
+            const requestPromise = new Promise((resolveRequest) => {
+                let timeout;
+                if (process.platform !== exports.WIN32_PLATFORM) {
+                    timeout = (0, timers_1.setTimeout)(() => {
+                        resolveRequest();
+                    }, 20_000);
+                }
                 server.on('close', () => {
-                    global.clearTimeout(timeout);
+                    if (timeout) {
+                        global.clearTimeout(timeout);
+                    }
                     resolveRequest();
                 });
             });