* Simplify test code by always setting up mock 'in memory' user and inventory data plugins for every TestCommunicationsManager

* imo the gain in simplcity of test code outweighs the very small cost of setting up some stuff that some tests will never use

1 files changed, 20 insertions, 3 deletions

diff --git a/OpenSim/Tests/Common/Mock/TestCommunicationsManager.cs b/OpenSim/Tests/Common/Mock/TestCommunicationsManager.cs
index 204ebd7..c234429 100644
--- a/OpenSim/Tests/Common/Mock/TestCommunicationsManager.cs
+++ b/OpenSim/Tests/Common/Mock/TestCommunicationsManager.cs
@@ -35,6 +35,18 @@ namespace OpenSim.Tests.Common.Mock
 {
     public class TestCommunicationsManager : CommunicationsManager
     {
+        public IUserDataPlugin UserDataPlugin
+        {
+            get { return m_userDataPlugin; }
+        }
+        private IUserDataPlugin m_userDataPlugin;
+        
+        public IInventoryDataPlugin InventoryDataPlugin
+        {
+            get { return m_inventoryDataPlugin; }
+        }
+        private IInventoryDataPlugin m_inventoryDataPlugin;
+        
         public TestCommunicationsManager()
             : this(null)
         {          
@@ -43,16 +55,21 @@ namespace OpenSim.Tests.Common.Mock
         public TestCommunicationsManager(NetworkServersInfo serversInfo)
             : base(serversInfo, null, null, false, null)
         {
+            m_userDataPlugin = new TestUserDataPlugin();
+            m_inventoryDataPlugin = new TestInventoryDataPlugin();
+                     
             LocalInventoryService lis = new LocalInventoryService();
+            lis.AddPlugin(m_inventoryDataPlugin);
             m_interServiceInventoryService = lis;
-            AddInventoryService(lis);
+            AddInventoryService(lis);      
             
             LocalUserServices lus = new LocalUserServices(991, 992, lis);
+            lus.AddPlugin(m_userDataPlugin);
             m_userService = lus;
-            m_userAdminService = lus;   
+            m_userAdminService = lus;         
             
             LocalBackEndServices gs = new LocalBackEndServices();
-            m_gridService = gs;
+            m_gridService = gs;          
         }
     }
 }