Files
f2625bf5c5 Removes IMapManager (#6584)
* Move MapManager queries to SharedMapSystem
Moves all variants of FindGridsIntersecting/TryFindGridAt to SharedMapSystem
Hollows out the MapManager methods and converts them into relays to SharedMapSystem

* Move CreateGrid to SharedMapSystem
Moves the functionality for CreateGrid and its variants to SharedMapSystem
Hollows out the MapManager methods and converts them into relays for the SharedMapSystem methods
Obsoletes them too
Also moves over the GetAllMapGrids and GetAllGrids methods

* Move RaiseOnTileChanged to SharedMapSystem
Also moves the SuppressOnTileChanged flag member to SharedMapSystem
Hollows out and obsoletes the MapManager versions

* Move default value constants to SharedMapSystem

* Converts map pausing events into LocalizedEntityCommands

* Move MapManager related delegates/structs to SharedMapSystem
Moves the GridCreationOptions struct to the same namespace as SharedMapSystem and converts it into a record struct
Move the GridCallback delegates to the same namespace as SharedMapSystem

* Move CullDeletionHistory to SharedMapSystem
Well, that was less painful than I thought it would be

* Actually obsolete the NetworkedMapManager method

* Rename file

* Doc comments for new SharedMapSystem methods

* Doc comment

* Doc comments

* Fix access

* Purge all references to IMapManagerInternal

* Cull easy IMapManager references

* The rest

* Purge IMapManager

* Private internal FindGridsIntersecting method

* please die

* 41

* notes

---------

Co-authored-by: ArtisticRoomba <145879011+ArtisticRoomba@users.noreply.github.com>
2026-07-01 19:32:17 -07:00

101 lines
3.6 KiB
C#

using System.Linq;
using System.Numerics;
using System.Threading.Tasks;
using NUnit.Framework;
using Robust.Shared;
using Robust.Shared.Configuration;
using Robust.Shared.GameObjects;
using Robust.Shared.Map;
using Robust.Shared.Maths;
using Robust.Shared.Player;
namespace Robust.UnitTesting.Server.GameStates;
public sealed class PvsSystemTests : RobustIntegrationTest
{
/// <summary>
/// Checks that there are no issues when an entity changes PVS chunk location multiple times in a single tick.
/// </summary>
[Test]
public async Task TestMultipleIndexChange()
{
await using var pair = await StartConnectedPair();
var (client, server) = pair;
var sEntMan = server.ResolveDependency<IEntityManager>();
var confMan = server.ResolveDependency<IConfigurationManager>();
var sPlayerMan = server.ResolveDependency<ISharedPlayerManager>();
var xforms = sEntMan.System<SharedTransformSystem>();
var maps = sEntMan.System<SharedMapSystem>();
var cEntMan = client.ResolveDependency<IEntityManager>();
var cPlayerMan = client.ResolveDependency<ISharedPlayerManager>();
server.Post(() => confMan.SetCVar(CVars.NetPVS, true));
await RunTicksSync(server, client, 10);
// Set up map and grid
EntityUid grid = default;
EntityUid map = default;
await server.WaitPost(() =>
{
map = server.System<SharedMapSystem>().CreateMap(out var mapId);
var gridComp = maps.CreateGridEntity(mapId);
maps.SetTile(gridComp, Vector2i.Zero, new Tile(1));
grid = gridComp.Owner;
});
// Spawn player entity on grid 1
EntityUid player = default;
EntityUid other = default;
TransformComponent otherXform = default!;
var gridCoords = new EntityCoordinates(grid, new Vector2(0.5f, 0.5f));
var mapCoords = new EntityCoordinates(map, new Vector2(2, 2));
await server.WaitPost(() =>
{
player = sEntMan.SpawnEntity(null, gridCoords);
other = sEntMan.SpawnEntity(null, gridCoords);
otherXform = sEntMan.GetComponent<TransformComponent>(other);
// Ensure map PVS chunk is not empty
sEntMan.SpawnEntity(null, mapCoords);
// Attach player.
var session = sPlayerMan.Sessions.First();
server.PlayerMan.SetAttachedEntity(session, player);
sPlayerMan.JoinGame(session);
});
await RunTicksSync(server, client, 10);
// Check player got properly attached
await client.WaitPost(() =>
{
var ent = cEntMan.GetNetEntity(cPlayerMan.LocalEntity);
Assert.That(ent, Is.EqualTo(sEntMan.GetNetEntity(player)));
});
// Move the player off-grid and back onto the grid in the same tick
xforms.SetCoordinates(other, otherXform, mapCoords);
xforms.SetCoordinates(other, otherXform, gridCoords);
// Run for a few ticks. The test just checks that no PVS asserts/errors happen.
await RunTicksSync(server, client, 10);
// Repeat but in the opposite direction ( map -> grid -> map )
// first move to map and wait a bit.
xforms.SetCoordinates(other, otherXform, mapCoords);
await RunTicksSync(server, client, 10);
// Move to and off grid in the same tick
xforms.SetCoordinates(other, otherXform, gridCoords);
xforms.SetCoordinates(other, otherXform, mapCoords);
// wait for errors.
await RunTicksSync(server, client, 10);
}
}