Invert the switch because it will be still backwards in the final rel…

…ease
nohwnd · Sep 24, 2020 · c5becb6 · c5becb6
1 parent cbf91f7
commit c5becb6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Microsoft.TestPlatform.Build/Tasks/VSTestTask.cs b/src/Microsoft.TestPlatform.Build/Tasks/VSTestTask.cs
@@ -182,7 +182,8 @@ public override bool Execute()
             // Avoid logging "Task returned false but did not log an error." on test failure, because we don't
             // write MSBuild error. https://github.com/dotnet/msbuild/blob/51a1071f8871e0c93afbaf1b2ac2c9e59c7b6491/src/Framework/IBuildEngine7.cs#L12
             var allowfailureWithoutError = BuildEngine.GetType().GetProperty("AllowFailureWithoutError");
-            allowfailureWithoutError?.SetValue(BuildEngine, true);
+            // setting this to false because the switch is implemented backwards and it won't be fixed till next release
+            allowfailureWithoutError?.SetValue(BuildEngine, false);
 
             vsTestForwardingApp = new VSTestForwardingApp(this.VSTestConsolePath, this.CreateArgument());
             if (!string.IsNullOrEmpty(this.VSTestFramework))