From 232d1af6634c590bc38f6f43c8e15ed9b553414f Mon Sep 17 00:00:00 2001
From: Sean Dague
Date: Thu, 18 Sep 2008 15:39:22 +0000
Subject: add some comments and some try blocks around object storage in the
 hopes this will help debug why this is failing for some people.

---
 OpenSim/Data/Tests/BasicRegionTest.cs | 29 ++++++++++++++++++++++++++---
 1 file changed, 26 insertions(+), 3 deletions(-)

(limited to 'OpenSim/Data/Tests/BasicRegionTest.cs')

diff --git a/OpenSim/Data/Tests/BasicRegionTest.cs b/OpenSim/Data/Tests/BasicRegionTest.cs
index 7b5284a..b5e9ed4 100644
--- a/OpenSim/Data/Tests/BasicRegionTest.cs
+++ b/OpenSim/Data/Tests/BasicRegionTest.cs
@@ -77,12 +77,31 @@ namespace OpenSim.Data.Tests
         {
             SceneObjectGroup sog = NewSOG("object1");
             SceneObjectGroup sog2 = NewSOG("object2");
-            
-            db.StoreObject(sog, region1);
-            db.StoreObject(sog2, region1);
+
+            // in case the objects don't store
+            try 
+            {
+                db.StoreObject(sog, region1);
+            }
+            catch (Exception e)
+            {
+                System.Console.WriteLine("Exception in storing object {0} {1}", sog.ToString(), e);
+                Assert.Fail();
+            }
+                    
+            try 
+            {
+                db.StoreObject(sog2, region1);
+            }
+            catch (Exception e)
+            {
+                System.Console.WriteLine("Exception in storing object {0} {1}", sog2.ToString(), e);
+                Assert.Fail();
+            }
 
             // This tests the ADO.NET driver
             List<SceneObjectGroup> objs = db.LoadObjects(region1);
+            
             Assert.That(objs.Count, Is.EqualTo(2));
         }
         
@@ -182,6 +201,10 @@ namespace OpenSim.Data.Tests
             return null;
         }
 
+        // This builds a minimalistic Prim, 1 SOG with 1 root SOP.  A
+        // common failure case is people adding new fields that aren't
+        // initialized, but have non-null db constraints.  We should
+        // honestly be passing more and more null things in here.
         private SceneObjectGroup NewSOG(string name)
         {
             SceneObjectGroup sog = new SceneObjectGroup();
-- 
cgit v1.1