[rust] Support for multiple browser names in Selenium Manager (#11352) (#11420)

bonigarcia · web-flow · commit 8467a8c605c2 · 2022-12-13T17:03:01.000+01:00
* [rust] Support for multiple browser names in Selenium Manager (#11352) * [rust] Use vector of broweser name to create a shorter conditional
diff --git a/rust/src/lib.rs b/rust/src/lib.rs
@@ -288,13 +288,22 @@ pub trait SeleniumManager {
 // ----------------------------------------------------------
 
 pub fn get_manager_by_browser(browser_name: String) -> Result<Box<dyn SeleniumManager>, String> {
-    if browser_name.eq_ignore_ascii_case("chrome") {
+    let browser_name_lower_case = browser_name.to_ascii_lowercase();
+    if browser_name_lower_case.eq("chrome") {
         Ok(ChromeManager::new())
-    } else if browser_name.eq_ignore_ascii_case("firefox") {
+    } else if browser_name.eq("firefox") {
         Ok(FirefoxManager::new())
-    } else if browser_name.eq_ignore_ascii_case("edge") {
+    } else if vec!["edge", "msedge", "microsoftedge"].contains(&browser_name_lower_case.as_str()) {
         Ok(EdgeManager::new())
-    } else if browser_name.eq_ignore_ascii_case("iexplorer") {
+    } else if vec![
+        "iexplorer",
+        "ie",
+        "internetexplorer",
+        "internet-explorer",
+        "internet_explorer",
+    ]
+    .contains(&browser_name_lower_case.as_str())
+    {
         Ok(IExplorerManager::new())
     } else {
         Err(format!("Invalid browser name: {browser_name}"))
