Skip to content
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

Update docker-compose file #61

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Conversation

uuf6429
Copy link
Member

@uuf6429 uuf6429 commented Jan 8, 2025

Mainly this is to remove "version", which has been obsoleted.

I've additionally changed the indentation as well.

Copy link

codecov bot commented Jan 8, 2025

❌ 3 Tests Failed:

Tests completed Failed Passed Skipped
240 3 237 20
View the top 2 failed tests by shortest run time
Behat.Mink.Tests.Driver.Js.WindowTest::testSwitchingToMissingWindow
Stack Traces | 300s run time
Behat\Mink\Tests\Driver\Js\WindowTest::testSwitchingToMissingWindow
Facebook\WebDriver\Exception\Internal\WebDriverCurlException: Curl error thrown for http POST to .../session/84194abd2281f3ea68a2bc87900c6ec8/url with params: {"url":"http://host.docker.internal:8002/window.html"}

Operation timed out after 180002 milliseconds with 0 bytes received

.../Exception/Internal/WebDriverCurlException.php:20
.../lib/Remote/HttpCommandExecutor.php:328
.../lib/Remote/RemoteWebDriver.php:601
.../lib/Remote/RemoteExecuteMethod.php:23
.../webdriver/lib/WebDriverNavigation.php:41
.../webdriver-classic-driver/src/WebdriverClassicDriver.php:175
.../mink/src/Session.php:175
.../tests/Js/WindowTest.php:115
Behat.Mink.Tests.Driver.Js.WindowTest::testGetWindowNames
Stack Traces | 300s run time
Behat\Mink\Tests\Driver\Js\WindowTest::testGetWindowNames
Facebook\WebDriver\Exception\Internal\WebDriverCurlException: Curl error thrown for http POST to .../84194abd2281f3ea68a2bc87900c6ec8/execute/sync with params: {"script":"return window.name;","args":[]}

Operation timed out after 180002 milliseconds with 0 bytes received

.../Exception/Internal/WebDriverCurlException.php:20
.../lib/Remote/HttpCommandExecutor.php:328
.../lib/Remote/RemoteWebDriver.php:601
.../lib/Remote/RemoteWebDriver.php:345
.../webdriver-classic-driver/src/WebdriverClassicDriver.php:681
.../webdriver-classic-driver/src/WebdriverClassicDriver.php:283
.../webdriver-classic-driver/src/WebdriverClassicDriver.php:273
.../mink/src/Session.php:300
.../tests/Js/WindowTest.php:64
View the full list of 1 ❄️ flaky tests
Behat.Mink.Tests.Driver.Js.WindowTest::testResizeWindow

Flake rate in main: 13.89% (Passed 62 times, Failed 10 times)

Stack Traces | 300s run time
Behat\Mink\Tests\Driver\Js\WindowTest::testResizeWindow
Facebook\WebDriver\Exception\Internal\WebDriverCurlException: Curl error thrown for http POST to .../session/84194abd2281f3ea68a2bc87900c6ec8/url with params: {"url":"http://host.docker.internal:8002/index.html"}

Operation timed out after 180001 milliseconds with 0 bytes received

.../Exception/Internal/WebDriverCurlException.php:20
.../lib/Remote/HttpCommandExecutor.php:328
.../lib/Remote/RemoteWebDriver.php:601
.../lib/Remote/RemoteExecuteMethod.php:23
.../webdriver/lib/WebDriverNavigation.php:41
.../webdriver-classic-driver/src/WebdriverClassicDriver.php:175
.../mink/src/Session.php:175
.../tests/Js/WindowTest.php:73

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@uuf6429
Copy link
Member Author

uuf6429 commented Jan 8, 2025

Seems that these sudden failures relate to the newly-added "switch to missing window" test.

I guess this PR could still be merged, but maybe it makes sense to revert that test, at least temporarily. 😞

@aik099
Copy link
Member

aik099 commented Jan 9, 2025

For me locall both "webdriver-classic-driver" (this driver) and MinkSelenium2Drver passed switching to missing window test.

@aik099
Copy link
Member

aik099 commented Jan 9, 2025

That's what I call a flaky test. Some of the window operation-related tests are randomly failing on GitHub Actions. I have no idea why.

@uuf6429
Copy link
Member Author

uuf6429 commented Jan 9, 2025

I'm starting to get the feeling there might be some problem with the github runners - I also had weird issues when I was trying to set up the chrome driver tests on github.

I'll investigate that later.

@uuf6429 uuf6429 merged commit fb117d5 into main Jan 9, 2025
13 of 32 checks passed
@uuf6429 uuf6429 deleted the chore/update-docker-compose-file branch January 9, 2025 09:28
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