Fix optional test for `ENETUNREACH` error condition

reactphp / socket

Async, streaming plaintext TCP/IP and secure TLS socket server and client connections for ReactPHP.

https://reactphp.org/socket/

MIT License

1.21k stars 157 forks source link

Fix optional test for `ENETUNREACH` error condition #288

Closed clue closed 2 years ago

clue commented 2 years ago

This minor changeset fixes the optional test for the ENETUNREACH error condition. This test is skipped on many platforms and as such also wasn't executed/spotted on GitHub Actions. It looks like this minor test issue has been introduced via #283. Originally introduced via #265. Spotted while working on https://github.com/reactphp/reactphp/pull/454

clue commented 2 years ago

For the reference: Given that this test is skipped on most platforms anyway, I've reduced the time it takes to skip from ~20s to ~2s. The total test time is now down from ~23s to ~5s.