diff options
| author | Justin Clark-Casey (justincc) | 2013-02-09 01:10:53 +0000 |
|---|---|---|
| committer | Justin Clark-Casey (justincc) | 2013-02-09 01:10:53 +0000 |
| commit | a8bc08ebe6523a2e66bb2e1d1d226d3159eba30c (patch) | |
| tree | 4bb6af75d83d9c7b42c390d3dc305ed37c55f957 /OpenSim/Region/OptionalModules/Scripting/JsonStore | |
| parent | Added physics parameters support to MSSQL and SQLite (not tested) (diff) | |
| download | opensim-SC-a8bc08ebe6523a2e66bb2e1d1d226d3159eba30c.zip opensim-SC-a8bc08ebe6523a2e66bb2e1d1d226d3159eba30c.tar.gz opensim-SC-a8bc08ebe6523a2e66bb2e1d1d226d3159eba30c.tar.bz2 opensim-SC-a8bc08ebe6523a2e66bb2e1d1d226d3159eba30c.tar.xz | |
Change TestDestroyStore() and TestJsonRemoveValue() to reflect the fact that the return values have changed.
Diffstat (limited to 'OpenSim/Region/OptionalModules/Scripting/JsonStore')
| -rw-r--r-- | OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs b/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs index ca88d1a..af97ac7 100644 --- a/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs +++ b/OpenSim/Region/OptionalModules/Scripting/JsonStore/Tests/JsonStoreScriptModuleTests.cs | |||
| @@ -144,8 +144,7 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore.Tests | |||
| 144 | 144 | ||
| 145 | int dsrv = (int)InvokeOp("JsonDestroyStore", fakeStoreId); | 145 | int dsrv = (int)InvokeOp("JsonDestroyStore", fakeStoreId); |
| 146 | 146 | ||
| 147 | // XXX: Current returns 'true' even though no such store existed. Need to ask if this is best behaviour. | 147 | Assert.That(dsrv, Is.EqualTo(0)); |
| 148 | Assert.That(dsrv, Is.EqualTo(1)); | ||
| 149 | } | 148 | } |
| 150 | 149 | ||
| 151 | [Test] | 150 | [Test] |
| @@ -211,9 +210,7 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore.Tests | |||
| 211 | 210 | ||
| 212 | // Test remove of non-existing value | 211 | // Test remove of non-existing value |
| 213 | int fakeValueRemove = (int)InvokeOp("JsonRemoveValue", storeId, "Hello"); | 212 | int fakeValueRemove = (int)InvokeOp("JsonRemoveValue", storeId, "Hello"); |
| 214 | 213 | Assert.That(fakeValueRemove, Is.EqualTo(0)); | |
| 215 | // XXX: Is this the best response to removing a value that isn't there? | ||
| 216 | Assert.That(fakeValueRemove, Is.EqualTo(1)); | ||
| 217 | 214 | ||
| 218 | // Test get from non-existing store | 215 | // Test get from non-existing store |
| 219 | UUID fakeStoreId = TestHelpers.ParseTail(0x500); | 216 | UUID fakeStoreId = TestHelpers.ParseTail(0x500); |