[JS] Add detailed error message for invalid cookie name validation in getCookie method

Author: harsha509

javascript/node/selenium-webdriver/lib/webdriver.js

@@ -1957,25 +1957,24 @@ class Options {
    * WebDriver wire protocol.
    *
    * @param {string} name The name of the cookie to retrieve.
+   * @throws {InvalidArgumentError} - If the cookie name is empty or invalid.
    * @return {!Promise<?Options.Cookie>} A promise that will be resolved
    *     with the named cookie
    * @throws {error.NoSuchCookieError} if there is no such cookie.
    */
   async getCookie(name) {
+    // Validate the cookie name is non-empty and properly trimmed.
+    if (!name?.trim()) {
+      throw new error.InvalidArgumentError('Cookie name cannot be empty')
+    }
+
     try {
       const cookie = await this.driver_.execute(new command.Command(command.Name.GET_COOKIE).setParameter('name', name))
       return cookie
     } catch (err) {
       if (!(err instanceof error.UnknownCommandError) && !(err instanceof error.UnsupportedOperationError)) {
         throw err
       }
-
-      const cookies = await this.getCookies()
-      for (let cookie of cookies) {
-        if (cookie && cookie['name'] === name) {
-          return cookie
-        }
-      }
       return null
     }
   }

javascript/node/selenium-webdriver/test/cookie_test.js

@@ -54,6 +54,15 @@ suite(function (env) {
         })
     })
 
+    it('throw error if name is null', async function () {
+      const cookie = createCookieSpec()
+      await driver.manage().addCookie(cookie)
+      await assert.rejects(async () => await driver.manage().getCookie(null), {
+        name: 'InvalidArgumentError',
+        message: `Cookie name cannot be empty`,
+      })
+    })
+
     it('can get all cookies', async function () {
       const cookie1 = createCookieSpec()
       const cookie2 = createCookieSpec()