Fix error when test names contain --

We currently store the test's FullyQualifiedName in the id and separate it from other id components with --

This led to incorrectly splitting the test name if it contained a --

It may be a good idea to dynamically add a field to TestItem for FullyQualifiedName similar to what was donen for TestFramework

Author: farlee2121

src/Components/TestExplorer.fs

@@ -766,8 +766,10 @@ module TestItem =
     let constructProjectRootId (projectPath: ProjectPath) : TestId = constructId projectPath ""
 
     let private componentizeId (testId: TestId) : (ProjectPath * FullTestName) =
+        // IMPORTANT: the fullname should be last and we should limit the number of substrings
+        //            to prevent incorrently splitting tests names with -- in them
         let split =
-            testId.Split(separator = [| idSeparator |], options = StringSplitOptions.None)
+            testId.Split(separator = [| idSeparator |], count = 2, options = StringSplitOptions.None)
 
         (split.[0], split.[1])