Refactor test function name for consistency and clarity

Remove commented-out code in msbuildPropBool for cleaner implementation

Author: TheAngryByrd

src/Ionide.ProjInfo/Library.fs

@@ -1,4 +1,4 @@
-namespace Ionide.ProjInfo
+namespace Ionide.ProjInfo
 
 open System
 open System.Collections.Generic
@@ -657,7 +657,6 @@ module ProjectLoader =
         else
             [|
                 yield! designTimeBuildTargetsCore
-                "DispatchToInnerBuilds"
             |]
 
     let setLegacyMsbuildProperties isOldStyleProjFile =
@@ -906,9 +905,6 @@ module ProjectLoader =
                 msbuildPropString "ProjectAssetsFile"
                 |> Option.defaultValue ""
             RestoreSuccess =
-                // match msbuildPropString "TargetFrameworkVersion" with
-                // | Some _ -> true
-                // | None ->
                 msbuildPropBool "RestoreSuccess"
                 |> Option.defaultValue false
 

test/Ionide.ProjInfo.Tests/ProjectLoader2Tests.fs

@@ -253,7 +253,7 @@ module ProjectLoader2Tests =
 
 
     let buildManagerSessionTests toolsPath =
-        ftestList "buildManagerSessionTests" [
+        testList "buildManagerSessionTests" [
             yield! applyTests "loader2-no-solution-with-2-projects" ``loader2-no-solution-with-2-projects``
 
             yield! applyTests "sample2-NetSdk-library2" ``sample2-NetSdk-library2``