Remove functionality that controllers do not serialize data

Author: Kaliumhexacyanoferrat

Modules/Reflection/ResponseProvider.cs

@@ -94,30 +94,30 @@ public ResponseProvider(SerializationRegistry? serialization)
                 return downloadResponse;
             }
 
-            if (Serialization is not null)
+            // basic types should produce a string value
+            if (type == typeof(DateOnly))
             {
-                // basic types should produce a string value
-                if (type == typeof(DateOnly))
-                {
-                    var date = (DateOnly)result;
+                var date = (DateOnly)result;
 
-                    return request.Respond()
-                                  .Content(date.ToString("yyyy-MM-dd"))
-                                  .Type(ContentType.TextPlain)
-                                  .Adjust(adjustments)
-                                  .Build();
-                }
+                return request.Respond()
+                              .Content(date.ToString("yyyy-MM-dd"))
+                              .Type(ContentType.TextPlain)
+                              .Adjust(adjustments)
+                              .Build();
+            }
 
-                if (type.IsPrimitive || type == typeof(string) || type.IsEnum || type == typeof(Guid))
-                {
-                    return request.Respond()
-                                  .Content(result.ToString() ?? string.Empty)
-                                  .Type(ContentType.TextPlain)
-                                  .Adjust(adjustments)
-                                  .Build();
-                }
+            if (type.IsPrimitive || type == typeof(string) || type.IsEnum || type == typeof(Guid))
+            {
+                return request.Respond()
+                              .Content(result.ToString() ?? string.Empty)
+                              .Type(ContentType.TextPlain)
+                              .Adjust(adjustments)
+                              .Build();
+            }
 
-                // serialize the result
+            // serialize the result
+            if (Serialization is not null)
+            {
                 var serializer = Serialization.GetSerialization(request);
 
                 if (serializer is null)

Testing/Acceptance/Modules/Controllers/ErrorHandlingTests.cs

@@ -1,12 +1,8 @@
 using System;
-using System.Net;
-using System.Net.Http;
-using System.Threading.Tasks;
 
 using Microsoft.VisualStudio.TestTools.UnitTesting;
 
 using GenHTTP.Modules.Controllers;
-using GenHTTP.Modules.Layouting;
 
 namespace GenHTTP.Testing.Acceptance.Modules.Controllers
 {
@@ -17,13 +13,6 @@ public sealed class ErrorHandlingTests
 
         #region Supporting data structures
 
-        public sealed class TestController
-        {
-
-            public int SimpleType() => 42;
-
-        }
-
         public sealed class ControllerWithNullablePath
         {
 
@@ -44,14 +33,6 @@ public sealed class ControllerWithComplexPath
 
         #region Tests
 
-        [TestMethod]
-        public async Task TestMustNotReturnSimpleType()
-        {
-            using var response = await Run("/c/simple-type/");
-
-            await response.AssertStatusAsync(HttpStatusCode.InternalServerError);
-        }
-
         [TestMethod]
         public void TestNoNullablePathArguments()
         {
@@ -74,20 +55,6 @@ public void TestNoComplexPathArguments()
 
         #endregion
 
-        #region Helpers
-
-        private async Task<HttpResponseMessage> Run(string uri)
-        {
-            var layout = Layout.Create()
-                               .AddController<TestController>("c");
-
-            using var runner = TestHost.Run(layout);
-
-            return await runner.GetResponseAsync(uri);
-        }
-
-        #endregion
-
     }
 
 }