-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chromedriver version does not match chrome version #894
Comments
Full story: I have a working |
@zburke Could you try updating to https://github.com/thefrontside/bigtest/blob/v0/packages/bigtest/CHANGELOG.md#0133
|
@minkimcello no luck:
Additional details:
|
@zburke While we try to work through this, are you able to update Chrome? |
@cowboyd, no, because Chrome doesn't believe itself to be outdated. I would love to force an update, but the update instructions indicate "If you can't find [the "Update Google Chrome"] button, you're on the latest version." I can't find that button. IOW, my Chrome thinks it's on the latest version. If I understand Google's statement that releases "will roll out over the coming days/weeks", the problem is precisely that the release hasn't "rolled out" to me. My npm-based install from last week does work, and I can muddle through with that. I just have to manually keep track of the interactors I patch inside |
There are no quick fixes here, but one which we're considering is to remove the hard dependency on |
Yes and no. If the problem is identifying which
That's not really not a big deal (in fact, not really that different from |
The basic problem here is that if we add a dependency on The alternative is for the user to add a dependency on chromedriver directly, so instead of doing:
They would have to do:
And then they can use whichever version of chromedriver they want. Also frustrating is that basically all CI systems actually ship a version of chromedriver on their base images, which obviously matches their installed version of chromedriver. So it'd be better to just use that one! But just having the The solution to that problem would be not installing via npm at all. Personally I have a bit of a distaste for installing this type of binary package via I think there is no perfect solution here, it's all tradeoffs. We would like to get to the sweet spot of having a user experience that just works with no fiddling, but also perfect integration with CI systems where updates to the platform don't break your tests, but I think this is just not possible. @jenweber I think this would be a good issue for you to take a look at as well, since it's very much relevant to making a good onboarding process. I'm happy to explain this in more detail if the above isn't too clear. |
Lots of good info there about why this is such a thorny problem, @jnicklas! For me, the most important point is "no perfect solution here, it's all tradeoffs" but also my corollary: it's really not a big deal as long as you know you need to do it. i.e. as long as you say, "Now go install a compatible version of |
Error Report
Following the instructions at https://frontside.com/bigtest/docs/platform:
fails with
Chrome 89 was released on 2021-03-02, six days ago, but it takes several days/weeks for Chrome releases to actually roll out. My Chrome install does not report that it is outdated, for example, and thus there is no way to update to v89.
This was all working on Thursday when I left the office, but was broken when I returned today and, for various reasons, nuked by lock file and ran a fresh build. I figured maybe I could pin the
bigtest
orChromeDriver
deps to whatever I was using on Thursday, but a resolutions entry forbigtest
0.13.2
didn't change anything andChromeDriver
is not actually a dependency:diagnostics
Argv
server,--launch=chrome
Stack
Error: session not created: This version of ChromeDriver only supports Chrome version 89
Current browser version is 88.0.4324.192 with binary path /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
at request (/Users/zburke/temp/e2e-yarn/node_modules/@bigtest/webdriver/dist/web-driver.js:65:19)
at request.next ()
at /Users/zburke/temp/e2e-yarn/node_modules/effection/dist/effection.js:1:8145
at e.r.advance (/Users/zburke/temp/e2e-yarn/node_modules/effection/dist/effection.js:1:7781)
at e.r.trapExit (/Users/zburke/temp/e2e-yarn/node_modules/effection/dist/effection.js:1:8107)
at t.n.finalize (/Users/zburke/temp/e2e-yarn/node_modules/effection/dist/effection.js:1:5618)
at t.n.resume (/Users/zburke/temp/e2e-yarn/node_modules/effection/dist/effection.js:1:4972)
at /Users/zburke/temp/e2e-yarn/node_modules/effection/dist/effection.js:1:7289
at processTicksAndRejections (internal/process/task_queues.js:93:5)
The text was updated successfully, but these errors were encountered: