From 02bb671d1551240b40b990adaaf56662be899c51 Mon Sep 17 00:00:00 2001
From: Hao Kung <HaoK@users.noreply.github.com>
Date: Sat, 18 Apr 2020 01:58:02 -0700
Subject: [PATCH] [Helix] Don't fail fast on ef tool install issues (may be
 already installed) (#20958)

---
 eng/helix/content/RunTests/TestRunner.cs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/eng/helix/content/RunTests/TestRunner.cs b/eng/helix/content/RunTests/TestRunner.cs
index 65dd451036..43e3cce490 100644
--- a/eng/helix/content/RunTests/TestRunner.cs
+++ b/eng/helix/content/RunTests/TestRunner.cs
@@ -141,7 +141,8 @@ namespace RunTests
                         $"tool install dotnet-ef --global --version {Options.EfVersion}",
                         environmentVariables: EnvironmentVariables,
                         outputDataReceived: Console.WriteLine,
-                        errorDataReceived: Console.Error.WriteLine);
+                        errorDataReceived: Console.Error.WriteLine,
+                        throwOnError: false); // EF tool is sometimes already installed so we can ignore this failure
 
                     // ';' is the path separator on Windows, and ':' on Unix
                     Options.Path += RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? ";" : ":";