fix(playwrighttesting): handled limitation of character in parameters. (Azure#47608)

* fix(playwrighttesting): handled limitation of character in parameters.

* update

* update

* message update

---------

Co-authored-by: guptakashish <[email protected]>

Author: kashish2508

sdk/playwrighttesting/Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/src/Constants.cs

@@ -215,11 +215,14 @@ internal class Constants
     internal static readonly string s_invalid_os_error = "Invalid operating system, supported values are 'linux' and 'windows'.";
     internal static readonly string s_workspace_mismatch_error = "The provided access token does not match the specified workspace URL. Please verify that both values are correct.";
     internal static readonly string s_invalid_service_endpoint_error_message = "The service endpoint provided is invalid. Please verify the endpoint URL and try again.";
+    internal static readonly string s_playwright_service_runId_length_exceeded_error_message = "Error: The Run Id you provided exceeds 200 characters. Please provide a shorter Run ID.";
 
     internal static readonly string s_playwright_service_disable_scalable_execution_environment_variable = "_MPT_DISABLE_SCALABLE_EXECUTION";
     internal static readonly string s_playwright_service_reporting_url_environment_variable = "_MPT_REPORTING_URL";
     internal static readonly string s_playwright_service_workspace_id_environment_variable = "_MPT_WORKSPACE_ID";
     internal static readonly string s_playwright_service_auth_type_environment_variable = "_MPT_AUTH_TYPE";
+
+    internal static readonly string s_playwright_service_runName_truncated_warning = "WARNING: Run name exceeds the maximum limit of 200 characters and will be truncated.";
 }
 
 internal class OSConstants

sdk/playwrighttesting/Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/src/PlaywrightReporter.cs

@@ -150,16 +150,26 @@ internal void InitializePlaywrightReporter(string xmlSettings)
             _environment.Exit(1);
             return;
         }
-
+        if (cloudRunId?.Length > 200)
+        {
+            _consoleWriter.WriteError(Constants.s_playwright_service_runId_length_exceeded_error_message);
+            _environment.Exit(1);
+            return;
+        }
         var baseUri = new Uri(baseUrl);
         var reporterUtils = new ReporterUtils();
         TokenDetails tokenDetails = reporterUtils.ParseWorkspaceIdFromAccessToken(jsonWebTokenHandler: _jsonWebTokenHandler, accessToken: accessToken);
         var workspaceId = tokenDetails.aid;
-
+        var runNameString = runName?.ToString();
+        if (runNameString?.Length > 200)
+        {
+            runNameString = runNameString.Substring(0, 200);
+            _consoleWriter.WriteLine(Constants.s_playwright_service_runName_truncated_warning);
+        }
         var cloudRunMetadata = new CloudRunMetadata
         {
             RunId = cloudRunId,
-            RunName = runName?.ToString(),
+            RunName = runNameString,
             WorkspaceId = workspaceId,
             BaseUri = baseUri,
             EnableResultPublish = _enableResultPublish,

sdk/playwrighttesting/Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/src/Processor/DataProcessor.cs

@@ -53,10 +53,10 @@ public TestRunDto GetTestRun()
                 CloudRunEnabled = false,
                 CiConfig = new CIConfig
                 {
-                    Branch = _cIInfo.Branch,
-                    Author = _cIInfo.Author,
-                    CommitId = _cIInfo.CommitId,
-                    RevisionUrl = _cIInfo.RevisionUrl,
+                    Branch = ReporterUtils.TruncateData(_cIInfo.Branch, 500),
+                    Author = ReporterUtils.TruncateData(_cIInfo.Author,500),
+                    CommitId = ReporterUtils.TruncateData(_cIInfo.CommitId,500),
+                    RevisionUrl = ReporterUtils.TruncateData(_cIInfo.RevisionUrl,1000),
                     CiProviderName = _cIInfo.Provider ?? CIConstants.s_dEFAULT
                 },
                 TestRunConfig = new ClientConfig // TODO fetch some of these dynamically
@@ -104,16 +104,17 @@ public TestResults GetTestCaseResultData(TestResult? testResultSource)
             };
             testCaseResultData.TestCombinationId = testCaseResultData.TestExecutionId; // TODO check
             testCaseResultData.TestId = testResultSource.TestCase.Id.ToString();
-            testCaseResultData.TestTitle = testResultSource.TestCase.DisplayName;
+            testCaseResultData.TestTitle = ReporterUtils.TruncateData(testResultSource.TestCase.DisplayName, 500)!;
+            testCaseResultData.TestTitle = ReporterUtils.TruncateData(testResultSource.TestCase.DisplayName, 500)!;
             var className = FetchTestClassName(testResultSource.TestCase.FullyQualifiedName);
-            testCaseResultData.SuiteTitle = className;
+            testCaseResultData.SuiteTitle = ReporterUtils.TruncateData(className,500)!;
             testCaseResultData.SuiteId = ReporterUtils.CalculateSha1Hash(className);
-            testCaseResultData.FileName = FetchFileName(testResultSource.TestCase.Source);
+            testCaseResultData.FileName = ReporterUtils.TruncateData(FetchFileName(testResultSource.TestCase.Source),300)!;
             testCaseResultData.LineNumber = testResultSource.TestCase.LineNumber;
             testCaseResultData.Retry = 0; // TODO Retry and PreviousRetries
             testCaseResultData.WebTestConfig = new WebTestConfig
             {
-                JobName = _cIInfo.JobId ?? "",
+                JobName = _cIInfo.JobId != null ? ReporterUtils.TruncateData(_cIInfo.JobId, 500) ?? "" : "",
                 //ProjectName = "playwright-dotnet", // TODO no project concept NA??
                 //BrowserName = "chromium", // TODO check if possible to get from test
                 Os = ReporterUtils.GetCurrentOS(),

sdk/playwrighttesting/Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/src/Utility/ReporterUtils.cs

@@ -40,6 +40,12 @@ internal static string CalculateSha1Hash(string input)
                 return BitConverter.ToString(hash).Replace("-", string.Empty).ToLower();
             }
         }
+        internal static string? TruncateData(string? value, int maxLength)
+        {
+            if (string.IsNullOrEmpty(value))
+                return value;
+            return value?.Length <= maxLength ? value : value?.Substring(0, maxLength);
+        }
 
         internal static string GetRunName(CIInfo ciInfo)
         {

sdk/playwrighttesting/Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/tests/Utility/ReporterUtilsTests.cs

@@ -141,4 +141,25 @@ public void GetRunName_GitHubActionsPullRequest_ReturnsExpectedValue()
         var expected = "PR# 543 on Repo: owner/repo (owner/repo/pull/543)";
         Assert.AreEqual(expected, result);
     }
+    [Test]
+    public void TruncateData_ValueExceedsMaxLength_ReturnsTruncatedString()
+    {
+        string value = "This is a very long string that exceeds the maximum length.";
+        int maxLength = 20;
+
+        var result = ReporterUtils.TruncateData(value, maxLength);
+
+        Assert.AreEqual("This is a very long ", result);
+    }
+
+    [Test]
+    public void TruncateData_ValueWithinMaxLength_ReturnsOriginalString()
+    {
+        string value = "Short string";
+        int maxLength = 20;
+
+        var result = ReporterUtils.TruncateData(value, maxLength);
+
+        Assert.AreEqual(value, result);
+    }
 }