diff options
author | Justin Clark-Casey (justincc) | 2012-03-07 00:31:18 +0000 |
---|---|---|
committer | Justin Clark-Casey (justincc) | 2012-03-07 00:31:18 +0000 |
commit | f3678d217f7b1d69faf4aaeb0097348f3d7f91b6 (patch) | |
tree | 8b011b384366a99b7759f230abed9c4c26a4fbf9 /OpenSim/Tests/Torture | |
parent | Add documentation to make more explicit the difference between OnRezScript an... (diff) | |
download | opensim-SC-f3678d217f7b1d69faf4aaeb0097348f3d7f91b6.zip opensim-SC-f3678d217f7b1d69faf4aaeb0097348f3d7f91b6.tar.gz opensim-SC-f3678d217f7b1d69faf4aaeb0097348f3d7f91b6.tar.bz2 opensim-SC-f3678d217f7b1d69faf4aaeb0097348f3d7f91b6.tar.xz |
Stop individually deleting objects at the end of each ObjectTortureTest.
We can now do this since the entire scene and all objects within it are now successfully gc'd at the end of these tests.
This greatly improves the time taken to run each test (by reducing teardown time, not the time to actually do the test work that we're interested in).
Slightly simplifies config read in Scene constructor to help facilitate this.
Diffstat (limited to 'OpenSim/Tests/Torture')
-rw-r--r-- | OpenSim/Tests/Torture/NPCTortureTests.cs | 4 | ||||
-rw-r--r-- | OpenSim/Tests/Torture/ObjectTortureTests.cs | 8 | ||||
-rw-r--r-- | OpenSim/Tests/Torture/ScriptTortureTests.cs | 8 |
3 files changed, 14 insertions, 6 deletions
diff --git a/OpenSim/Tests/Torture/NPCTortureTests.cs b/OpenSim/Tests/Torture/NPCTortureTests.cs index 8078d9d..65732db 100644 --- a/OpenSim/Tests/Torture/NPCTortureTests.cs +++ b/OpenSim/Tests/Torture/NPCTortureTests.cs | |||
@@ -75,6 +75,10 @@ namespace OpenSim.Tests.Torture | |||
75 | [TestFixtureTearDown] | 75 | [TestFixtureTearDown] |
76 | public void TearDown() | 76 | public void TearDown() |
77 | { | 77 | { |
78 | scene.Close(); | ||
79 | GC.Collect(); | ||
80 | GC.WaitForPendingFinalizers(); | ||
81 | |||
78 | // We must set this back afterwards, otherwise later tests will fail since they're expecting multiple | 82 | // We must set this back afterwards, otherwise later tests will fail since they're expecting multiple |
79 | // threads. Possibly, later tests should be rewritten not to worry about such things. | 83 | // threads. Possibly, later tests should be rewritten not to worry about such things. |
80 | Util.FireAndForgetMethod = Util.DefaultFireAndForgetMethod; | 84 | Util.FireAndForgetMethod = Util.DefaultFireAndForgetMethod; |
diff --git a/OpenSim/Tests/Torture/ObjectTortureTests.cs b/OpenSim/Tests/Torture/ObjectTortureTests.cs index e83186a..7e9946b 100644 --- a/OpenSim/Tests/Torture/ObjectTortureTests.cs +++ b/OpenSim/Tests/Torture/ObjectTortureTests.cs | |||
@@ -156,11 +156,6 @@ namespace OpenSim.Tests.Torture | |||
156 | // objects will be clean up by the garbage collector before the next stress test is run. | 156 | // objects will be clean up by the garbage collector before the next stress test is run. |
157 | scene.Update(); | 157 | scene.Update(); |
158 | 158 | ||
159 | // Currently, we need to do this in order to garbage collect the scene objects ready for the next test run. | ||
160 | // However, what we really need to do is find out why the entire scene is not garbage collected in | ||
161 | // teardown. | ||
162 | scene.DeleteAllSceneObjects(); | ||
163 | |||
164 | Console.WriteLine( | 159 | Console.WriteLine( |
165 | "Took {0}ms, {1}MB ({2} - {3}) to create {4} objects each containing {5} prim(s)", | 160 | "Took {0}ms, {1}MB ({2} - {3}) to create {4} objects each containing {5} prim(s)", |
166 | Math.Round(elapsed.TotalMilliseconds), | 161 | Math.Round(elapsed.TotalMilliseconds), |
@@ -170,7 +165,8 @@ namespace OpenSim.Tests.Torture | |||
170 | objectsToAdd, | 165 | objectsToAdd, |
171 | primsInEachObject); | 166 | primsInEachObject); |
172 | 167 | ||
173 | scene = null; | 168 | scene.Close(); |
169 | // scene = null; | ||
174 | } | 170 | } |
175 | } | 171 | } |
176 | } \ No newline at end of file | 172 | } \ No newline at end of file |
diff --git a/OpenSim/Tests/Torture/ScriptTortureTests.cs b/OpenSim/Tests/Torture/ScriptTortureTests.cs index d94bbde..87932cb 100644 --- a/OpenSim/Tests/Torture/ScriptTortureTests.cs +++ b/OpenSim/Tests/Torture/ScriptTortureTests.cs | |||
@@ -91,6 +91,14 @@ namespace OpenSim.Tests.Torture | |||
91 | m_scene.StartScripts(); | 91 | m_scene.StartScripts(); |
92 | } | 92 | } |
93 | 93 | ||
94 | [TearDown] | ||
95 | public void TearDown() | ||
96 | { | ||
97 | m_scene.Close(); | ||
98 | GC.Collect(); | ||
99 | GC.WaitForPendingFinalizers(); | ||
100 | } | ||
101 | |||
94 | [Test] | 102 | [Test] |
95 | public void TestCompileAndStart100Scripts() | 103 | public void TestCompileAndStart100Scripts() |
96 | { | 104 | { |