Skip to content

Commit

Permalink
fix: Explicitly set executable_path for Selenium (#7715)
Browse files Browse the repository at this point in the history
* fix: Explicitly set `executable_path` for Selenium

So it finds `geckodriver` again.

* Minimize diff

* fix: use existing executable_name

Co-authored-by: Lars Eggert <[email protected]>

---------

Co-authored-by: Robert Sparks <[email protected]>
  • Loading branch information
larseggert and rjsparks committed Jul 21, 2024
1 parent c5ca0ea commit 363c01e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ietf/utils/jstest.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
print(" "+skip_message)

def start_web_driver():
service = Service(log_output=f"{executable_name}.log", service_args=['--log-no-truncate'])
service = Service(executable_path=f"/usr/bin/{executable_name}", log_output=f"{executable_name}.log", service_args=['--log-no-truncate'])
options = Options()
options.add_argument("--headless")
os.environ["MOZ_REMOTE_SETTINGS_DEVTOOLS"] = "1"
Expand Down

0 comments on commit 363c01e

Please sign in to comment.