Discover tests without building on initial workspace load

If they've never built their solution, then we won't be able to load tests until they restore anyway.

If they have built their solution, we probably don't need to build it again. It's much faster to discover tests without the build and, unlike clicking the refresh button, there is a much smaller chance they're looking for missing test or otherwise solving a weird explorer state where we'd want fresh build artifacts.

Author: farlee2121

src/Components/TestExplorer.fs

@@ -237,7 +237,9 @@ module TestResultOutcome =
         | TestOutcomeDTO.Skipped -> TestResultOutcome.Skipped
         | TestOutcomeDTO.None -> TestResultOutcome.NotExecuted
         | TestOutcomeDTO.NotFound -> TestResultOutcome.NotExecuted
-        | _ -> failwith $"Unknown value for TestOutcomeDTO: {outcomeDto}. The language server may have changed its possible values."
+        | _ ->
+            failwith
+                $"Unknown value for TestOutcomeDTO: {outcomeDto}. The language server may have changed its possible values."
 
 
 type TestFrameworkId = string
@@ -1729,11 +1731,7 @@ module Interactions =
               HasIncludeFilter = hasIncludeFilter
               Tests = replaceProjectRootIfPresent tests })
 
-    let private discoverTests_WithLanguageServer
-        testItemFactory
-        (rootTestCollection: TestItemCollection)
-        tryGetLocation
-        =
+    let discoverTests_WithLanguageServer testItemFactory (rootTestCollection: TestItemCollection) tryGetLocation =
         withProgress NoCancel
         <| fun p progressCancelToken ->
             promise {
@@ -2319,16 +2317,19 @@ let activate (context: ExtensionContext) =
                 let initialTests = trxTests workspaceProjects
                 initialTests |> Array.iter testController.items.add
 
-            let cancellationTokenSource = vscode.CancellationTokenSource.Create()
-            // NOTE: Trx results can be partial if the last test run was filtered, so also queue a refresh to make sure we discover all tests
-            Interactions.refreshTestList
-                testItemFactory
-                testController.items
-                tryGetLocation
-                makeTrxPath
-                useLegacyDotnetCliIntegration
-                cancellationTokenSource.token
-            |> Promise.start
+                let cancellationTokenSource = vscode.CancellationTokenSource.Create()
+                // NOTE: Trx results can be partial if the last test run was filtered, so also queue a refresh to make sure we discover all tests
+                Interactions.refreshTestList
+                    testItemFactory
+                    testController.items
+                    tryGetLocation
+                    makeTrxPath
+                    useLegacyDotnetCliIntegration
+                    cancellationTokenSource.token
+                |> Promise.start
+            else
+                Interactions.discoverTests_WithLanguageServer testItemFactory testController.items tryGetLocation
+                |> Promise.start
 
         None)
     |> unbox