From 4748c19bdbcdcaf6050e1f04a5f7394a88e0bf3e Mon Sep 17 00:00:00 2001 From: Dan Lake Date: Thu, 6 Oct 2011 22:47:33 -0700 Subject: Refactored "known child region" in ScenePresence. There were 4 different ways to access the list/dictionary of child regions and locking was inconsistent. There are now public properties which enforce locks. Callers are no longer required to create new copies of lists. --- .../Region/Framework/Scenes/Tests/ScenePresenceAgentTests.cs | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'OpenSim/Region/Framework/Scenes/Tests/ScenePresenceAgentTests.cs') diff --git a/OpenSim/Region/Framework/Scenes/Tests/ScenePresenceAgentTests.cs b/OpenSim/Region/Framework/Scenes/Tests/ScenePresenceAgentTests.cs index ce9d418..119eb1f 100644 --- a/OpenSim/Region/Framework/Scenes/Tests/ScenePresenceAgentTests.cs +++ b/OpenSim/Region/Framework/Scenes/Tests/ScenePresenceAgentTests.cs @@ -215,11 +215,9 @@ namespace OpenSim.Region.Framework.Scenes.Tests string cap = presence.ControllingClient.RequestClientInfo().CapsPath; presence.AddNeighbourRegion(region2, cap); - presence.AddNeighbourRegion(region3, cap); - - List neighbours = presence.GetKnownRegionList(); - - Assert.That(neighbours.Count, Is.EqualTo(2)); + presence.AddNeighbourRegion(region3, cap); + + Assert.That(presence.KnownRegionCount, Is.EqualTo(2)); } [Test] @@ -230,8 +228,7 @@ namespace OpenSim.Region.Framework.Scenes.Tests ScenePresence presence = scene.GetScenePresence(agent1); presence.RemoveNeighbourRegion(region3); - List neighbours = presence.GetKnownRegionList(); - Assert.That(neighbours.Count,Is.EqualTo(1)); + Assert.That(presence.KnownRegionCount,Is.EqualTo(1)); /* presence.MakeChildAgent; presence.MakeRootAgent; -- cgit v1.1