Merge 8de73fe into 669a493

titusfortner · web-flow · commit 7805592b6d2e · 2023-06-04T00:54:51.000Z
diff --git a/.bazelrc b/.bazelrc
@@ -55,7 +55,6 @@ build --test_env=GITHUB_ACTIONS
 build --test_env=MOZ_HEADLESS
 build --test_env=PATH  # Remove once browser pinning works
 build --test_env=SELENIUM_BROWSER
-build --test_env=SELENIUM_MANAGER_BASE_PATH
 build --test_env=TRAVIS
 build --test_env=PYTHON_VERSION
 
diff --git a/.github/workflows/bazel.yml b/.github/workflows/bazel.yml
@@ -55,6 +55,10 @@ jobs:
       - name: Free space
         if: inputs.os != 'windows'
         run: ./scripts/github-actions/free-disk-space.sh
+      - name: Remove driver directories
+        if: inputs.os != 'windows'
+        run: |
+          sudo rm -rf $CHROMEWEBDRIVER $EDGEWEBDRIVER $GECKOWEBDRIVER
       - name: Set Ruby version
         if: inputs.ruby-version != ''
         run: echo 'RUBY_VERSION = "${{ inputs.ruby-version }}"' > rb/ruby_version.bzl
diff --git a/javascript/node/selenium-webdriver/BUILD.bazel b/javascript/node/selenium-webdriver/BUILD.bazel
@@ -90,6 +90,9 @@ jasmine_node_test(
     local = True,
     templated_args = ["--node_options=--require=$$(rlocation $(rootpath :tools/init_jasmine.js))"],
     deps = [
+        ":manager-linux",
+        ":manager-macos",
+        ":manager-windows",
         "//javascript/node/selenium-webdriver/lib/atoms:find-elements",
         "//javascript/node/selenium-webdriver/lib/atoms:get_attribute",
         "//javascript/node/selenium-webdriver/lib/atoms:is_displayed",
diff --git a/javascript/node/selenium-webdriver/common/seleniumManager.js b/javascript/node/selenium-webdriver/common/seleniumManager.js
@@ -47,12 +47,7 @@ function getBinary() {
   const file =
     directory === 'windows' ? 'selenium-manager.exe' : 'selenium-manager'
 
-  let seleniumManagerBasePath
-  if (process.env.SELENIUM_MANAGER_BASE_PATH) {
-    seleniumManagerBasePath = process.env.SELENIUM_MANAGER_BASE_PATH
-  } else {
-    seleniumManagerBasePath = path.join(__dirname, '..', '/bin')
-  }
+  let seleniumManagerBasePath = path.join(__dirname, '..', '/bin')
 
   const filePath = path.join(seleniumManagerBasePath, directory, file)
 
