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

* fix(): logger name and flaky tests * fix(): run name fetching logic name from git --------- Co-authored-by: Siddharth Singha Roy <ssingharoy@microsoft.com>
Azure · Oct 22, 2024 · 8981012 · 8981012
1 parent fe44601
commit 8981012
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 6 deletions.
diff --git a/...icrosoftPlaywrightTesting.NUnit/samples/Sample1_CustomisingServiceParameters.md b/...icrosoftPlaywrightTesting.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>

diff --git a/...esting/Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/src/Utility/ReporterUtils.cs b/...esting/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")

diff --git a/...zure.Developer.MicrosoftPlaywrightTesting.TestLogger/tests/Utility/CiInfoProviderTests.cs b/...zure.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();

diff --git a/...Azure.Developer.MicrosoftPlaywrightTesting.TestLogger/tests/Utility/ReporterUtilsTests.cs b/...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)