enable some of the tests

src/Build.OM.UnitTests/Definition/ProjectCollection_Tests.cs

@@ -181,11 +181,7 @@ public void GlobalPropertyInheritLoadFromFile()
         /// <summary>
         /// Loading a project from a file inherits the project collection's global properties
         /// </summary>
-#if FEATURE_INSTALLED_MSBUILD
         [Fact]
-#else
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/276")]
-#endif
         public void GlobalPropertyInheritLoadFromFile2()
         {
             string path = null;
@@ -196,7 +192,7 @@ public void GlobalPropertyInheritLoadFromFile2()
 
                 using ProjectCollection collection = new ProjectCollection();
                 collection.SetGlobalProperty("p", "v");
-                Project project = collection.LoadProject(path, "4.0");
+                Project project = collection.LoadProject(path, "Current");
 
                 Assert.Equal("v", project.GlobalProperties["p"]);
             }
@@ -212,11 +208,7 @@ public void GlobalPropertyInheritLoadFromFile2()
         /// <summary>
         /// Loading a project from a file inherits the project collection's global properties
         /// </summary>
-#if FEATURE_INSTALLED_MSBUILD
         [Fact]
-#else
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/276")]
-#endif
         public void GlobalPropertyInheritLoadFromFile3()
         {
             string path = null;
@@ -227,7 +219,7 @@ public void GlobalPropertyInheritLoadFromFile3()
 
                 using ProjectCollection collection = new ProjectCollection();
                 collection.SetGlobalProperty("p", "v");
-                Project project = collection.LoadProject(path, null, "4.0");
+                Project project = collection.LoadProject(path, null, "Current");
 
                 Assert.Equal("v", project.GlobalProperties["p"]);
             }
@@ -1095,11 +1087,7 @@ public void DefaultToolsVersion()
         /// <summary>
         /// Current default from registry is 4.0 if 2.0 is not installed
         /// </summary>
-#if FEATURE_INSTALLED_MSBUILD
         [Fact]
-#else
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/276")]
-#endif
         public void DefaultToolsVersion2()
         {
             if (ToolLocationHelper.GetPathToDotNetFramework(TargetDotNetFrameworkVersion.Version20) != null)

src/Build.OM.UnitTests/Definition/Project_Tests.cs

@@ -980,9 +980,6 @@ public void ChangeGlobalPropertiesPreexisting()
         /// </summary>
 #if FEATURE_INSTALLED_MSBUILD
         [Fact]
-#else
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/276")]
-#endif
         public void ChangeGlobalPropertiesInitiallyFromProjectCollection()
         {
             Dictionary<string, string> initial = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
@@ -1002,6 +999,7 @@ public void ChangeGlobalPropertiesInitiallyFromProjectCollection()
             collection.GlobalProperties["p0"].ShouldBe("v0");
             collection.GlobalProperties["p1"].ShouldBe("v1");
         }
+#endif
 
         /// <summary>
         /// Changing global property to the same value should not dirty the project.

src/Build.UnitTests/BackEnd/TargetEntry_Tests.cs

@@ -826,7 +826,7 @@ public void TestTargetOutputsOnFinishedEventNoVariableSet()
         /// <summary>
         /// Make sure that if an after target fails that the build result is reported as failed.
         /// </summary>
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/515")]
+        [Fact]
         public void AfterTargetsShouldReportFailedBuild()
         {
             // Since we're creating our own BuildManager, we need to make sure that the default

src/Build.UnitTests/Construction/SolutionProjectGenerator_Tests.cs

@@ -2137,7 +2137,7 @@ public void TestSkipInvalidConfigurationsCase(bool useNewParser)
         /// <summary>
         /// When we have a bad framework moniker we expect the build to fail.
         /// </summary>
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/515")]
+        [Fact]
         public void BadFrameworkMonkierExpectBuildToFail()
         {
             string tmpFileName = FileUtilities.GetTemporaryFileName();
@@ -2225,7 +2225,7 @@ public void BadFrameworkMonkierExpectBuildToFail()
         /// When we have a bad framework moniker we expect the build to fail. In this case we are passing a poorly formatted framework moniker.
         /// This will test the exception path where the framework name is invalid rather than just not .netFramework
         /// </summary>
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/515")]
+        [Fact]
         public void BadFrameworkMonkierExpectBuildToFail2()
         {
             string tmpFileName = FileUtilities.GetTemporaryFileName();

src/Tasks.UnitTests/MSBuild_Tests.cs

@@ -383,11 +383,7 @@ public void LogErrorWhenBuildingVCProj()
         /// However, it's a situation where the project author doesn't have control over the
         /// property value and so he can't escape it himself.
         /// </summary>
-#if RUNTIME_TYPE_NETCORE
-        [Fact(Skip = "https://github.com/dotnet/msbuild/issues/259")]
-#else
         [Fact]
-#endif
         public void PropertyOverridesContainSemicolon()
         {
             ObjectModelHelpers.DeleteTempProjectDirectory();

src/Utilities.UnitTests/ToolLocationHelper_Tests.cs

@@ -149,11 +149,7 @@ public void GetSDKRootLocation()
             unversionedSDKValue.ShouldBe(expectedValue);
         }
 
-#if RUNTIME_TYPE_NETCORE
-        [WindowsOnlyFact(Skip = "https://github.com/dotnet/msbuild/issues/1250")]
-#else
-        [WindowsOnlyFact(Skip = "https://github.com/dotnet/msbuild/issues/2569")]
-#endif
+        [WindowsOnlyFact]
         public void GetUnversionedSDKUnionMetadataLocation()
         {
             string sdkRootPath = ToolLocationHelper.GetPlatformSDKLocation("Windows", "10.0");