fix(playwrighttesting): logger name and flaky tests (Azure#46746)

* fix(): logger name and flaky tests

* fix(): run name fetching logic name from git

---------

Co-authored-by: Siddharth Singha Roy <[email protected]>

Author: Sid200026

sdk/playwrighttesting/Azure.Developer.MicrosoftPlaywrightTesting.NUnit/samples/Sample1_CustomisingServiceParameters.md

@@ -15,17 +15,15 @@ This guide explains the different options available to you in the Azure.Develope
         <!-- The below parameters are optional -->
         <Parameter name="Os" value="linux" />
         <Parameter name="RunId" value="sample-run-id" />
-        <Parameter name="ExposeNetwork" value="<loopback>" />
         <Parameter name="ServiceAuthType" value="EntraId" />
         <Parameter name="UseCloudHostedBrowsers" value="true" />
         <Parameter name="AzureTokenCredentialType" value="DefaultAzureCredential" />
-        <Parameter name="ManagedIdentityClientId" value="77bfc267-86cb-4eeb-9e4a-747a217a318c" />
         <Parameter name="EnableGitHubSummary" value="false" />
     </TestRunParameters>
     <!-- Enable Reporting feature -->
     <LoggerRunSettings>
         <Loggers>
-            <Logger friendlyName="ms-playwright-service" enabled="true" />
+            <Logger friendlyName="microsoft-playwright-testing" enabled="true" />
         </Loggers>
     </LoggerRunSettings>
 </RunSettings>

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

@@ -44,7 +44,7 @@ internal static string GetRunName(CIInfo ciInfo)
         {
             string GIT_VERSION_COMMAND = "git --version";
             string GIT_REV_PARSE = "git rev-parse --is-inside-work-tree";
-            string GIT_COMMIT_MESSAGE_COMMAND = "git log -1 --pretty=%B";
+            string GIT_COMMIT_MESSAGE_COMMAND = "git log -1 --pretty=format:\"%s\"";
 
             if (ciInfo.Provider == CIConstants.s_gITHUB_ACTIONS &&
             Environment.GetEnvironmentVariable("GITHUB_EVENT_NAME") == "pull_request")

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

@@ -11,7 +11,6 @@
 namespace Azure.Developer.MicrosoftPlaywrightTesting.TestLogger.Tests.Utility
 {
     [TestFixture]
-    [Parallelizable(ParallelScope.Self)]
     public class CiInfoProviderTests
     {
         private Dictionary<string, string> _originalEnvironmentVariables = new();

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

@@ -11,7 +11,6 @@
 namespace Azure.Developer.MicrosoftPlaywrightTesting.TestLogger.Tests.Utility;
 
 [TestFixture]
-[Parallelizable(ParallelScope.Self)]
 public class ReporterUtilsTests
 {
     private static string GetToken(Dictionary<string, object> claims, DateTime? expires = null)