Address comments

Author: sergioifg94

src/swagger/client.ts

@@ -13,6 +13,7 @@ import "./config-utils";
 import {
   FUNCTIONAL_TESTING_API_KEY_HEADER,
   FunctionalTestingAPI,
+  FunctionalTestingApiEnv,
 } from "./client/functional-testing-api";
 import type {
   GetFunctionalTestingExecutionParams,
@@ -117,6 +118,7 @@ export class SwaggerClient implements Client {
       this.ftApi = new FunctionalTestingAPI(
         () => this.getFtAuthToken(),
         USER_AGENT,
+        this.getFtEnv(),
       );
     }
   }
@@ -151,6 +153,12 @@ export class SwaggerClient implements Client {
     return this._ftApiToken || null;
   }
 
+  getFtEnv(): FunctionalTestingApiEnv {
+    return process.env.NODE_ENV === "development"
+      ? FunctionalTestingApiEnv.Development
+      : FunctionalTestingApiEnv.Production;
+  }
+
   isConfigured(): boolean {
     return this.api !== undefined || this.ftApi !== undefined;
   }

src/swagger/client/functional-testing-api.ts

@@ -9,12 +9,23 @@ const API_HOSTNAME = "api.reflect.run";
 
 export const FUNCTIONAL_TESTING_API_KEY_HEADER = "X-API-KEY";
 
+export enum FunctionalTestingApiEnv {
+  Production,
+  Development,
+}
+
+const URL_BASES: Record<FunctionalTestingApiEnv, string> = {
+  [FunctionalTestingApiEnv.Development]: `http://${API_HOSTNAME_DEV}/api`,
+  [FunctionalTestingApiEnv.Production]: `https://${API_HOSTNAME}/v1`,
+};
+
 export class FunctionalTestingAPI {
   private urlBase: string | undefined;
 
   constructor(
     private readonly getToken: () => string | null,
     private readonly userAgent: string,
+    private readonly env: FunctionalTestingApiEnv = FunctionalTestingApiEnv.Production,
   ) {}
 
   getFtHeaders(): Record<string, string> {
@@ -89,18 +100,13 @@ export class FunctionalTestingAPI {
   }
 
   /**
-   * Get the URL base for API calls. When running against a local environment (NODE_ENV: development),
-   * the endpoints are exposed at the `/api` path. When running against production, the endpoints
-   * are exposed at `/v1`.
-   *
-   * If NODE_ENV is anything other than `development`, defaults to the production URL
+   * Get the URL base for API calls. When running against a local environment,
+   * the endpoints are exposed at the `/api` path. When running against production,
+   * the endpoints are exposed at `/v1`.
    */
   private getUrlBase(): string {
     if (!this.urlBase) {
-      this.urlBase =
-        process.env.NODE_ENV === "development"
-          ? `http://${API_HOSTNAME_DEV}/api`
-          : `https://${API_HOSTNAME}/v1`;
+      this.urlBase = URL_BASES[this.env];
     }
 
     return this.urlBase;

src/swagger/client/functional-testing-tools.ts

@@ -18,15 +18,15 @@ export const FUNCTIONAL_TESTING_TOOLS: SwaggerToolParams[] = [
     title: "Run Test",
     toolset: "Functional Testing",
     summary:
-      "Runs a specific API test in your Swagger Functional Testing workspace and returns the result. " +
-      "Use this tool when you need to verify expected API functionality by executing a single test.",
+      "Runs a specific API test in your Swagger Functional Testing workspace. The execution is asynchronous — it returns an executionId, not the result directly.  Use swagger_get_test_status with that executionId to track progress and retrieve the final result.",
     inputSchema: RunFunctionalTestingTestParamsSchema,
     handler: "runFunctionalTestingTest",
   },
   {
     title: "Get Test Status",
     toolset: "Functional Testing",
-    summary: "Get the status of a Swagger Functional Testing test execution",
+    summary:
+      "Get the status of a Swagger Functional Testing test execution. It returns information about the execution such as its status (running, passed or failed), run time, as well as the break down of the status of each test step.",
     inputSchema: GetFunctionalTestingExecutionSchema,
     handler: "getFunctionalTestingExecution",
   },

src/tests/unit/swagger/functional-testing/functional-testing-api.test.ts

@@ -1,6 +1,9 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
 import createFetchMock from "vitest-fetch-mock";
-import { FunctionalTestingAPI } from "../../../../swagger/client/functional-testing-api";
+import {
+  FunctionalTestingAPI,
+  FunctionalTestingApiEnv,
+} from "../../../../swagger/client/functional-testing-api";
 
 const fetchMock = createFetchMock(vi);
 fetchMock.enableMocks();
@@ -18,6 +21,7 @@ describe("FunctionalTestingAPI", () => {
     api = new FunctionalTestingAPI(
       () => "test-api-key",
       "SmartBear MCP Server/test",
+      FunctionalTestingApiEnv.Production,
     );
   });