fix(playwrighttesting): ADO job name from environment variables (Azure#32620)

### Packages impacted by this PR

@azure/microsoft-playwright-testing

### 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)

Author: Sid200026

sdk/playwrighttesting/microsoft-playwright-testing/src/reporter/mptReporter.ts

@@ -266,7 +266,7 @@ class MPTReporter implements Reporter {
       const testResultObject: MPTTestResult = this.reporterUtils.getTestResultObject(
         test,
         result,
-        this.ciInfo.jobId!,
+        this.ciInfo.jobName!,
       );
       this.testResultBatch.add(testResultObject);
       // Store test attachments in array

sdk/playwrighttesting/microsoft-playwright-testing/src/utils/cIInfoProvider.ts

@@ -17,7 +17,7 @@ export type CIInfo = {
   revisionUrl: string | null;
   runId: string | null;
   runAttempt: number | null;
-  jobId: string | null;
+  jobName: string | null;
 };
 
 export class CIInfoProvider {
@@ -52,7 +52,7 @@ export class CIInfoProvider {
         runAttempt: process.env["GITHUB_RUN_ATTEMPT"]
           ? parseInt(process.env["GITHUB_RUN_ATTEMPT"], 10)
           : null,
-        jobId: process.env["GITHUB_JOB"] || null,
+        jobName: process.env["GITHUB_JOB"] || null,
       };
     } else if (ciProvider === CI_PROVIDERS.ADO) {
       // Logic to get Azure DevOps CIInfo
@@ -69,9 +69,8 @@ export class CIInfoProvider {
         runAttempt: process.env["RELEASE_ATTEMPTNUMBER"]
           ? parseInt(process.env["RELEASE_ATTEMPTNUMBER"], 10)
           : parseInt(process.env["SYSTEM_JOBATTEMPT"] ?? "", 10),
-        jobId: process.env["RELEASE_DEPLOYMENTID"]
-          ? process.env["RELEASE_DEPLOYMENTID"]
-          : process.env["SYSTEM_JOBID"] || null,
+        jobName:
+          process.env["SYSTEM_JOBDISPLAYNAME"] ?? process.env["RELEASE_DEPLOYMENTID"] ?? null,
       };
     } else {
       // Handle unsupported CI provider
@@ -84,7 +83,7 @@ export class CIInfoProvider {
         revisionUrl: process.env["REVISION_URL"] ?? null,
         runId: process.env["RUN_ID"] ?? null,
         runAttempt: process.env["RUN_ATTEMPT"] ? parseInt(process.env["RUN_ATTEMPT"], 10) : null,
-        jobId: process.env["JOB_ID"] ?? null,
+        jobName: process.env["JOB_ID"] ?? null,
       };
     }
   }

sdk/playwrighttesting/microsoft-playwright-testing/test/utils/clInfoProvider.spec.ts

@@ -38,7 +38,7 @@ describe("CIInfoProvider", () => {
     expect(ciInfo.revisionUrl).to.equal("https://github.com/repo/commit/sampleSha");
     expect(ciInfo.runId).to.equal("runId123");
     expect(ciInfo.runAttempt).to.equal(1);
-    expect(ciInfo.jobId).to.equal("testJob");
+    expect(ciInfo.jobName).to.equal("testJob");
   });
 
   it("should return GitHub CIInfo with null fields when Github environment variables are not set", () => {
@@ -54,7 +54,7 @@ describe("CIInfoProvider", () => {
     expect(ciInfo.revisionUrl).to.be.null;
     expect(ciInfo.runId).to.be.null;
     expect(ciInfo.runAttempt).to.be.null;
-    expect(ciInfo.jobId).to.be.null;
+    expect(ciInfo.jobName).to.be.null;
   });
 
   it("should return Azure DevOps CIInfo when Azure DevOps environment variables are set", () => {
@@ -72,6 +72,7 @@ describe("CIInfoProvider", () => {
     process.env["SYSTEM_DEFINITIONID"] = "def123";
     process.env["SYSTEM_JOBATTEMPT"] = "1";
     process.env["RELEASE_DEFINITIONID"] = "Rdef123";
+    process.env["SYSTEM_JOBDISPLAYNAME"] = "jobName123";
     process.env["RELEASE_DEPLOYMENTID"] = "Rdep123";
 
     const ciInfo = CIInfoProvider.getCIInfo();
@@ -85,7 +86,37 @@ describe("CIInfoProvider", () => {
       "https://dev.azure.com/project123/_git/repo123/commit/commitSha123",
     );
     expect(ciInfo.runId).to.equal("Rdef123-Rdep123");
-    expect(ciInfo.jobId).to.equal("Rdep123");
+    expect(ciInfo.jobName).to.equal("jobName123");
+  });
+
+  it("should return Azure DevOps CIInfo when Azure DevOps environment variables are set (SYSTEM_JOBDISPLAYNAME not set)", () => {
+    process.env["AZURE_HTTP_USER_AGENT"] = "someAgent";
+    process.env["TF_BUILD"] = "true";
+    process.env["BUILD_REPOSITORY_ID"] = "repo123";
+    process.env["BUILD_SOURCEBRANCH"] = "main";
+    process.env["BUILD_REQUESTEDFOR"] = "testAuthor";
+    process.env["BUILD_SOURCEVERSION"] = "commitSha123";
+    process.env["SYSTEM_TEAMFOUNDATIONCOLLECTIONURI"] = "https://dev.azure.com/";
+    process.env["SYSTEM_TEAMPROJECT"] = "project123";
+    process.env["BUILD_REPOSITORY_NAME"] = "repo123";
+    process.env["SYSTEM_JOBID"] = "job123";
+    process.env["SYSTEM_DEFINITIONID"] = "def123";
+    process.env["SYSTEM_JOBATTEMPT"] = "1";
+    process.env["RELEASE_DEFINITIONID"] = "Rdef123";
+    process.env["RELEASE_DEPLOYMENTID"] = "Rdep123";
+
+    const ciInfo = CIInfoProvider.getCIInfo();
+
+    expect(ciInfo.provider).to.equal(CI_PROVIDERS.ADO);
+    expect(ciInfo.repo).to.equal("repo123");
+    expect(ciInfo.branch).to.equal("main");
+    expect(ciInfo.author).to.equal("testAuthor");
+    expect(ciInfo.commitId).to.equal("commitSha123");
+    expect(ciInfo.revisionUrl).to.equal(
+      "https://dev.azure.com/project123/_git/repo123/commit/commitSha123",
+    );
+    expect(ciInfo.runId).to.equal("Rdef123-Rdep123");
+    expect(ciInfo.jobName).to.equal("Rdep123");
   });
 
   it("should return Azure DevOps CIInfo with null fields when Azure DevOps environment variables are not set", () => {
@@ -101,7 +132,7 @@ describe("CIInfoProvider", () => {
     expect(ciInfo.commitId).to.be.null;
     expect(ciInfo.revisionUrl).to.be.null;
     expect(ciInfo.runId).to.be.null;
-    expect(ciInfo.jobId).to.be.null;
+    expect(ciInfo.jobName).to.be.null;
   });
 
   it("should return default CIInfo when no supported CI environment is detected", () => {
@@ -124,7 +155,7 @@ describe("CIInfoProvider", () => {
     expect(ciInfo.revisionUrl).to.equal("https://default.com/repo/commit/defaultCommit");
     expect(ciInfo.runId).to.equal("defaultRunId");
     expect(ciInfo.runAttempt).to.equal(2);
-    expect(ciInfo.jobId).to.equal("defaultJobId");
+    expect(ciInfo.jobName).to.equal("defaultJobId");
   });
 
   it("should return default CIInfo with null fields when no supported CI environment is detected", () => {
@@ -138,6 +169,6 @@ describe("CIInfoProvider", () => {
     expect(ciInfo.revisionUrl).to.be.null;
     expect(ciInfo.runId).to.be.null;
     expect(ciInfo.runAttempt).to.be.null;
-    expect(ciInfo.jobId).to.be.null;
+    expect(ciInfo.jobName).to.be.null;
   });
 });