orion-ecs / keen-eye / 20607590849

65%

Build Type

push

github

Committed by tyevco

Commit Message

Fix transport test port collisions in CI

Add LocalPort property to TcpTransport and UdpTransport that returns
the actual port bound after calling ListenAsync(). This enables tests
to use port 0 (OS-assigned ephemeral port) instead of random ports,
eliminating port collisions when tests run in parallel.

Also fix UdpTransport client socket binding - the socket must be bound
before starting the receive loop, otherwise ReceiveAsync fails with
"You must call the Bind method before performing this operation".

Changes:
- Add LocalPort property to TcpTransport (returns listener endpoint port)
- Add LocalPort property to UdpTransport (returns socket endpoint port)
- Fix UdpTransport.ConnectAsync to bind socket before receive loop
- Update all TCP tests to use port 0 and server.LocalPort
- Update all UDP tests to use port 0 and server.LocalPort
- Remove GetRandomPort() helper methods from test classes
- Add LocalPort_BeforeListen_ReturnsNegativeOne tests

Coverage Stats

4450 of 4801 branches covered (92.69%)

Branch coverage included in aggregate %.

3 of 3 new or added lines in 2 files covered. (100.0%)

7 existing lines in 3 files now uncovered.

28608 of 31167 relevant lines covered (91.79%)

1.27 hits per line