fix(webdriverjs): use new headless argument to fix [email protected] breaking release (#992)

* fix(webdriverjs): use new headless argument to fix [email protected] breaking release

* fix webdriverjs tests to use addArguments too

* fix cli test

* use --headless instead of --headless=new

* use headless to reduce change diff

Author: straker

package-lock.json

@@ -52,7 +52,7 @@
     "chromedriver": "latest",
     "colors": "^1.4.0",
     "commander": "^9.4.1",
-    "selenium-webdriver": "^4.8.1"
+    "selenium-webdriver": "~4.17.0"
   },
   "devDependencies": {
     "@types/chai": "^4.3.3",

packages/cli/package.json

@@ -15,7 +15,16 @@ const startDriver = async (
       config.chromedriverPath || chromedriver.path
     );
 
-    let options = new chrome.Options().headless();
+    let options = new chrome.Options();
+    // selenium-webdriver < 4.17.0
+    if (typeof options.headless === 'function') {
+      options.headless();
+    }
+    // selenium-webdriver >= 4.17.0
+    else {
+      options.addArguments('headless');
+    }
+
     if (config.chromeOptions?.length) {
       options = config.chromeOptions.reduce(function (options, arg) {
         return options.addArguments(arg);

packages/cli/src/lib/webdriver.ts

@@ -5,7 +5,7 @@ import chrome from 'selenium-webdriver/chrome';
 export const Webdriver = (): WebDriver => {
   const builder = new Builder()
     .forBrowser('chrome')
-    .setChromeOptions(new chrome.Options().headless())
+    .setChromeOptions(new chrome.Options().addArguments('headless'))
     .setChromeService(
       new chrome.ServiceBuilder(
         process.env.CHROMEDRIVER_PATH || chromedriver.path