Skip to content

Make WSL detection more generic#187

Merged
kapoorlakshya merged 1 commit intotitusfortner:masterfrom
rbclark:patch-1
Dec 18, 2020
Merged

Make WSL detection more generic#187
kapoorlakshya merged 1 commit intotitusfortner:masterfrom
rbclark:patch-1

Conversation

@rbclark
Copy link
Contributor

@rbclark rbclark commented Dec 17, 2020

On WSL2, the contents of /proc/version seem to have changed slightly, the command currently in webdrivers currently returns false:

irb(main):001:0> File.open('/proc/version').read.include?('Microsoft')
=> false
irb(main):002:0> File.open('/proc/version').read.downcase.include?('microsoft')
=> true

Full contents for completeness:

irb(main):003:0> File.open('/proc/version').read
=> "Linux version 4.19.104-microsoft-standard (oe-user@oe-host) (gcc version 8.2.0 (GCC)) #1 SMP Wed Feb 19 06:37:35 UTC 2020\n"

Unfortunately this does not seem to fix all of the issues, although I am unsure if the rest of the issues are in the webdrivers gem or not. I am still receiving a Selenium::WebDriver::Error::WebDriverError: unable to connect to chromedriver 127.0.0.1:9515 error when trying to run on WSL2 with a pretty barebones Rails system test setup.

@kapoorlakshya
Copy link
Collaborator

@rbclark Thanks for your contribution! Will try to release all the recent changes this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants