fix: Work around msedgedriver 115+ bug (#60)

Specify the explicit path to microsoft-edge if found, to work around
MicrosoftEdge/EdgeWebDriver#102 (comment)

This also incorporates
shaka-project/webdriver-installer#35 to revert a
previous workaround at that level.

index.js

@@ -19,6 +19,7 @@ const fs = require('fs');
 const os = require('os');
 const path = require('path');
 const wd = require('wd');
+const which = require('which');
 const _ = require('lodash');
 
 const {installWebDrivers} = require('webdriver-installer');
@@ -41,6 +42,18 @@ const PLATFORM_MAP = {
   'linux': 'Linux',
 };
 
+function mergeOptions(base, custom) {
+  // _.mergeWith modifies the first argument, so clone the base structure first.
+  const output = _.cloneDeep(base);
+  return _.mergeWith(output, custom,
+      // Concatenate arrays instead of overwriting them.
+      (objValue, srcValue) => {
+        if (Array.isArray(objValue)) {
+          return objValue.concat(srcValue);
+        }
+      });
+}
+
 // Subclasses must define these static members:
 //  - BROWSER_NAME: browser name as presented to WebDriver
 //  - LAUNCHER_NAME: launcher name as presented to Karma
@@ -68,14 +81,9 @@ const LocalWebDriverBase = function(baseBrowserDecorator, args, logger) {
 
   log.debug('config:', JSON.stringify(config));
 
-  const extraSpecs = _.mergeWith(
+  const extraSpecs = mergeOptions(
       this.constructor.EXTRA_WEBDRIVER_SPECS,
-      args.config,
-      (objValue, srcValue) => {
-        if (Array.isArray(objValue)) {
-          return objValue.concat(srcValue);
-        }
-      });
+      args.config);
   log.debug('extraSpecs:', extraSpecs);
 
   // These names ("browser" and "spec") are needed for compatibility with
@@ -288,23 +296,35 @@ const LocalWebDriverChromeHeadless = generateSubclass(
 
 // TODO: Add Chrome on android?
 
+// If a binary is found with the name "microsoft-edge" in the PATH, specify
+// that explicitly.  This works around the following edgedriver bug:
+// https://github.com/MicrosoftEdge/EdgeWebDriver/issues/102#issuecomment-1710724173
+const edgeOptions = {};
+const edgeBinary = which.sync('microsoft-edge');
+if (edgeBinary) {
+  edgeOptions['ms:edgeOptions'] = {
+    binary: edgeBinary,
+  };
+}
+
 const LocalWebDriverEdge = generateSubclass(
     'MSEdge', 'MSEdge',
     'msedgedriver',
-    (port) => ['--port=' + port]);
+    (port) => ['--port=' + port],
+    edgeOptions);
 
 const LocalWebDriverEdgeHeadless = generateSubclass(
     'MSEdge', 'MSEdgeHeadless',
     'msedgedriver',
     (port) => ['--port=' + port],
-    {
+    mergeOptions(edgeOptions, {
       'ms:edgeOptions': {
         args: [
           '--headless',
           '--disable-gpu',
         ],
       },
-    });
+    }));
 
 const LocalWebDriverFirefox = generateSubclass(
     'Firefox', 'Firefox',

package.json

@@ -18,8 +18,9 @@
   "author": "Joey Parrish <joeyparrish@google.com>",
   "dependencies": {
     "lodash": "^4.17.21",
+    "node-which": "^1.0.0",
     "wd": "^1.14.0",
-    "webdriver-installer": "^1.1.8"
+    "webdriver-installer": "^1.1.9"
   },
   "peerDependencies": {
     "karma": "^6.2.0"