Update BrowserManager.cs

HaoK · HaoK · commit bcacfdd32b30 · 2021-03-10T20:18:49.000-08:00
diff --git a/src/Shared/BrowserTesting/src/BrowserManager.cs b/src/Shared/BrowserTesting/src/BrowserManager.cs
@@ -46,7 +46,10 @@ private async Task InitializeAsync()
 
             async Task InitializeCore()
             {
-                Playwright = await PlaywrightSharp.Playwright.CreateAsync(_loggerFactory/*, debug: "pw:api"*/);
+                // Work around weird driverExecutable lookup logic
+                var helixDirectory = Environment.GetEnvironmentVariable("HELIX_WORKITEM_ROOT")
+                var driverPath = string.isNullOrEmpty(helixDirectory) ? null : Path.Combine(helixDirectory, "runtests.sh");
+                Playwright = await PlaywrightSharp.Playwright.CreateAsync(_loggerFactory, driverExecutablePath: driverPath /*, debug: "pw:api"*/);
                 foreach (var (browserName, options) in _browserManagerConfiguration.BrowserOptions)
                 {
                     if (!_launchBrowsers.ContainsKey(browserName))

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,10 @@ private async Task InitializeAsync()`
`46`	`46`
`47`	`47`	`async Task InitializeCore()`
`48`	`48`	`{`
`49`		`- Playwright = await PlaywrightSharp.Playwright.CreateAsync(_loggerFactory/, debug: "pw:api"/);`
	`49`	`+ // Work around weird driverExecutable lookup logic`
	`50`	`+ var helixDirectory = Environment.GetEnvironmentVariable("HELIX_WORKITEM_ROOT")`
	`51`	`+ var driverPath = string.isNullOrEmpty(helixDirectory) ? null : Path.Combine(helixDirectory, "runtests.sh");`
	`52`	`+ Playwright = await PlaywrightSharp.Playwright.CreateAsync(_loggerFactory, driverExecutablePath: driverPath /, debug: "pw:api"/);`
`50`	`53`	`foreach (var (browserName, options) in _browserManagerConfiguration.BrowserOptions)`
`51`	`54`	`{`
`52`	`55`	`if (!_launchBrowsers.ContainsKey(browserName))`