Merge branch 'trunk' into js-bidi-storage

Author: navin772

rb/lib/selenium/webdriver/remote/bridge.rb

@@ -323,7 +323,7 @@ def add_cookie(cookie)
         end
 
         def delete_cookie(name)
-          raise ArgumentError, 'Cookie name cannot be null or empty' if name.nil? || name.strip.empty?
+          raise ArgumentError, 'Cookie name cannot be null or empty' if name.nil? || name.to_s.strip.empty?
 
           execute :delete_cookie, name: name
         end

rb/spec/integration/selenium/webdriver/manager_spec.rb

@@ -268,6 +268,12 @@ module WebDriver
           expect { driver.manage.delete_cookie(nil) }
             .to raise_error(ArgumentError, /Cookie name cannot be null or empty/)
         end
+
+        it 'allows deleting a cookies using a symbol' do
+          driver.manage.add_cookie name: :foo, value: 'bar'
+          driver.manage.delete_cookie(:foo)
+          expect(driver.manage.all_cookies).to be_empty
+        end
       end
     end # Options
   end # WebDriver

rust/tests/browser_tests.rs

@@ -20,6 +20,7 @@ use crate::common::{assert_output, get_selenium_manager, get_stdout};
 use exitcode::DATAERR;
 use rstest::rstest;
 use std::env::consts::OS;
+use std::path::Path;
 
 mod common;
 
@@ -134,7 +135,7 @@ fn invalid_geckodriver_version_test() {
     r"/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"
 )]
 fn browser_path_test(#[case] os: String, #[case] browser: String, #[case] browser_path: String) {
-    if OS.eq(&os) {
+    if OS.eq(&os) && Path::new(&browser_path).exists() {
         let mut cmd = get_selenium_manager();
         cmd.args(["--browser", &browser, "--browser-path", &browser_path])
             .assert()