* Fix unit tests so that they correctly handle 404 missing asset response

* I didn't think there were tests in this area - my bad

3 files changed, 14 insertions, 9 deletions

diff --git a/OpenSim/Framework/Servers/Tests/CachedGetAssetStreamHandlerTests.cs b/OpenSim/Framework/Servers/Tests/CachedGetAssetStreamHandlerTests.cs
index cf19468..fed4707 100644
--- a/OpenSim/Framework/Servers/Tests/CachedGetAssetStreamHandlerTests.cs
+++ b/OpenSim/Framework/Servers/Tests/CachedGetAssetStreamHandlerTests.cs
@@ -94,9 +94,11 @@ namespace OpenSim.Framework.Servers.Tests
         public void TestHandleFetchMissingAsset()
         {
             IAssetCache assetCache = new TestAssetCache();
-            CachedGetAssetStreamHandler handler = new CachedGetAssetStreamHandler(assetCache);
+            CachedGetAssetStreamHandler handler;
+            OSHttpResponse response;
+            AssetBase asset = CreateTestEnvironment(out handler, out response);
 
-            GetAssetStreamHandlerTestHelpers.BaseFetchMissingAsset(handler);
+            GetAssetStreamHandlerTestHelpers.BaseFetchMissingAsset(handler, response);
         }
 
         [Test]
diff --git a/OpenSim/Framework/Servers/Tests/GetAssetStreamHandlerTests.cs b/OpenSim/Framework/Servers/Tests/GetAssetStreamHandlerTests.cs
index 7a05f93..6a4918d 100644
--- a/OpenSim/Framework/Servers/Tests/GetAssetStreamHandlerTests.cs
+++ b/OpenSim/Framework/Servers/Tests/GetAssetStreamHandlerTests.cs
@@ -94,10 +94,11 @@ namespace OpenSim.Framework.Servers.Tests
         [Test]
         public void TestHandleFetchMissingAsset()
         {
-            IAssetDataPlugin assetDataPlugin = new TestAssetDataPlugin();
-            GetAssetStreamHandler handler = new GetAssetStreamHandler(assetDataPlugin);
+            GetAssetStreamHandler handler;
+            OSHttpResponse response;
+            AssetBase asset = CreateTestEnvironment(out handler, out response);
 
-            GetAssetStreamHandlerTestHelpers.BaseFetchMissingAsset(handler);
+            GetAssetStreamHandlerTestHelpers.BaseFetchMissingAsset(handler, response);
         }
 
         [Test]
diff --git a/OpenSim/Tests/Common/Setup/GetAssetStreamHandlerTestHelpers.cs b/OpenSim/Tests/Common/Setup/GetAssetStreamHandlerTestHelpers.cs
index 138b690..d338f00 100644
--- a/OpenSim/Tests/Common/Setup/GetAssetStreamHandlerTestHelpers.cs
+++ b/OpenSim/Tests/Common/Setup/GetAssetStreamHandlerTestHelpers.cs
@@ -80,8 +80,7 @@ namespace OpenSim.Tests.Common.Setup
 
         public static void BaseFetchExistingAssetMetaDataTest(AssetBase asset, BaseGetAssetStreamHandler handler, OSHttpResponse response)
         {
-                                XmlSerializer xs =
-                            new XmlSerializer(typeof(AssetMetadata));
+            XmlSerializer xs = new XmlSerializer(typeof(AssetMetadata));
 
             byte[] expected = ServerUtils.SerializeResult(xs, asset.Metadata);
 
@@ -112,9 +111,12 @@ namespace OpenSim.Tests.Common.Setup
             return asset;
         }
 
-        public static void BaseFetchMissingAsset(BaseGetAssetStreamHandler handler)
+        public static void BaseFetchMissingAsset(BaseGetAssetStreamHandler handler, OSHttpResponse response)
         {
-            Assert.AreEqual(BaseRequestHandlerTestHelper.EmptyByteArray, handler.Handle("/assets/" + Guid.NewGuid(), null, null, null), "Failed on bad guid.");
+            Assert.AreEqual(
+                BaseRequestHandlerTestHelper.EmptyByteArray, 
+                handler.Handle("/assets/" + Guid.NewGuid(), null, null, response), "Failed on bad guid.");
+            Assert.AreEqual((int)HttpStatusCode.NotFound, response.StatusCode, "Response code wrong in BaseFetchMissingAsset");
         }
     }
 }