branchrust_reboot/main/hascloseconnections_fixcancel
6 Commits over 0 Days - ∞cph!
Bugfix: ensure players are detected on the border of the network cell
- expanded unit tests to cover these cases
Tests: ran unit tests
Bugfix: fix HasConnectionsClose & GetCloseConnections missing players in medium and large ranges
- also add a unit test to validate Connections overload
Tests: ran unit tests
Bugfix(tests): adjust player spawn distance for TestNetworkRange.OutsideRange cases
They can land on a boundary, making it more difficult to reason when working across small/medium/large layers
Tests: ran the tests, same results
Bugfix(tests): expand TestServer's network grid from 1k to 2k
Previously it was too easy to accidentally setup an out-of-bounds scenario, leading to confusing test failures
Tests: ran related tests, discovered couple tests are failing (unrelated to this change)
New(tests): tests for BaseNetworkable.HasCloseConnections and GetCloseConnections
- named cell sizes for various layers in NetworkVisibilityGrid
Tests: ran tests, failing where expected
