search

walkor / workerman

An asynchronous event driven PHP socket framework. Supports HTTP, Websocket, SSL and other custom protocols.
http://www.workerman.net
MIT License
11.03k stars 2.25k forks source link

Add Windows and Mac Os to tests #951

Closed joanhey closed 10 months ago

joanhey commented 11 months ago

So we'll find problems faster for that OSes.

I think that will be good to also add test for each event-loop system(event, swoole, revolt, ...)

joanhey commented 11 months ago

Seems that exist a problem with Guzzle

image

It's working, but it'll fail in the future. I'll try to fix it in other PR.

joanhey commented 11 months ago

@walkor if you want, you can merge it in another branch, till we fix the code or tests. But first merge the PR #950, so we will have the extra info.

joanhey commented 11 months ago

In Mac only fail the unit UdpConnectionTest.

In windows can't install posix and pcntl, and it's normal, but need to pass the test without both image And 5 failed tests, that perhaps are from the symfony/process, we need more investigation. In Adapterman with Workerman 4.1, Windows only fail in the Upload test (investigating).

We can skip tests in Windows/Mac/Linux if it's necessary. image

joanhey commented 10 months ago

@walkor you can check the Adapterman tests working with Ubuntu, MacOS and Windows. Using the same code and .yml.

https://github.com/joanhey/AdapterMan/actions/runs/6094582441/job/16536405869#step:5:11

We need to update the tests to work. Or mark the tests than never will work with an OS.