Enable one sub-test in TestJsonSetValue() which now works (using identifier with embedded .).

Need to look further at other still commented tests.
Still need to check coverage against some of Mic's scripts.

1 files changed, 13 insertions, 17 deletions

diff --git a/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs b/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs
index ffa8250..3d9ad16 100644
--- a/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs
+++ b/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs
@@ -519,17 +519,16 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore.Tests
                 Assert.That(value, Is.EqualTo("Times"));
             }
 
-            // Commented out as this currently unexpectedly fails.
             // Test setting a key containing periods with delineation
-//            {
-//                UUID storeId = (UUID)InvokeOp("JsonCreateStore", "{}"); 
-//
-//                int result = (int)InvokeOp("JsonSetValue", storeId, "{Fun.Circus}", "Times");
-//                Assert.That(result, Is.EqualTo(1));
-//
-//                string value = (string)InvokeOp("JsonGetValue", storeId, "{Fun.Circus}");
-//                Assert.That(value, Is.EqualTo("Times"));
-//            }
+            {
+                UUID storeId = (UUID)InvokeOp("JsonCreateStore", "{}"); 
+
+                int result = (int)InvokeOp("JsonSetValue", storeId, "{Fun.Circus}", "Times");
+                Assert.That(result, Is.EqualTo(1));
+
+                string value = (string)InvokeOp("JsonGetValue", storeId, "{Fun.Circus}");
+                Assert.That(value, Is.EqualTo("Times"));
+            }
 
             // *** Test [] ***
 
@@ -599,7 +598,7 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore.Tests
                 Assert.That(value, Is.EqualTo("Times"));
             }
 
-            // Commented out as this currently unexpectedly fails.
+//            // Commented out as this currently unexpectedly fails.
 //            // Test setting a key containing brackets around an integer with delineation
 //            {
 //                UUID storeId = (UUID)InvokeOp("JsonCreateStore", "{}"); 
@@ -607,7 +606,7 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore.Tests
 //                int result = (int)InvokeOp("JsonSetValue", storeId, "{Fun[0]Circus}", "Times");
 //                Assert.That(result, Is.EqualTo(1));
 //
-//                string value = (string)InvokeOp("JsonGetValue", storeId, "{Fun[]Circus}");
+//                string value = (string)InvokeOp("JsonGetValue", storeId, "{Fun[0]Circus}");
 //                Assert.That(value, Is.EqualTo("Times"));
 //            }
 
@@ -635,16 +634,13 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore.Tests
                 Assert.That(value, Is.EqualTo(""));
             }
 
-            // Commented out as this currently unexpectedly fails.
+//            // Commented out as this currently unexpectedly fails.
 //            // Test setting a key containing unbalanced }
 //            {
 //                UUID storeId = (UUID)InvokeOp("JsonCreateStore", "{}"); 
 //
 //                int result = (int)InvokeOp("JsonSetValue", storeId, "{Fun}Circus}", "Times");
-//                Assert.That(result, Is.EqualTo(1));
-//
-//                string value = (string)InvokeOp("JsonGetValue", storeId, "{Fun}Circus}");
-//                Assert.That(value, Is.EqualTo("Times"));
+//                Assert.That(result, Is.EqualTo(0));
 //            }
 
             // Test setting a key containing unbalanced { with delineation