[dotnet] Selenium Manager use binary from Browser Options

diemol · diemol · commit e2bcb00f6005 · 2023-04-17T15:03:26.000+02:00
#11351
diff --git a/dotnet/src/webdriver/SeleniumManager.cs b/dotnet/src/webdriver/SeleniumManager.cs
@@ -60,9 +60,40 @@ public static string DriverPath(DriverOptions options)
                 argsBuilder.AppendFormat(CultureInfo.InvariantCulture, " --browser-version {0}", options.BrowserVersion);
             }
 
+            string browserBinary = BrowserBinary(options);
+            if (!string.IsNullOrEmpty(browserBinary))
+            {
+                argsBuilder.AppendFormat(CultureInfo.InvariantCulture, " --browser-path \"{0}\"", browserBinary);
+            }
+
+
             return RunCommand(binaryFile, argsBuilder.ToString());
         }
 
+
+        /// <summary>
+        /// Extracts the browser binary location from the vendor options when present. Only Chrome, Firefox, and Edge.
+        /// </summary>
+        private static string BrowserBinary(DriverOptions options)
+        {
+            ICapabilities capabilities = options.ToCapabilities();
+            string[] vendorOptionsCapabilities = { "moz:firefoxOptions", "goog:chromeOptions", "ms:edgeOptions" };
+            foreach (string vendorOptionsCapability in vendorOptionsCapabilities)
+            {
+                try
+                {
+                    Dictionary<string, object> vendorOptions = capabilities.GetCapability(vendorOptionsCapability) as Dictionary<string, object>;
+                    return vendorOptions["binary"] as string;
+                }
+                catch (Exception)
+                {
+                    // no-op, it would be ideal to at least log the exception but the C# do not log anything at the moment 
+                }
+            }
+
+            return null;
+        }
+
         /// <summary>
         /// Gets the location of the correct Selenium Manager binary.
         /// </summary>
