[WIP] - Riok.Mapperly

CHANGELOG.md

@@ -1,13 +1,3 @@
-# 1.6.0 (16 August 2024)
-- [#1042](https://github.com/WireMock-Net/WireMock.Net/pull/1042) - Update + add fluent builder methods [feature] contributed by [StefH](https://github.com/StefH)
-- [#1109](https://github.com/WireMock-Net/WireMock.Net/pull/1109) - Add Aspire Extension [feature] contributed by [StefH](https://github.com/StefH)
-- [#1148](https://github.com/WireMock-Net/WireMock.Net/pull/1148) - Use Guid.TryParseExact with format "D" contributed by [StefH](https://github.com/StefH)
-- [#1157](https://github.com/WireMock-Net/WireMock.Net/pull/1157) - Fix FormUrlEncodedMatcher (MatchOperator.And) [bug] contributed by [StefH](https://github.com/StefH)
-- [#1158](https://github.com/WireMock-Net/WireMock.Net/pull/1158) - Allow setting Content-Length header on the response [feature] contributed by [StefH](https://github.com/StefH)
-- [#720](https://github.com/WireMock-Net/WireMock.Net/issues/720) - Response Header Content-Length not available when call HEAD Method [feature]
-- [#1145](https://github.com/WireMock-Net/WireMock.Net/issues/1145) - Response is auto converting string to guid [bug]
-- [#1156](https://github.com/WireMock-Net/WireMock.Net/issues/1156) - FormUrlEncodedMatcher is not requiring to match all properties when MatchOperator.And  [bug]
-
 # 1.5.62 (27 July 2024)
 - [#1147](https://github.com/WireMock-Net/WireMock.Net/pull/1147) - Add FormUrlEncodedMatcher [feature] contributed by [StefH](https://github.com/StefH)
 - [#1143](https://github.com/WireMock-Net/WireMock.Net/issues/1143) - FormEncoded Request fails (404 Not Found) if key value pairs order in mapping is different from request body order [bug]

Directory.Build.props

@@ -4,7 +4,7 @@
     </PropertyGroup>
 
     <PropertyGroup>
-        <VersionPrefix>1.6.0</VersionPrefix>
+        <VersionPrefix>1.5.62</VersionPrefix>
         <PackageIcon>WireMock.Net-Logo.png</PackageIcon>
         <PackageProjectUrl>https://github.com/WireMock-Net/WireMock.Net</PackageProjectUrl>
         <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>

Generate-ReleaseNotes.cmd

@@ -1,6 +1,6 @@
 rem https://github.com/StefH/GitHubReleaseNotes
 
-SET version=1.6.0
+SET version=1.5.62
 
 GitHubReleaseNotes --output CHANGELOG.md --skip-empty-releases --exclude-labels question invalid doc duplicate example environment --version %version% --token %GH_TOKEN%
 

PackageReleaseNotes.txt

@@ -1,11 +1,5 @@
-# 1.6.0 (16 August 2024)
-- #1042 Update + add fluent builder methods [feature]
-- #1109 Add Aspire Extension [feature]
-- #1148 Use Guid.TryParseExact with format "D"
-- #1157 Fix FormUrlEncodedMatcher (MatchOperator.And) [bug]
-- #1158 Allow setting Content-Length header on the response [feature]
-- #720 Response Header Content-Length not available when call HEAD Method [feature]
-- #1145 Response is auto converting string to guid [bug]
-- #1156 FormUrlEncodedMatcher is not requiring to match all properties when MatchOperator.And  [bug]
+# 1.5.62 (27 July 2024)
+- #1147 Add FormUrlEncodedMatcher [feature]
+- #1143 FormEncoded Request fails (404 Not Found) if key value pairs order in mapping is different from request body order [bug]
 
 The full release notes can be found here: https://github.com/WireMock-Net/WireMock.Net/blob/master/CHANGELOG.md

examples/WireMock.Net.Console.Net452.Classic/MainApp.cs

@@ -137,6 +137,7 @@ message HelloReply {
         public static void Run()
         {
             RunOnLocal();
+            return;
 
             var mappingBuilder = new MappingBuilder();
             mappingBuilder
@@ -307,6 +308,17 @@ message HelloReply {
                 .RespondWith(Response.Create()
                     .WithBody("GraphQL is ok")
                 );
+
+            //server
+            //    .AddGraphQLSchema("my-graphql", TestSchema, customScalars)
+            //    .Given(Request.Create()
+            //        .WithPath("/graphql2")
+            //        .UsingPost()
+            //    )
+            //    .WithGraphQLSchema("my-graphql")
+            //    .RespondWith(Response.Create()
+            //        .WithBody("GraphQL is ok")
+            //    );
 #endif
 
 #if MIMEKIT
@@ -365,15 +377,6 @@ message HelloReply {
                         .WithHeader("Content-Type", "text/plain")
                 );
 
-            server
-                .Given(Request.Create()
-                    .UsingHead()
-                    .WithPath("/cl")
-                )
-                .RespondWith(Response.Create()
-                    .WithHeader("Content-Length", "42")
-                );
-
             server
                 .Given(Request.Create()
                     .UsingMethod("GET")

src/WireMock.Net/Http/HttpKnownHeaderNames.cs

@@ -14,12 +14,12 @@ namespace WireMock.Http;
 /// </summary>
 internal static class HttpKnownHeaderNames
 {
-    // - https://docs.microsoft.com/en-us/dotnet/api/system.net.webheadercollection.isrestricted
-    // - ContentLength is allowed per #720
+    // https://docs.microsoft.com/en-us/dotnet/api/system.net.webheadercollection.isrestricted
     private static readonly string[] RestrictedResponseHeaders =
     {
         Accept,
         Connection,
+        ContentLength,
         ContentType,
         Date, // RFC1123Pattern
         Expect,

src/WireMock.Net/Owin/Mappers/OwinResponseMapper.cs

@@ -37,19 +37,12 @@ namespace WireMock.Owin.Mappers
         private readonly Encoding _utf8NoBom = new UTF8Encoding(false);
 
         // https://msdn.microsoft.com/en-us/library/78h415ay(v=vs.110).aspx
-        private static readonly IDictionary<string, Action<IResponse, bool, WireMockList<string>>> ResponseHeadersToFix =
-            new Dictionary<string, Action<IResponse, bool, WireMockList<string>>>(StringComparer.OrdinalIgnoreCase)
-            {
-                { HttpKnownHeaderNames.ContentType, (r, _, v) => r.ContentType = v.FirstOrDefault() },
-                { HttpKnownHeaderNames.ContentLength, (r, hasBody, v) =>
-                    {
-                        // Only set the Content-Length header if the response does not have a body
-                        if (!hasBody && long.TryParse(v.FirstOrDefault(), out var contentLength))
-                        {
-                            r.ContentLength = contentLength;
-                        }
-                    }
-                }
+#if !USE_ASPNETCORE
+        private static readonly IDictionary<string, Action<IResponse, WireMockList<string>>> ResponseHeadersToFix = new Dictionary<string, Action<IResponse, WireMockList<string>>>(StringComparer.OrdinalIgnoreCase) {
+#else
+        private static readonly IDictionary<string, Action<IResponse, WireMockList<string>>> ResponseHeadersToFix = new Dictionary<string, Action<IResponse, WireMockList<string>>>(StringComparer.OrdinalIgnoreCase) {
+#endif
+            { HttpKnownHeaderNames.ContentType, (r, v) => r.ContentType = v.FirstOrDefault() }
         };
 
         /// <summary>
@@ -90,21 +83,23 @@ namespace WireMock.Owin.Mappers
             }
 
             var statusCodeType = responseMessage.StatusCode?.GetType();
-            if (statusCodeType != null)
-            {
-                if (statusCodeType == typeof(int) || statusCodeType == typeof(int?) || statusCodeType.GetTypeInfo().IsEnum)
+            switch (statusCodeType)
             {
+                case { } when statusCodeType == typeof(int) || statusCodeType == typeof(int?) || statusCodeType.GetTypeInfo().IsEnum:
                     response.StatusCode = MapStatusCode((int)responseMessage.StatusCode!);
-                }
-                else if (statusCodeType == typeof(string))
-                {
+                    break;
+
+                case { } when statusCodeType == typeof(string):
                     // Note: this case will also match on null 
-                    int.TryParse(responseMessage.StatusCode as string, out var statusCodeTypeAsInt);
-                    response.StatusCode = MapStatusCode(statusCodeTypeAsInt);
-                }
+                    int.TryParse(responseMessage.StatusCode as string, out var result);
+                    response.StatusCode = MapStatusCode(result);
+                    break;
+
+                default:
+                    break;
             }
 
-            SetResponseHeaders(responseMessage, bytes, response);
+            SetResponseHeaders(responseMessage, response);
 
             if (bytes != null)
             {
@@ -165,7 +160,7 @@ namespace WireMock.Owin.Mappers
             return null;
         }
 
-        private static void SetResponseHeaders(IResponseMessage responseMessage, byte[]? bytes, IResponse response)
+        private static void SetResponseHeaders(IResponseMessage responseMessage, IResponse response)
         {
             // Force setting the Date header (#577)
             AppendResponseHeader(
@@ -184,11 +179,11 @@ namespace WireMock.Owin.Mappers
                 var value = item.Value;
                 if (ResponseHeadersToFix.TryGetValue(headerName, out var action))
                 {
-                    action?.Invoke(response, bytes != null, value);
+                    action?.Invoke(response, value);
                 }
                 else
                 {
-                    // Check if this response header can be added (#148, #227 and #720)
+                    // Check if this response header can be added (#148 and #227)
                     if (!HttpKnownHeaderNames.IsRestrictedResponseHeader(headerName))
                     {
                         AppendResponseHeader(response, headerName, value.ToArray());
@@ -211,7 +206,7 @@ namespace WireMock.Owin.Mappers
                 var value = item.Value;
                 if (ResponseHeadersToFix.TryGetValue(headerName, out var action))
                 {
-                    action?.Invoke(response, false, value);
+                    action?.Invoke(response, value);
                 }
                 else
                 {

src/WireMock.Net/Util/TinyMapperUtils.cs

@@ -1,56 +1,24 @@
 // Copyright © WireMock.Net
 
-using Nelibur.ObjectMapper;
+using Riok.Mapperly.Abstractions;
 using WireMock.Admin.Mappings;
 using WireMock.Admin.Settings;
 using WireMock.Settings;
 
 namespace WireMock.Util;
 
-internal sealed class TinyMapperUtils
+[Mapper]
+internal static partial class TinyMapperUtils
 {
-    public static TinyMapperUtils Instance { get; } = new();
+    public static partial ProxyAndRecordSettingsModel? Map(ProxyAndRecordSettings? instance);
 
-    private TinyMapperUtils()
-    {
-        TinyMapper.Bind<ProxyAndRecordSettings, ProxyAndRecordSettingsModel>();
-        TinyMapper.Bind<WebProxySettings, WebProxySettingsModel>();
-        TinyMapper.Bind<WebProxySettings, WebProxyModel>();
-        TinyMapper.Bind<ProxyUrlReplaceSettings, ProxyUrlReplaceSettingsModel>();
+    public static partial ProxyAndRecordSettings? Map(ProxyAndRecordSettingsModel? model);
 
-        TinyMapper.Bind<ProxyAndRecordSettingsModel, ProxyAndRecordSettings>();
-        TinyMapper.Bind<WebProxySettingsModel, WebProxySettings>();
-        TinyMapper.Bind<WebProxyModel, WebProxySettings>();
-        TinyMapper.Bind<ProxyUrlReplaceSettingsModel, ProxyUrlReplaceSettings>();
-    }
+    public static partial ProxyUrlReplaceSettingsModel? Map(ProxyUrlReplaceSettings? instance);
 
-    public ProxyAndRecordSettingsModel? Map(ProxyAndRecordSettings? instance)
-    {
-        return instance == null ? null : TinyMapper.Map<ProxyAndRecordSettingsModel>(instance);
-    }
+    public static partial ProxyUrlReplaceSettings? Map(ProxyUrlReplaceSettingsModel? model);
 
-    public ProxyAndRecordSettings? Map(ProxyAndRecordSettingsModel? model)
-    {
-        return model == null ? null : TinyMapper.Map<ProxyAndRecordSettings>(model);
-    }
+    public static partial WebProxyModel? Map(WebProxySettings? instance);
 
-    public ProxyUrlReplaceSettingsModel? Map(ProxyUrlReplaceSettings? instance)
-    {
-        return instance == null ? null : TinyMapper.Map<ProxyUrlReplaceSettingsModel>(instance);
-    }
-
-    public ProxyUrlReplaceSettings? Map(ProxyUrlReplaceSettingsModel? model)
-    {
-        return model == null ? null : TinyMapper.Map<ProxyUrlReplaceSettings>(model);
-    }
-
-    public WebProxyModel? Map(WebProxySettings? instance)
-    {
-        return instance == null ? null : TinyMapper.Map<WebProxyModel>(instance);
-    }
-
-    public WebProxySettings? Map(WebProxyModel? model)
-    {
-        return model == null ? null : TinyMapper.Map<WebProxySettings>(model);
-    }
+    public static partial WebProxySettings? Map(WebProxyModel? model);
 }

src/WireMock.Net/WireMock.Net.csproj

@@ -63,10 +63,10 @@
         <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
         <PackageReference Include="NJsonSchema.Extensions" Version="0.1.0" />
         <PackageReference Include="NSwag.Core" Version="13.16.1" />
+        <PackageReference Include="Riok.Mapperly" Version="3.6.0" />
         <PackageReference Include="SimMetrics.Net" Version="1.0.5" />
         <PackageReference Include="JmesPath.Net" Version="1.0.125" />
         <PackageReference Include="AnyOf" Version="0.3.0" />
-        <PackageReference Include="TinyMapper" Version="3.0.3" />
     </ItemGroup>
 
     <ItemGroup Condition=" '$(TargetFramework)' != 'netstandard1.3' ">

test/WireMock.Net.Tests/Owin/Mappers/OwinResponseMapperTests.cs

@@ -25,8 +25,8 @@ using Response = Microsoft.AspNetCore.Http.HttpResponse;
 using Microsoft.Extensions.Primitives;
 #endif
 
-namespace WireMock.Net.Tests.Owin.Mappers;
-
+namespace WireMock.Net.Tests.Owin.Mappers
+{
     public class OwinResponseMapperTests
     {
         private static readonly Task CompletedTask = Task.FromResult(true);
@@ -299,3 +299,4 @@ public class OwinResponseMapperTests
             _stream.Verify(s => s.WriteAsync(It.IsAny<byte[]>(), 0, It.Is<int>(count => count >= 0), It.IsAny<CancellationToken>()), Times.Once);
         }
     }
+}

test/WireMock.Net.Tests/WireMockServerTests.cs

@@ -14,7 +14,6 @@ using FluentAssertions;
 using Newtonsoft.Json;
 using NFluent;
 using WireMock.Admin.Mappings;
-using WireMock.Http;
 using WireMock.Matchers;
 using WireMock.Net.Tests.Serialization;
 using WireMock.Net.Xunit;
@@ -352,7 +351,7 @@ public partial class WireMockServerTests
     //}
 
     [Fact]
-    public async Task WireMockServer_Should_Exclude_RestrictedResponseHeader()
+    public async Task WireMockServer_Should_exclude_restrictedResponseHeader()
     {
         // Assign
         string path = $"/foo_{Guid.NewGuid()}";
@@ -372,26 +371,6 @@ public partial class WireMockServerTests
         server.Stop();
     }
 
-    [Fact] // #720
-    public async Task WireMockServer_Should_AllowResponseHeaderContentLength_For_HEAD()
-    {
-        // Assign
-        const string length = "42";
-        var path = $"/cl_{Guid.NewGuid()}";
-        using var server = WireMockServer.Start();
-
-        server
-            .Given(Request.Create().WithPath(path).UsingHead())
-            .RespondWith(Response.Create().WithHeader(HttpKnownHeaderNames.ContentLength, length));
-
-        // Act
-        var httpRequestMessage = new HttpRequestMessage(HttpMethod.Head, path);
-        var response = await server.CreateClient().SendAsync(httpRequestMessage).ConfigureAwait(false);
-
-        // Assert
-        response.Content.Headers.GetValues(HttpKnownHeaderNames.ContentLength).Should().Contain(length);
-    }
-
 #if !NET452 && !NET461
     [Theory]
     [InlineData("TRACE")]