modified delete cookie added code and test (SeleniumHQ#15386)

Author: rpallavisharma

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

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

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

@@ -237,7 +237,6 @@ module WebDriver
 
         it 'deletes one' do
           driver.manage.add_cookie name: 'foo', value: 'bar'
-
           driver.manage.delete_cookie('foo')
           expect(driver.manage.all_cookies.find { |c| c[:name] == 'foo' }).to be_nil
         end
@@ -253,6 +252,16 @@ module WebDriver
           expect { driver.manage.cookie_named('non-existent') }
             .to raise_exception(Error::NoSuchCookieError)
         end
+
+        it 'throws an error when cookie name is an empty string' do
+          expect { driver.manage.delete_cookie('') }
+            .to raise_error(ArgumentError, /Cookie name cannot be null or empty/)
+        end
+
+        it 'throws an error when cookie name is a nil string' do
+          expect { driver.manage.delete_cookie(nil) }
+            .to raise_error(ArgumentError, /Cookie name cannot be null or empty/)
+        end
       end
     end # Options
   end # WebDriver