Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Massive tab and trailing space cleanup | Melanie Thielker | 2017-01-05 | 1 | -40/+40 |
| | |||||
* | refactor: consistently put all test classes in the OpenSim.Tests.Common ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -1/+0 |
| | | | | | | package rather than some in OpenSim.Tests.Common.Mock the separate mock package was not useful and was just another using line to always add | ||||
* | fix all instances of "non-existant" to "non-existent" (spelling mistakes) ↵ | Michael Cerquoni | 2014-07-17 | 1 | -3/+3 |
| | | | | thanks Ai Austin for pointing this out. | ||||
* | Per discussions with justincc... split the JsonStore type | Mic Bowman | 2013-03-05 | 1 | -25/+25 |
| | | | | | functions into one for node type and one for value type. Define and export constants for both nodes and values. | ||||
* | Convert JsonTestPath() use in json regression tests to JsonGetPathType() instead | Justin Clark-Casey (justincc) | 2013-02-19 | 1 | -10/+10 |
| | |||||
* | minor: Rename regression test method TestGetArrayLength() -> ↵ | Justin Clark-Casey (justincc) | 2013-02-18 | 1 | -1/+1 |
| | | | | JsonTestGetArrayLength() to match others | ||||
* | Enable one sub-test in TestJsonSetValue() which now works (using identifier ↵ | Justin Clark-Casey (justincc) | 2013-02-15 | 1 | -17/+13 |
| | | | | | | | with embedded .). Need to look further at other still commented tests. Still need to check coverage against some of Mic's scripts. | ||||
* | Rename JsonSetValueJson() -> JsonSetJson() and JsonGetValueJson() -> ↵ | Justin Clark-Casey (justincc) | 2013-02-15 | 1 | -13/+13 |
| | | | | | | | | JsonGetJson() This is because JsonGetJson() is getting json from anywhere in the structure, not just values. Equally, JsonSetJson() is setting any type of json, not just json which represents a value. Agreed with cmickeyb | ||||
* | Comment out regression TestJsonTestPath and TestJsonTestPathJson as these ↵ | Justin Clark-Casey (justincc) | 2013-02-15 | 1 | -62/+62 |
| | | | | will go away soon | ||||
* | Add regression TestJsonList2Path() | Justin Clark-Casey (justincc) | 2013-02-15 | 1 | -2/+27 |
| | |||||
* | Add regression TestGetArrayLength() | Justin Clark-Casey (justincc) | 2013-02-14 | 1 | -0/+33 |
| | |||||
* | Add regression TestJsonGetPathType() | Justin Clark-Casey (justincc) | 2013-02-14 | 1 | -0/+47 |
| | |||||
* | Add more sub-tests to TestJsonSetValue for paths containing []{} without {} ↵ | Justin Clark-Casey (justincc) | 2013-02-13 | 1 | -5/+86 |
| | | | | | | | delineation. As expected, values are not set and the set call returns FALSE (0). As a reminder, these tests are not currently running on jenkins continuous integration as the functionality is only available on .net 4 (mono 2.8 and later). | ||||
* | Extend JsonTestSetValue() with tests for escaping brackets, periods and ↵ | Justin Clark-Casey (justincc) | 2013-02-13 | 1 | -2/+71 |
| | | | | | | unbalanced braces from paths The sub-tests that are commented out are currently those which fail unexpectedly based on my understanding of the path syntax | ||||
* | Add test for array as root element in TestJsonCreateStore() | Justin Clark-Casey (justincc) | 2013-02-13 | 1 | -0/+9 |
| | |||||
* | Extend TestJsonRemoveValue() with tests for non-penultimate nodes and arrays | Justin Clark-Casey (justincc) | 2013-02-13 | 1 | -13/+57 |
| | |||||
* | Extend TestJsonCreateStore() with a one key input and an input with raw ↵ | Justin Clark-Casey (justincc) | 2013-02-12 | 1 | -2/+20 |
| | | | | number values | ||||
* | Re-enable subtest for single quoted token in TestJsonSetValueJson() | Justin Clark-Casey (justincc) | 2013-02-12 | 1 | -9/+9 |
| | | | | This is in response to the resolution of http://opensimulator.org/mantis/view.php?id=6540 | ||||
* | Add regression TestJsonSetValueJson() | Justin Clark-Casey (justincc) | 2013-02-12 | 1 | -0/+58 |
| | | | | | The part to test setting of single leaf-node string tokens is currently commented out. See http://opensimulator.org/mantis/view.php?id=6540 | ||||
* | Add regression TestJsonGetValueJson() | Justin Clark-Casey (justincc) | 2013-02-11 | 1 | -0/+33 |
| | |||||
* | Add regression TestJsonTestPathJson() | Justin Clark-Casey (justincc) | 2013-02-11 | 1 | -0/+32 |
| | |||||
* | Add section to TestJsonGetValue() to test call on a sub-tree | Justin Clark-Casey (justincc) | 2013-02-11 | 1 | -9/+21 |
| | |||||
* | Extend TestJsonTestPath() for non-terminating section of path (i.e. one that ↵ | Justin Clark-Casey (justincc) | 2013-02-11 | 1 | -8/+20 |
| | | | | does not point to a value/leaf) | ||||
* | Add section to TestJsonSetValue() to test attempted set of value where the ↵ | Justin Clark-Casey (justincc) | 2013-02-11 | 1 | -8/+23 |
| | | | | penultimate section of path does not exist | ||||
* | Add test to try reading notecard into an invalid path in ↵ | Justin Clark-Casey (justincc) | 2013-02-11 | 1 | -7/+13 |
| | | | | TestJsonReadNotecard() regression test | ||||
* | Change TestDestroyStore() and TestJsonRemoveValue() to reflect the fact that ↵ | Justin Clark-Casey (justincc) | 2013-02-09 | 1 | -5/+2 |
| | | | | the return values have changed. | ||||
* | Refine TestJsonReadNotecard() and use / instead of . to separate paths. | Justin Clark-Casey (justincc) | 2013-02-08 | 1 | -15/+46 |
| | | | | | | | An attack of the stupid meant that I was using / as a path separator. Fixing this makes the tests behave better, though still with some questions. Also, I imagine / shouldn't really put data in the root as that's not a valid identifier. This commit also fix the / mistake in other tests those this does not affect their outcomes. | ||||
* | Extend TestJsonReadNotecard() for reads to non-root locations and fake stores. | Justin Clark-Casey (justincc) | 2013-02-08 | 1 | -9/+33 |
| | | | | Assertions for loading to non-root paths are currently commented out because this doesn't seem to be working. Will be raising mantis to resolve. | ||||
* | Stop wrongly create a receiving store already populated with "Hello":"World" ↵ | Justin Clark-Casey (justincc) | 2013-02-08 | 1 | -1/+3 |
| | | | | in TestJsonReadNotecard() | ||||
* | Extend TestJsonWriteNotecard() regression test for cases with fake paths and ↵ | Justin Clark-Casey (justincc) | 2013-02-08 | 1 | -8/+61 |
| | | | | | | fake stores. Also separates out TestJsonWriteReadNotecard() into separate write and read tests | ||||
* | Extend TestJsonSetValue() regressio ntes tto test against a fake store | Justin Clark-Casey (justincc) | 2013-02-08 | 1 | -0/+5 |
| | |||||
* | Extend TestJsonTestPath() regression test with checks against fake values ↵ | Justin Clark-Casey (justincc) | 2013-02-08 | 1 | -0/+8 |
| | | | | and fake stores | ||||
* | Add tests for removing fake values/from fake store in TestJsonRemoveValue() | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -0/+11 |
| | | | | Again, need to check if returning true for removing a value that doesn't exist is most appropriate. | ||||
* | Add testing for getting non-existing values and values from a non-existing ↵ | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -1/+10 |
| | | | | datastore to TestJsonGetValue() | ||||
* | Add TestJsonDestoreStoreNotExists() | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -0/+14 |
| | | | | | This still returns true even if we ask to destroy a store that does not exist. Need to check that this is more appropriate behaviour. | ||||
* | Merge branch 'master' of ssh://opensimulator.org/var/git/opensim | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -1/+60 |
|\ | |||||
| * | Add TestJsonWriteReadNotecard() regression test | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -1/+44 |
| | | |||||
| * | Make json store tests operate on a single thread to ensure we don't run into ↵ | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -0/+16 |
| | | | | | | | | any race related test failures in the future. | ||||
* | | Fix a recent regression in e17392a where JsonSetValue() stopped working ↵ | Justin Clark-Casey (justincc) | 2013-02-07 | 1 | -4/+4 |
|/ | | | | | | | | | (probably other functions as well). Fix is to call through to the no-arg constructor from the string constructor in JsonStore, which I suspect was just forgotten. This was actually picked up by the TestJsonSetValue() regression test failing But this isn't being run on jenkins due to the .net version issue. This commit also puts the full stack trace in logged messages and makes these error level messages instead of info | ||||
* | Add JsonDestroyStore() basic regression test | Justin Clark-Casey (justincc) | 2013-01-30 | 1 | -0/+15 |
| | |||||
* | refactor: Make invocations of json store functions from the regression test ↵ | Justin Clark-Casey (justincc) | 2013-01-30 | 1 | -42/+17 |
| | | | | simpler | ||||
* | Add regression test for script func JsonRemoveValue() | Justin Clark-Casey (justincc) | 2013-01-30 | 1 | -0/+52 |
| | |||||
* | minor: change name of json script tests to JsonStoreScriptModuleTests ↵ | Justin Clark-Casey (justincc) | 2013-01-29 | 1 | -1/+1 |
| | | | | instead of copy/pasted LSL_ApiInventoryTests | ||||
* | Try ignoring json tests if they can't be run due to being on <=.net 3.5 | Justin Clark-Casey (justincc) | 2013-01-29 | 1 | -6/+17 |
| | |||||
* | Disable json tests for now - I see this is failing on jenkins because mono ↵ | Justin Clark-Casey (justincc) | 2013-01-25 | 1 | -4/+4 |
| | | | | | | 2.4.3 doesn't have Funcs >5 params, though mono 2.6 onwards does. Standardize logging in JsonStoreScriptModule and some minor code formatting. | ||||
* | Increase logging by enabling during test setup rather than during individual ↵ | Justin Clark-Casey (justincc) | 2013-01-25 | 1 | -4/+6 |
| | | | | tests of for JsonStore | ||||
* | Enable logging in new json store tests to find out why they fail on jenkins ↵ | Justin Clark-Casey (justincc) | 2013-01-25 | 1 | -4/+4 |
| | | | | but not locally. | ||||
* | Add JsonTestPath() regression test | Justin Clark-Casey (justincc) | 2013-01-25 | 1 | -0/+17 |
| | |||||
* | Add regression test for JsonSetValue() | Justin Clark-Casey (justincc) | 2013-01-25 | 1 | -0/+23 |
| | |||||
* | Add basic JsonGetValue() regression test. | Justin Clark-Casey (justincc) | 2013-01-25 | 1 | -0/+17 |
| |