fix(loadtesting): correct tenant name in url (Azure#37277)

### Packages impacted by this PR
azure/playwright

### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: kashish2508

sdk/loadtesting/playwright/src/common/constants.ts

@@ -68,6 +68,27 @@ export const Constants = {
   HTTP_CALL_TIMEOUT: 10000,
 };
 
+export const ArmConstants = {
+  TenantsApiUrl: "https://management.azure.com/tenants",
+  TenantsApiVersion: "2025-04-01",
+};
+
+export const UrlConstants = {
+  AzurePortalBaseUrl: "https://ms.portal.azure.com",
+  ReportingApiSubdomain: "reporting.api",
+  ApiSubdomain: "api",
+  PlaywrightWorkspacesPath: "playwrightworkspaces",
+  TestRunsPath: "test-runs",
+  LoadTestServiceProvider: "Microsoft.LoadTestService",
+  PlaywrightWorkspacesResourceType: "playwrightWorkspaces",
+  TestRunsRoute: "TestRuns",
+  ResourceGroupsPath: "resourcegroups",
+  ResourcePath: "/resource",
+  SubscriptionsPath: "/subscriptions",
+  ResourceGroupsUrlPath: "/resourceGroups",
+  ProvidersPath: "/providers",
+};
+
 export const InternalEnvironmentVariables = {
   MPT_PLAYWRIGHT_VERSION: "_MPT_PLAYWRIGHT_VERSION",
   MPT_SETUP_FATAL_ERROR: "_MPT_SETUP_FATAL_ERROR",

sdk/loadtesting/playwright/src/common/types.ts

@@ -212,6 +212,11 @@ export type WorkspaceMetaData = {
   reporting?: string;
 };
 
+export type TenantInfo = {
+  tenantId?: string;
+  defaultDomain?: string;
+};
+
 export interface UploadResult {
   success: boolean;
   errorMessage?: string;

sdk/loadtesting/playwright/src/reporter/playwrightReporter.ts

@@ -6,6 +6,7 @@ import {
   getHtmlReporterOutputFolder,
   getPortalTestRunUrl,
   getVersionInfo,
+  resolveTenantDomain,
 } from "../utils/utils.js";
 import { coreLogger } from "../common/logger.js";
 import { PlaywrightServiceConfig } from "../common/playwrightServiceConfig.js";
@@ -22,6 +23,7 @@ import type { WorkspaceMetaData, UploadResult } from "../common/types.js";
 export default class PlaywrightReporter implements Reporter {
   private config: FullConfig | undefined;
   private workspaceMetadata: WorkspaceMetaData | null = null;
+  private tenantDomain: string | undefined;
   private isReportingEnabled = false;
 
   /**
@@ -107,6 +109,20 @@ export default class PlaywrightReporter implements Reporter {
         return;
       }
 
+      // Resolve tenant domain for portal URL (if tenantId is available)
+      if (this.workspaceMetadata.tenantId) {
+        try {
+          const tenants = await playwrightServiceApiClient.getTenants();
+          this.tenantDomain = resolveTenantDomain(this.workspaceMetadata.tenantId, tenants);
+        } catch (error) {
+          coreLogger.error(`Failed to resolve tenant domain: ${error}`);
+        }
+      } else {
+        coreLogger.info(
+          "Workspace metadata does not contain tenantId; skipping tenant domain resolution.",
+        );
+      }
+
       this.isReportingEnabled = true;
       console.log(ServiceErrorMessageConstants.REPORTING_ENABLED.message);
     } catch (error) {
@@ -142,7 +158,7 @@ export default class PlaywrightReporter implements Reporter {
         }
         // Display portal URL for both full and partial success
         if (this.workspaceMetadata) {
-          const portalUrl = getPortalTestRunUrl(this.workspaceMetadata);
+          const portalUrl = getPortalTestRunUrl(this.workspaceMetadata, this.tenantDomain);
           console.log(ServiceErrorMessageConstants.TEST_REPORT_VIEW_URL.formatWithUrl(portalUrl));
         }
       } else {

sdk/loadtesting/playwright/src/utils/PlaywrightServiceClient.ts

@@ -8,8 +8,8 @@ import {
 } from "./utils.js";
 import { ServiceErrorMessageConstants } from "../common/messages.js";
 import { HttpService } from "../common/httpService.js";
-import { TestRunCreatePayload, WorkspaceMetaData } from "../common/types.js";
-import { Constants, InternalEnvironmentVariables } from "../common/constants.js";
+import { TestRunCreatePayload, WorkspaceMetaData, TenantInfo } from "../common/types.js";
+import { Constants, InternalEnvironmentVariables, ArmConstants } from "../common/constants.js";
 
 export class PlaywrightServiceClient {
   private httpService: HttpService;
@@ -102,4 +102,39 @@ export class PlaywrightServiceClient {
       throw new Error(ServiceErrorMessageConstants.FAILED_TO_GET_WORKSPACE_METADATA.message);
     }
   }
+
+  async getTenants(): Promise<TenantInfo[]> {
+    try {
+      const token = getAccessToken();
+      if (!token) {
+        throw new Error("PLAYWRIGHT_SERVICE_ACCESS_TOKEN environment variable is not set.");
+      }
+      const url = new URL(ArmConstants.TenantsApiUrl);
+      url.searchParams.set("api-version", ArmConstants.TenantsApiVersion);
+      const method = "GET";
+      const correlationId = crypto.randomUUID();
+
+      const response = await this.httpService.callAPI(
+        method,
+        url.toString(),
+        null,
+        token,
+        "",
+        correlationId,
+      );
+      if (response.status !== 200) {
+        const errorMessage = extractErrorMessage(response?.bodyAsText ?? "");
+        throw new Error(
+          errorMessage || `HTTP ${response.status}: Failed to retrieve tenant information`,
+        );
+      }
+      const responseBody = response.bodyAsText ? JSON.parse(response.bodyAsText) : {};
+      return responseBody.value ?? [];
+    } catch (error) {
+      if (error instanceof Error) {
+        throw error;
+      }
+      throw new Error("Failed to retrieve tenant information.");
+    }
+  }
 }

sdk/loadtesting/playwright/src/utils/utils.ts

@@ -7,13 +7,15 @@ import type {
   JwtPayload,
   RunConfig,
   WorkspaceMetaData,
+  TenantInfo,
 } from "../common/types.js";
 import {
   Constants,
   InternalEnvironmentVariables,
   ServiceEnvironmentVariable,
   RunConfigConstants,
   GitHubActionsConstants,
+  UrlConstants,
 } from "../common/constants.js";
 import { ServiceErrorMessageConstants } from "../common/messages.js";
 import { coreLogger } from "../common/logger.js";
@@ -262,7 +264,7 @@ export function getTestRunApiUrl(): string {
   if (!result?.region || !result?.domain || !result?.accountId) {
     exitWithFailureMessage(ServiceErrorMessageConstants.NO_SERVICE_URL_ERROR);
   }
-  const baseUrl = `https://${result?.region}.reporting.api.${result?.domain}/playwrightworkspaces/${result?.accountId}/test-runs`;
+  const baseUrl = `https://${result?.region}.${UrlConstants.ReportingApiSubdomain}.${result?.domain}/${UrlConstants.PlaywrightWorkspacesPath}/${result?.accountId}/${UrlConstants.TestRunsPath}`;
   const url = runId ? `${baseUrl}/${runId}` : baseUrl;
 
   return `${url}?api-version=${Constants.LatestAPIVersion}`;
@@ -337,7 +339,7 @@ export function getWorkspaceMetaDataApiUrl(): string {
   if (!result?.region || !result?.domain || !result?.accountId) {
     exitWithFailureMessage(ServiceErrorMessageConstants.NO_SERVICE_URL_ERROR);
   }
-  const baseUrl = `https://${result?.region}.api.${result?.domain}/playwrightworkspaces/${result?.accountId}`;
+  const baseUrl = `https://${result?.region}.${UrlConstants.ApiSubdomain}.${result?.domain}/${UrlConstants.PlaywrightWorkspacesPath}/${result?.accountId}`;
 
   return `${baseUrl}?api-version=${Constants.LatestAPIVersion}`;
 }
@@ -473,7 +475,24 @@ export function collectAllFiles(
   return files;
 }
 
-export function getPortalTestRunUrl(workspaceMetadata: WorkspaceMetaData | null): string {
+export function resolveTenantDomain(
+  tenantId: string | undefined,
+  tenants: TenantInfo[],
+): string | undefined {
+  if (!tenantId || tenants.length === 0) {
+    return undefined;
+  }
+  const matchingTenant = tenants.find((t) => t.tenantId === tenantId);
+  coreLogger.info(
+    `Resolved tenant domain: ${JSON.stringify(matchingTenant?.defaultDomain)} for tenant ID: ${tenantId}`,
+  );
+  return matchingTenant?.defaultDomain;
+}
+
+export function getPortalTestRunUrl(
+  workspaceMetadata: WorkspaceMetaData | null,
+  tenantDomain?: string,
+): string {
   const { subscriptionId, resourceId, name } = workspaceMetadata ?? {};
   if (!subscriptionId || !resourceId || !name) {
     throw new Error(
@@ -484,15 +503,16 @@ export function getPortalTestRunUrl(workspaceMetadata: WorkspaceMetaData | null)
   // Extract resource group from resourceId
   const resourceIdParts = resourceId.split("/");
   const resourceGroupIndex = resourceIdParts.findIndex(
-    (part) => part.toLowerCase() === "resourcegroups",
+    (part) => part.toLowerCase() === UrlConstants.ResourceGroupsPath,
   );
 
   if (resourceGroupIndex === -1 || resourceGroupIndex + 1 >= resourceIdParts.length) {
     throw new Error("Invalid resourceId format: could not extract resource group name");
   }
 
   const resourceGroupName = resourceIdParts[resourceGroupIndex + 1];
-  return `https://ms.portal.azure.com/#@microsoft.onmicrosoft.com/resource/subscriptions/${encodeURIComponent(subscriptionId)}/resourceGroups/${encodeURIComponent(resourceGroupName)}/providers/Microsoft.LoadTestService/playwrightWorkspaces/${encodeURIComponent(name)}/TestRuns`;
+  const tenantFragment = tenantDomain ? `#@${tenantDomain}` : "#";
+  return `${UrlConstants.AzurePortalBaseUrl}/${tenantFragment}${UrlConstants.ResourcePath}${UrlConstants.SubscriptionsPath}/${encodeURIComponent(subscriptionId)}${UrlConstants.ResourceGroupsUrlPath}/${encodeURIComponent(resourceGroupName!)}${UrlConstants.ProvidersPath}/${UrlConstants.LoadTestServiceProvider}/${UrlConstants.PlaywrightWorkspacesResourceType}/${encodeURIComponent(name)}/${UrlConstants.TestRunsRoute}`;
 }
 
 export const getStorageAccountNameFromUri = (storageUri: string): string | null => {

sdk/loadtesting/playwright/test/reporter/playwrightReporter.spec.ts

@@ -22,9 +22,12 @@ vi.mock("../../src/utils/playwrightReporterStorageManager.js", () => {
 
 vi.mock("../../src/utils/PlaywrightServiceClient.js", () => {
   const getWorkspaceMetadataMock = vi.fn();
+  const getTenantsMock = vi.fn();
   (globalThis as any).__getWorkspaceMetadataMock = getWorkspaceMetadataMock;
+  (globalThis as any).__getTenantsMock = getTenantsMock;
   class MockApiCall {
     getWorkspaceMetadata = getWorkspaceMetadataMock;
+    getTenants = getTenantsMock;
   }
   return {
     PlaywrightServiceClient: MockApiCall,
@@ -35,12 +38,15 @@ vi.mock("../../src/utils/utils.js", async (importActual) => {
   const actual = await importActual<typeof import("../../src/utils/utils.js")>();
   const getHtmlReporterOutputFolderMock = vi.fn().mockReturnValue("playwright-report");
   const getPortalTestRunUrlMock = vi.fn().mockReturnValue("https://portal/link");
+  const resolveTenantDomainMock = vi.fn().mockReturnValue(undefined);
   (globalThis as any).__getHtmlReporterOutputFolderMock = getHtmlReporterOutputFolderMock;
   (globalThis as any).__getPortalTestRunUrlMock = getPortalTestRunUrlMock;
+  (globalThis as any).__resolveTenantDomainMock = resolveTenantDomainMock;
   return {
     ...actual,
     getHtmlReporterOutputFolder: getHtmlReporterOutputFolderMock,
     getPortalTestRunUrl: getPortalTestRunUrlMock,
+    resolveTenantDomain: resolveTenantDomainMock,
   };
 });
 
@@ -68,6 +74,8 @@ describe("PlaywrightReporter", () => {
     process.env[InternalEnvironmentVariables.TEST_RUN_CREATION_SUCCESS] = "true";
     (globalThis as any).__getHtmlReporterOutputFolderMock.mockReturnValue("playwright-report");
     (globalThis as any).__getPortalTestRunUrlMock.mockReturnValue("https://portal/link");
+    (globalThis as any).__resolveTenantDomainMock.mockReturnValue(undefined);
+    (globalThis as any).__getTenantsMock.mockResolvedValue([]);
     // Set default Playwright version to supported version
     (globalThis as any).__getPlaywrightVersionMock.mockReturnValue("1.57.0");
   });
@@ -128,8 +136,12 @@ describe("PlaywrightReporter", () => {
       resourceId:
         "/subscriptions/sub-id/resourceGroups/my-rg/providers/Microsoft.LoadTestService/playwrightWorkspaces/workspace-name",
       name: "workspace-name",
+      tenantId: "tenant-1",
     };
+    const tenantList = [{ tenantId: "tenant-1", defaultDomain: "contoso.onmicrosoft.com" }];
     (globalThis as any).__getWorkspaceMetadataMock.mockResolvedValue(workspaceMetadata);
+    (globalThis as any).__getTenantsMock.mockResolvedValue(tenantList);
+    (globalThis as any).__resolveTenantDomainMock.mockReturnValue("contoso.onmicrosoft.com");
     (globalThis as any).__getHtmlReporterOutputFolderMock.mockReturnValue("custom-report");
     (globalThis as any).__uploadHtmlReportAfterTestsMock.mockResolvedValue({ success: true });
     (globalThis as any).__getPortalTestRunUrlMock.mockReturnValue("https://portal/link/test");
@@ -142,11 +154,19 @@ describe("PlaywrightReporter", () => {
     await reporter.onEnd();
 
     expect((globalThis as any).__getWorkspaceMetadataMock).toHaveBeenCalled();
+    expect((globalThis as any).__getTenantsMock).toHaveBeenCalled();
+    expect((globalThis as any).__resolveTenantDomainMock).toHaveBeenCalledWith(
+      "tenant-1",
+      tenantList,
+    );
     expect((globalThis as any).__uploadHtmlReportAfterTestsMock).toHaveBeenCalledWith(
       "custom-report",
       workspaceMetadata,
     );
-    expect((globalThis as any).__getPortalTestRunUrlMock).toHaveBeenCalledWith(workspaceMetadata);
+    expect((globalThis as any).__getPortalTestRunUrlMock).toHaveBeenCalledWith(
+      workspaceMetadata,
+      "contoso.onmicrosoft.com",
+    );
     expect(consoleLogSpy).toHaveBeenCalledWith(
       ServiceErrorMessageConstants.COLLECTING_ARTIFACTS.message,
     );

sdk/loadtesting/playwright/test/utils/PlaywrightServiceClient.spec.ts

@@ -3,7 +3,11 @@
 
 import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 import { PlaywrightServiceClient } from "../../src/utils/PlaywrightServiceClient.js";
-import { Constants, InternalEnvironmentVariables } from "../../src/common/constants.js";
+import {
+  Constants,
+  InternalEnvironmentVariables,
+  ArmConstants,
+} from "../../src/common/constants.js";
 import { ServiceErrorMessageConstants } from "../../src/common/messages.js";
 import { TestRunCreatePayload } from "../../src/common/types.js";
 
@@ -213,4 +217,75 @@ describe("PlaywrightServiceClient", () => {
       expect(mockState.extractErrorMessage).toHaveBeenCalledWith(mockResponse.bodyAsText);
     });
   });
+
+  describe("getTenants", () => {
+    it("should call the ARM tenants API and return parsed tenant list", async () => {
+      const tenantList = [
+        { tenantId: "tenant-1", defaultDomain: "contoso.onmicrosoft.com" },
+        { tenantId: "tenant-2", defaultDomain: "fabrikam.onmicrosoft.com" },
+      ];
+      const mockResponse = {
+        status: 200,
+        bodyAsText: JSON.stringify({ value: tenantList }),
+      };
+      mockState.callAPI.mockResolvedValue(mockResponse);
+
+      const result = await apiCall.getTenants();
+
+      expect(mockState.getAccessToken).toHaveBeenCalledTimes(1);
+      expect(mockState.callAPI).toHaveBeenCalledWith(
+        "GET",
+        `${ArmConstants.TenantsApiUrl}?api-version=${ArmConstants.TenantsApiVersion}`,
+        null,
+        "mock-token",
+        "",
+        "mock-uuid",
+      );
+      expect(result).toEqual(tenantList);
+    });
+
+    it("should throw when access token is missing", async () => {
+      mockState.getAccessToken.mockReturnValue(undefined);
+
+      await expect(apiCall.getTenants()).rejects.toThrow(
+        "PLAYWRIGHT_SERVICE_ACCESS_TOKEN environment variable is not set.",
+      );
+
+      expect(mockState.callAPI).not.toHaveBeenCalled();
+    });
+
+    it("should throw with error message when API returns non-200 status", async () => {
+      const mockResponse = {
+        status: 403,
+        bodyAsText: JSON.stringify({ error: { message: "Forbidden" } }),
+      };
+      mockState.callAPI.mockResolvedValue(mockResponse);
+      mockState.extractErrorMessage.mockReturnValue("Forbidden");
+
+      await expect(apiCall.getTenants()).rejects.toThrow("Forbidden");
+      expect(mockState.extractErrorMessage).toHaveBeenCalledWith(mockResponse.bodyAsText);
+    });
+
+    it("should return empty array when response body has no value", async () => {
+      const mockResponse = {
+        status: 200,
+        bodyAsText: JSON.stringify({}),
+      };
+      mockState.callAPI.mockResolvedValue(mockResponse);
+
+      const result = await apiCall.getTenants();
+      expect(result).toEqual([]);
+    });
+
+    it("should return empty array when response body is empty", async () => {
+      const mockResponse = {
+        status: 200,
+        bodyAsText: "",
+      };
+      mockState.callAPI.mockResolvedValue(mockResponse);
+
+      const result = await apiCall.getTenants();
+      expect(result).toEqual([]);
+    });
+  });
 });