Fixed a bug with method arguments in test names

Author: daveaglick

ReleaseNotes.md

@@ -1,6 +1,7 @@
 # 0.4.1
 
 - [Fix] Fixed a bug with source parsing and file paths
+- [Fix] Fixed a bug with method arguments in test names
 
 # 0.4.0
 

src/AzurePipelines.TestLogger/LoggerQueue.cs

@@ -156,7 +156,13 @@ internal IEnumerable<IGrouping<string, ITestResult>> GroupTestResultsByParent(IT
                 }
 
                 // At this point, name should always have at least one '.' to represent the Class.Method
-                return name.Substring(0, name.LastIndexOfAny(new[] { '.' }));
+                // We need to start at the opening method paren if there is one
+                int startIndex = name.IndexOf('(');
+                if(startIndex < 0)
+                {
+                    startIndex = name.Length - 1;
+                }
+                return name.Substring(0, name.LastIndexOfAny(new[] { '.' }, startIndex));
             });
 
         // Internal for testing

tests/AzurePipelines.TestLogger.Tests/LoggerQueueTests.cs

@@ -136,7 +136,7 @@ public void GroupTestResults()
                 },
                 new TestTestResult
                 {
-                    FullyQualifiedName = "Fizz.Buzz.FutzFixture.BooMethod(\"x\")"
+                    FullyQualifiedName = "Fizz.Buzz.FutzFixture.BooMethod(\"x.y\")"
                 },
                 new TestTestResult
                 {
@@ -190,7 +190,7 @@ public void CreateParents()
                 },
                 new TestTestResult
                 {
-                    FullyQualifiedName = "Fizz.Buzz.FutzFixture.NestedFixture.BooMethod(\"x\")"
+                    FullyQualifiedName = "Fizz.Buzz.FutzFixture.NestedFixture.BooMethod(\"x.y\")"
                 }
             };
             IEnumerable<IGrouping<string, ITestResult>> testResultsByParent = loggerQueue.GroupTestResultsByParent(testResults);