You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Listening on same port twice does not fail on Windows #56
Test suite for v0.4.4 fails on Windows, using both 5.6.29 and 7.0.13, with PHPUnit 4.8.31
PHPUnit 4.8.31 by Sebastian Bergmann and contributors.
...............F
Time: 21.31 seconds, Memory: 8.00MB
There was 1 failure:
1) React\Tests\Socket\ServerTest::testListenOnBusyPortThrows
Failed asserting that exception of type "React\Socket\ConnectionException" is thrown.
FAILURES!
Tests: 16, Assertions: 15, Failures: 1.
Edit: expanded information to include full phpunit output
@shaunbramley Thanks for reporting issue. Do you have some more information about ? Like "what happens if the exception is not throwned" ? And "how did you run test suite" (environment/terminal/etc) ?
Confirmed test suite works as expected on macOS Sierra w/ PHP 7.0.14 (via homebrew)
@Nek- Windows 10 Pro, ran within Powershell (both as Administrator, and regular user). Steps to reproduce:
git -b checkout dev-pin-phpunit-4.8 v0.4.4
composer install
composer require --dev phpunit/phpunit:4.8
vendor/bin/phpunit
Something else that would indicate that something is going on "under the hood" is that when executed on Windows is the time required for the test suite to execute. On windows the test suite takes ~20s; whereas on my macOS system it takes ~230ms.
I'd like to make this actionable in order to resolve / close this ticket. It looks like this may be a Windows-only issue, so we may also skip this test on affected versions.
Does anybody else happen to have installed PHP on Windows and can help to reproduce the following does indeed not emit any errors?
Activity
Nek- commentedon Jan 6, 2017
@shaunbramley Thanks for reporting issue. Do you have some more information about ? Like "what happens if the exception is not throwned" ? And "how did you run test suite" (environment/terminal/etc) ?
clue commentedon Jan 7, 2017
@shaunbramley thanks for reporting! 👍
Unfortunately I do not have a system to test this on, so I guess we could use some help to track this down.
Which version of Windows has this been tested on, can others confirm this issue?
The test essentially boils down to this:
See also https://3v4l.org/XjhRa
Can you and/or anybody else verify this fails? Thanks! 👍
shaunbramley commentedon Jan 7, 2017
@clue - sorry for the delay. I've had my son all week. :)
Some extended testing would seem to indicate that this might be a PHP issue.
expected output:
Actual output:
Confirmed test suite works as expected on macOS Sierra w/ PHP 7.0.14 (via homebrew)
@Nek- Windows 10 Pro, ran within Powershell (both as Administrator, and regular user). Steps to reproduce:
Something else that would indicate that something is going on "under the hood" is that when executed on Windows is the time required for the test suite to execute. On windows the test suite takes ~20s; whereas on my macOS system it takes ~230ms.
[-]Failed listening on busy port[/-][+]Listening on same port twice does not fail on Windows[/+]clue commentedon Jan 8, 2017
@shaunbramley thanks for reporting back and no worries, family time is precious :) 👍
Thanks for the steps to reproduce this, I've updated the issue title and hope anybody else may have some ideas here 👍
clue commentedon Feb 5, 2017
I'd like to make this actionable in order to resolve / close this ticket. It looks like this may be a Windows-only issue, so we may also skip this test on affected versions.
Does anybody else happen to have installed PHP on Windows and can help to reproduce the following does indeed not emit any errors?
Any input would be appreciated here 👍 (Please note Windows and PHP version)
jsor commentedon Feb 5, 2017
I tested that on Windows 7 with PHP 5.6.30 and 7.1.1 and both do not emit an error but return a resource for
$aand$b.