Add tests

Modules/IO/Providers/DownloadProvider.cs

@@ -42,7 +42,7 @@ public DownloadProvider(IResource resourceProvider, string? fileName, FlexibleCo
 
         if (!request.HasType(RequestMethod.Get, RequestMethod.Head))
         {
-            throw new ProviderException(ResponseStatus.MethodNotAllowed, "Only GET requests are allowed by this handler", (b) => b.Header("Allow", "GET, HEAD"));
+            throw new ProviderException(ResponseStatus.MethodNotAllowed, "Only GET requests are allowed by this handler", (b) => b.Header("Allow", "GET"));
         }
 
         var response = request.Respond()

Testing/Acceptance/Modules/ErrorHandling/HtmlErrorMapperTest.cs

@@ -0,0 +1,75 @@
+using System.Net;
+using GenHTTP.Api.Content;
+using GenHTTP.Api.Protocol;
+using GenHTTP.Modules.ErrorHandling;
+using GenHTTP.Modules.Functional;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+namespace GenHTTP.Testing.Acceptance.Modules.ErrorHandling;
+
+[TestClass]
+public class HtmlErrorMapperTest
+{
+
+    [TestMethod]
+    [MultiEngineTest]
+    public async Task TestNotFound(TestEngine engine)
+    {
+        await using var host = await TestHost.RunAsync(Inline.Create().Add(ErrorHandler.Html()), engine: engine);
+
+        using var response = await host.GetResponseAsync();
+
+        await response.AssertStatusAsync(HttpStatusCode.NotFound);
+    }
+
+    [TestMethod]
+    [MultiEngineTest]
+    public async Task TestGeneralError(TestEngine engine)
+    {
+        var handler = Inline.Create()
+                            .Get(() => DoThrow(new Exception("Oops")))
+                            .Add(ErrorHandler.Html());
+
+        await using var host = await TestHost.RunAsync(handler, engine: engine);
+
+        using var response = await host.GetResponseAsync();
+
+        await response.AssertStatusAsync(HttpStatusCode.InternalServerError);
+    }
+
+    [TestMethod]
+    [MultiEngineTest]
+    public async Task TestProviderError(TestEngine engine)
+    {
+        var handler = Inline.Create()
+                            .Get(() => DoThrow(new ProviderException(ResponseStatus.Locked, "Locked up!")))
+                            .Add(ErrorHandler.Html());
+
+        await using var host = await TestHost.RunAsync(handler, engine: engine);
+
+        using var response = await host.GetResponseAsync();
+
+        await response.AssertStatusAsync(HttpStatusCode.Locked);
+    }
+
+    [TestMethod]
+    [MultiEngineTest]
+    public async Task TestNoTraceInProduction(TestEngine engine)
+    {
+        var handler = Inline.Create()
+                            .Get(() => DoThrow(new Exception("Oops")))
+                            .Add(ErrorHandler.Html());
+
+        await using var host = await TestHost.RunAsync(handler, development: false, engine: engine);
+
+        using var response = await host.GetResponseAsync();
+
+        await response.AssertStatusAsync(HttpStatusCode.InternalServerError);
+    }
+
+    private static void DoThrow(Exception e)
+    {
+        throw e;
+    }
+
+}

Testing/Acceptance/Modules/IO/DownloadTests.cs

@@ -54,6 +54,8 @@ public async Task DownloadsCannotBeModified(TestEngine engine)
         using var response = await runner.GetResponseAsync(request);
 
         await response.AssertStatusAsync(HttpStatusCode.MethodNotAllowed);
+
+        Assert.AreEqual("GET", response.GetContentHeader("Allow"));
     }
 
     [TestMethod]

Testing/Acceptance/Modules/ServerSentEvents/ProtocolTests.cs

@@ -105,6 +105,8 @@ public async Task TestGetOnly(TestEngine engine)
         using var response = await host.GetResponseAsync(request);
 
         await response.AssertStatusAsync(HttpStatusCode.MethodNotAllowed);
+
+        Assert.AreEqual("GET", response.GetContentHeader("Allow"));
     }
 
     [TestMethod]

Testing/Acceptance/Modules/Webservices/WebserviceTests.cs

@@ -201,7 +201,7 @@ public async Task TestUnsupportedDownloadEnforcesDefault(TestEngine engine)
     [MultiEngineTest]
     public async Task TestWrongMethod(TestEngine engine)
     {
-        await WithResponse(engine, "entity", HttpMethod.Put, "123", null, null, async r => { await r.AssertStatusAsync(HttpStatusCode.MethodNotAllowed); });
+        await WithResponse(engine, "entity", HttpMethod.Put, "123", null, null, async r => { await r.AssertStatusAsync(HttpStatusCode.MethodNotAllowed); Assert.AreEqual("POST", r.GetContentHeader("Allow")); });
     }
 
     [TestMethod]