-
Notifications
You must be signed in to change notification settings - Fork 85
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
chore: upgrade SauceLabs launcher to the latest version #8573
Conversation
4d38c5f
to
0df4863
Compare
There are some problems with custom "set-window-height" command, I'll look into switching to
|
Created #8577 to fix the CSS override for |
b860f87
to
e675786
Compare
e675786
to
2231624
Compare
|
@web-padawan We'll need the SauceLabs version bump also in version branches in order to run visual tests from there. |
Right, added labels to cherry-pick this PR. I think we can ignore webdriver errors in the logs for now since the tests pass. |
Hi @web-padawan and @web-padawan, when i performed cherry-pick to this commit to 24.5, i have encountered the following issue. Can you take a look and pick it manually? |
Hi @web-padawan and @web-padawan, when i performed cherry-pick to this commit to 23.5, i have encountered the following issue. Can you take a look and pick it manually? |
Co-authored-by: Serhii Kulykov <[email protected]>
This ticket/PR has been released with Vaadin 24.7.0.alpha7 and is also targeting the upcoming stable 24.7.0 version. |
Description
My PR to
@web/test-runner-saucelabs
was merged and released: modernweb-dev/web#2852package.json
, let's see if visual tests pass.filterBrowserLogs
to filter outwebdriver
debug warnings from their customElements wrapperType of change
Note
There are some webdriver errors when running tests for more than a single file.
Added a comment about that here webdriverio/webdriverio#13467 (comment)
UPD: reported a new issue for this error webdriverio/webdriverio#14129