.

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>
@@ -45,7 +45,7 @@
 
     <ItemGroup>
         <PackageReference Include="JetBrains.Annotations" Version="2023.3.0" PrivateAssets="All" />
-        <PackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0" PrivateAssets="All" />
+        <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1" PrivateAssets="All" />
     </ItemGroup>
 
     <ItemGroup Condition=" '$(TargetFramework)' == 'net8.0' ">

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)
+# 1.5.62 (27 July 2024)
-- #1042 Update + add fluent builder methods [feature]
+- #1147 Add FormUrlEncodedMatcher [feature]
-- #1109 Add Aspire Extension [feature]
+- #1143 FormEncoded Request fails (404 Not Found) if key value pairs order in mapping is different from request body order [bug]
-- #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]
 
 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.Aspire/WireMock.Net.Aspire.csproj

@@ -37,6 +37,7 @@
 
     <ItemGroup>
         <PackageReference Include="Aspire.Hosting" Version="8.0.0" />
+        <PackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0" PrivateAssets="All" />
     </ItemGroup>
 
     <ItemGroup>

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
+    // https://docs.microsoft.com/en-us/dotnet/api/system.net.webheadercollection.isrestricted
-    // - ContentLength is allowed per #720
     private static readonly string[] RestrictedResponseHeaders =
     {
         Accept,
         Connection,
+        ContentLength,
         ContentType,
         Date, // RFC1123Pattern
         Expect,

src/WireMock.Net/Matchers/FormUrlEncodedMatcher.cs

@@ -1,7 +1,6 @@
 // Copyright © WireMock.Net
 
 using System.Collections.Generic;
-using System.Linq;
 using AnyOfTypes;
 using Stef.Validation;
 using WireMock.Models;
@@ -28,7 +27,7 @@ public class FormUrlEncodedMatcher : IStringMatcher, IIgnoreCaseMatcher
     /// </summary>
     /// <param name="pattern">The pattern.</param>
     /// <param name="ignoreCase">Ignore the case from the pattern.</param>
-    /// <param name="matchOperator">The <see cref="MatchOperator"/> to use. (default = "Or")</param>
+    /// <param name="matchOperator">The <see cref="Matchers.MatchOperator"/> to use. (default = "Or")</param>
     public FormUrlEncodedMatcher(
         AnyOf<string, StringPattern> pattern,
         bool ignoreCase = false,
@@ -43,7 +42,7 @@ public class FormUrlEncodedMatcher : IStringMatcher, IIgnoreCaseMatcher
     /// <param name="matchBehaviour">The match behaviour.</param>
     /// <param name="pattern">The pattern.</param>
     /// <param name="ignoreCase">Ignore the case from the pattern.</param>
-    /// <param name="matchOperator">The <see cref="MatchOperator"/> to use. (default = "Or")</param>
+    /// <param name="matchOperator">The <see cref="Matchers.MatchOperator"/> to use. (default = "Or")</param>
     public FormUrlEncodedMatcher(
         MatchBehaviour matchBehaviour,
         AnyOf<string, StringPattern> pattern,
@@ -58,7 +57,7 @@ public class FormUrlEncodedMatcher : IStringMatcher, IIgnoreCaseMatcher
     /// </summary>
     /// <param name="patterns">The patterns.</param>
     /// <param name="ignoreCase">Ignore the case from the pattern.</param>
-    /// <param name="matchOperator">The <see cref="MatchOperator"/> to use. (default = "Or")</param>
+    /// <param name="matchOperator">The <see cref="Matchers.MatchOperator"/> to use. (default = "Or")</param>
     public FormUrlEncodedMatcher(
         AnyOf<string, StringPattern>[] patterns,
         bool ignoreCase = false,
@@ -73,7 +72,7 @@ public class FormUrlEncodedMatcher : IStringMatcher, IIgnoreCaseMatcher
     /// <param name="matchBehaviour">The match behaviour.</param>
     /// <param name="patterns">The patterns.</param>
     /// <param name="ignoreCase">Ignore the case from the pattern.</param>
-    /// <param name="matchOperator">The <see cref="MatchOperator"/> to use. (default = "Or")</param>
+    /// <param name="matchOperator">The <see cref="Matchers.MatchOperator"/> to use. (default = "Or")</param>
     public FormUrlEncodedMatcher(
         MatchBehaviour matchBehaviour,
         AnyOf<string, StringPattern>[] patterns,
@@ -113,20 +112,7 @@ public class FormUrlEncodedMatcher : IStringMatcher, IIgnoreCaseMatcher
             return new MatchResult(MatchScores.Mismatch);
         }
 
-        var matches = GetMatches(inputNameValueCollection);
-
-        var score = MatchScores.ToScore(matches, MatchOperator);
-        return new MatchResult(MatchBehaviourHelper.Convert(MatchBehaviour, score));
-    }
-
-    private bool[] GetMatches(IDictionary<string, string> inputNameValueCollection)
-    {
         var matches = new List<bool>();
-        if (_pairs.Count > inputNameValueCollection.Count)
-        {
-            matches.AddRange(Enumerable.Repeat(false, _pairs.Count - inputNameValueCollection.Count));
-        }
-
         foreach (var inputKeyValuePair in inputNameValueCollection)
         {
             var match = false;
@@ -146,7 +132,8 @@ public class FormUrlEncodedMatcher : IStringMatcher, IIgnoreCaseMatcher
             matches.Add(match);
         }
 
-        return matches.ToArray();
+        var score = MatchScores.ToScore(matches.ToArray(), MatchOperator);
+        return new MatchResult(MatchBehaviourHelper.Convert(MatchBehaviour, score));
     }
 
     /// <inheritdoc />

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

@@ -37,20 +37,13 @@ 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 =
+#if !USE_ASPNETCORE
-            new Dictionary<string, Action<IResponse, bool, WireMockList<string>>>(StringComparer.OrdinalIgnoreCase)
+        private static readonly IDictionary<string, Action<IResponse, WireMockList<string>>> ResponseHeadersToFix = new Dictionary<string, Action<IResponse, WireMockList<string>>>(StringComparer.OrdinalIgnoreCase) {
-            {
+#else
-                { HttpKnownHeaderNames.ContentType, (r, _, v) => r.ContentType = v.FirstOrDefault() },
+        private static readonly IDictionary<string, Action<IResponse, WireMockList<string>>> ResponseHeadersToFix = new Dictionary<string, Action<IResponse, WireMockList<string>>>(StringComparer.OrdinalIgnoreCase) {
-                { HttpKnownHeaderNames.ContentLength, (r, hasBody, v) =>
+#endif
-                    {
+            { HttpKnownHeaderNames.ContentType, (r, v) => r.ContentType = v.FirstOrDefault() }
-                        // 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;
-                        }
-                    }
-                }
-            };
 
         /// <summary>
         /// Constructor
@@ -90,21 +83,23 @@ namespace WireMock.Owin.Mappers
             }
 
             var statusCodeType = responseMessage.StatusCode?.GetType();
-            if (statusCodeType != null)
+            switch (statusCodeType)
             {
-                if (statusCodeType == typeof(int) || statusCodeType == typeof(int?) || statusCodeType.GetTypeInfo().IsEnum)
+                case { } when statusCodeType == typeof(int) || statusCodeType == typeof(int?) || statusCodeType.GetTypeInfo().IsEnum:
-                {
                     response.StatusCode = MapStatusCode((int)responseMessage.StatusCode!);
-                }
+                    break;
-                else if (statusCodeType == typeof(string))
+
-                {
+                case { } when statusCodeType == typeof(string):
                     // Note: this case will also match on null 
-                    int.TryParse(responseMessage.StatusCode as string, out var statusCodeTypeAsInt);
+                    int.TryParse(responseMessage.StatusCode as string, out var result);
-                    response.StatusCode = MapStatusCode(statusCodeTypeAsInt);
+                    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/Serialization/MappingFileNameSanitizer.cs

@@ -10,28 +10,22 @@ namespace WireMock.Serialization;
 /// <summary>
 /// Creates sanitized file names for mappings
 /// </summary>
-public class MappingFileNameSanitizer
+internal class MappingFileNameSanitizer
 {
+    private const string SpaceChar = " ";
     private const char ReplaceChar = '_';
 
-    private readonly WireMockServerSettings _settings;
-
-    public MappingFileNameSanitizer(WireMockServerSettings settings)
-    {
-        _settings = Guard.NotNull(settings);
-    }
-
     /// <summary>
     /// Creates sanitized file names for mappings
     /// </summary>
-    public string BuildSanitizedFileName(IMapping mapping)
+    public string BuildSanitizedFileName(IMapping mapping, ProxyAndRecordSettings? proxyAndRecordSettings)
     {
         string name;
         if (!string.IsNullOrEmpty(mapping.Title))
         {
             // remove 'Proxy Mapping for ' and an extra space character after the HTTP request method
-            name = mapping.Title.Replace(ProxyAndRecordSettings.DefaultPrefixForSavedMappingFile, "").Replace(' '.ToString(), string.Empty);
+            name = mapping.Title!.Replace(ProxyAndRecordSettings.DefaultPrefixForSavedMappingFile, string.Empty).Replace(SpaceChar, string.Empty);
-            if (_settings.ProxyAndRecordSettings?.AppendGuidToSavedMappingFile == true)
+            if (proxyAndRecordSettings?.AppendGuidToSavedMappingFile == true)
             {
                 name += $"{ReplaceChar}{mapping.Guid}";
             }
@@ -41,10 +35,11 @@ public class MappingFileNameSanitizer
             name = mapping.Guid.ToString();
         }
 
-        if (!string.IsNullOrEmpty(_settings.ProxyAndRecordSettings?.PrefixForSavedMappingFile))
+        if (!string.IsNullOrEmpty(proxyAndRecordSettings?.PrefixForSavedMappingFile))
         {
-            name = $"{_settings.ProxyAndRecordSettings.PrefixForSavedMappingFile}{ReplaceChar}{name}";
+            name = $"{proxyAndRecordSettings.PrefixForSavedMappingFile}{ReplaceChar}{name}";
         }
+
         return $"{Path.GetInvalidFileNameChars().Aggregate(name, (current, c) => current.Replace(c, ReplaceChar))}.json";
     }
 }

test/WireMock.Net.Tests/Matchers/FormUrlEncodedMatcherTests.cs

@@ -75,25 +75,4 @@ public class FormUrlEncodedMatcherTest
         // Assert
         score.Should().Be(expected);
     }
-
-    [Fact]
-    public async Task FormUrlEncodedMatcher_IsMatch_And_MatchAllProperties()
-    {
-        // Arrange
-        var content = new FormUrlEncodedContent(new[]
-        {
-            new KeyValuePair<string, string>("name", "John Doe"),
-            new KeyValuePair<string, string>("email", "johndoe@example.com")
-        });
-        var contentAsString = await content.ReadAsStringAsync();
-
-        // The expectation is that the matcher requires all properties to be present in the content.
-        var matcher = new FormUrlEncodedMatcher(["name=*", "email=*", "required=*"], matchOperator: MatchOperator.And);
-
-        // Act
-        var score = matcher.IsMatch(contentAsString).IsPerfect();
-
-        // Assert
-        score.Should().BeFalse();
-    }
 }

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

@@ -25,277 +25,278 @@ 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);
+    public class OwinResponseMapperTests
-    private readonly OwinResponseMapper _sut;
-    private readonly Mock<IResponse> _responseMock;
-    private readonly Mock<Stream> _stream;
-    private readonly Mock<IHeaderDictionary> _headers;
-    private readonly Mock<IFileSystemHandler> _fileSystemHandlerMock;
-    private readonly Mock<IWireMockMiddlewareOptions> _optionsMock;
-
-    public OwinResponseMapperTests()
     {
-        _stream = new Mock<Stream>();
+        private static readonly Task CompletedTask = Task.FromResult(true);
-        _stream.SetupAllProperties();
+        private readonly OwinResponseMapper _sut;
-        _stream.Setup(s => s.WriteAsync(It.IsAny<byte[]>(), It.IsAny<int>(), It.IsAny<int>(), It.IsAny<CancellationToken>())).Returns(CompletedTask);
+        private readonly Mock<IResponse> _responseMock;
+        private readonly Mock<Stream> _stream;
+        private readonly Mock<IHeaderDictionary> _headers;
+        private readonly Mock<IFileSystemHandler> _fileSystemHandlerMock;
+        private readonly Mock<IWireMockMiddlewareOptions> _optionsMock;
 
-        _fileSystemHandlerMock = new Mock<IFileSystemHandler>();
+        public OwinResponseMapperTests()
-        _fileSystemHandlerMock.SetupAllProperties();
+        {
+            _stream = new Mock<Stream>();
+            _stream.SetupAllProperties();
+            _stream.Setup(s => s.WriteAsync(It.IsAny<byte[]>(), It.IsAny<int>(), It.IsAny<int>(), It.IsAny<CancellationToken>())).Returns(CompletedTask);
 
-        _optionsMock = new Mock<IWireMockMiddlewareOptions>();
+            _fileSystemHandlerMock = new Mock<IFileSystemHandler>();
-        _optionsMock.SetupAllProperties();
+            _fileSystemHandlerMock.SetupAllProperties();
-        _optionsMock.SetupGet(o => o.FileSystemHandler).Returns(_fileSystemHandlerMock.Object);
 
-        _headers = new Mock<IHeaderDictionary>();
+            _optionsMock = new Mock<IWireMockMiddlewareOptions>();
-        _headers.SetupAllProperties();
+            _optionsMock.SetupAllProperties();
+            _optionsMock.SetupGet(o => o.FileSystemHandler).Returns(_fileSystemHandlerMock.Object);
+
+            _headers = new Mock<IHeaderDictionary>();
+            _headers.SetupAllProperties();
 #if NET452
-        _headers.Setup(h => h.AppendValues(It.IsAny<string>(), It.IsAny<string[]>()));
+            _headers.Setup(h => h.AppendValues(It.IsAny<string>(), It.IsAny<string[]>()));
 #else
-        _headers.Setup(h => h.Add(It.IsAny<string>(), It.IsAny<StringValues>()));
+            _headers.Setup(h => h.Add(It.IsAny<string>(), It.IsAny<StringValues>()));
 #endif
 
-        _responseMock = new Mock<IResponse>();
+            _responseMock = new Mock<IResponse>();
-        _responseMock.SetupAllProperties();
+            _responseMock.SetupAllProperties();
-        _responseMock.SetupGet(r => r.Body).Returns(_stream.Object);
+            _responseMock.SetupGet(r => r.Body).Returns(_stream.Object);
-        _responseMock.SetupGet(r => r.Headers).Returns(_headers.Object);
+            _responseMock.SetupGet(r => r.Headers).Returns(_headers.Object);
 
-        _sut = new OwinResponseMapper(_optionsMock.Object);
+            _sut = new OwinResponseMapper(_optionsMock.Object);
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public async Task OwinResponseMapper_MapAsync_Null()
+        public async Task OwinResponseMapper_MapAsync_Null()
-    {
-        // Act
-        await _sut.MapAsync(null, _responseMock.Object).ConfigureAwait(false);
-    }
-
-    [Theory]
-    [InlineData(300, 300)]
-    [InlineData(500, 500)]
-    public async Task OwinResponseMapper_MapAsync_Valid_StatusCode(object code, int expected)
-    {
-        // Arrange
-        var responseMessage = new ResponseMessage
         {
-            StatusCode = code
+            // Act
-        };
+            await _sut.MapAsync(null, _responseMock.Object).ConfigureAwait(false);
+        }
 
-        // Act
+        [Theory]
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+        [InlineData(300, 300)]
-
+        [InlineData(500, 500)]
-        // Assert
+        public async Task OwinResponseMapper_MapAsync_Valid_StatusCode(object code, int expected)
-        _responseMock.VerifySet(r => r.StatusCode = expected, Times.Once);
-    }
-
-    [Theory]
-    [InlineData(0, 200)]
-    [InlineData(-1, 200)]
-    [InlineData(10000, 200)]
-    [InlineData(300, 300)]
-    public async Task OwinResponseMapper_MapAsync_Invalid_StatusCode_When_AllowOnlyDefinedHttpStatusCodeInResponseSet_Is_True(object code, int expected)
-    {
-        // Arrange
-        _optionsMock.SetupGet(o => o.AllowOnlyDefinedHttpStatusCodeInResponse).Returns(true);
-        var responseMessage = new ResponseMessage
         {
-            StatusCode = code
+            // Arrange
-        };
+            var responseMessage = new ResponseMessage
+            {
+                StatusCode = code
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _responseMock.VerifySet(r => r.StatusCode = expected, Times.Once);
+            _responseMock.VerifySet(r => r.StatusCode = expected, Times.Once);
-    }
+        }
 
-    [Fact]
+        [Theory]
-    public async Task OwinResponseMapper_MapAsync_StatusCode_Is_Null()
+        [InlineData(0, 200)]
-    {
+        [InlineData(-1, 200)]
-        // Arrange
+        [InlineData(10000, 200)]
-        var responseMessage = new ResponseMessage
+        [InlineData(300, 300)]
+        public async Task OwinResponseMapper_MapAsync_Invalid_StatusCode_When_AllowOnlyDefinedHttpStatusCodeInResponseSet_Is_True(object code, int expected)
         {
-            StatusCode = null
+            // Arrange
-        };
+            _optionsMock.SetupGet(o => o.AllowOnlyDefinedHttpStatusCodeInResponse).Returns(true);
+            var responseMessage = new ResponseMessage
+            {
+                StatusCode = code
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _responseMock.VerifySet(r => r.StatusCode = It.IsAny<int>(), Times.Never);
+            _responseMock.VerifySet(r => r.StatusCode = expected, Times.Once);
-    }
+        }
 
-    [Theory]
+        [Fact]
-    [InlineData(0, 0)]
+        public async Task OwinResponseMapper_MapAsync_StatusCode_Is_Null()
-    [InlineData(-1, -1)]
-    [InlineData(10000, 10000)]
-    [InlineData(300, 300)]
-    public async Task OwinResponseMapper_MapAsync_StatusCode_Is_NotInEnumRange(object code, int expected)
-    {
-        // Arrange
-        var responseMessage = new ResponseMessage
         {
-            StatusCode = code
+            // Arrange
-        };
+            var responseMessage = new ResponseMessage
+            {
+                StatusCode = null
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _responseMock.VerifySet(r => r.StatusCode = expected, Times.Once);
+            _responseMock.VerifySet(r => r.StatusCode = It.IsAny<int>(), Times.Never);
-    }
+        }
 
-    [Fact]
+        [Theory]
-    public async Task OwinResponseMapper_MapAsync_NoBody()
+        [InlineData(0, 0)]
-    {
+        [InlineData(-1, -1)]
-        // Arrange
+        [InlineData(10000, 10000)]
-        var responseMessage = new ResponseMessage
+        [InlineData(300, 300)]
+        public async Task OwinResponseMapper_MapAsync_StatusCode_Is_NotInEnumRange(object code, int expected)
         {
-            Headers = new Dictionary<string, WireMockList<string>>()
+            // Arrange
-        };
+            var responseMessage = new ResponseMessage
+            {
+                StatusCode = code
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _stream.Verify(s => s.WriteAsync(It.IsAny<byte[]>(), It.IsAny<int>(), It.IsAny<int>(), It.IsAny<CancellationToken>()), Times.Never);
+            _responseMock.VerifySet(r => r.StatusCode = expected, Times.Once);
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public async Task OwinResponseMapper_MapAsync_Body()
+        public async Task OwinResponseMapper_MapAsync_NoBody()
-    {
-        // Arrange
-        string body = "abcd";
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>>(),
+            // Arrange
-            BodyData = new BodyData { DetectedBodyType = BodyType.String, BodyAsString = body }
+            var responseMessage = new ResponseMessage
-        };
+            {
+                Headers = new Dictionary<string, WireMockList<string>>()
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _stream.Verify(s => s.WriteAsync(new byte[] { 97, 98, 99, 100 }, 0, 4, It.IsAny<CancellationToken>()), Times.Once);
+            _stream.Verify(s => s.WriteAsync(It.IsAny<byte[]>(), It.IsAny<int>(), It.IsAny<int>(), It.IsAny<CancellationToken>()), Times.Never);
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public async Task OwinResponseMapper_MapAsync_BodyAsBytes()
+        public async Task OwinResponseMapper_MapAsync_Body()
-    {
-        // Arrange
-        var bytes = new byte[] { 48, 49 };
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>>(),
+            // Arrange
-            BodyData = new BodyData { DetectedBodyType = BodyType.Bytes, BodyAsBytes = bytes }
+            string body = "abcd";
-        };
+            var responseMessage = new ResponseMessage
+            {
+                Headers = new Dictionary<string, WireMockList<string>>(),
+                BodyData = new BodyData { DetectedBodyType = BodyType.String, BodyAsString = body }
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _stream.Verify(s => s.WriteAsync(bytes, 0, bytes.Length, It.IsAny<CancellationToken>()), Times.Once);
+            _stream.Verify(s => s.WriteAsync(new byte[] { 97, 98, 99, 100 }, 0, 4, It.IsAny<CancellationToken>()), Times.Once);
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public async Task OwinResponseMapper_MapAsync_BodyAsJson()
+        public async Task OwinResponseMapper_MapAsync_BodyAsBytes()
-    {
-        // Arrange
-        var json = new { t = "x", i = (string?)null };
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>>(),
+            // Arrange
-            BodyData = new BodyData { DetectedBodyType = BodyType.Json, BodyAsJson = json, BodyAsJsonIndented = false }
+            var bytes = new byte[] { 48, 49 };
-        };
+            var responseMessage = new ResponseMessage
+            {
+                Headers = new Dictionary<string, WireMockList<string>>(),
+                BodyData = new BodyData { DetectedBodyType = BodyType.Bytes, BodyAsBytes = bytes }
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _stream.Verify(s => s.WriteAsync(new byte[] { 123, 34, 116, 34, 58, 34, 120, 34, 125 }, 0, 9, It.IsAny<CancellationToken>()), Times.Once);
+            _stream.Verify(s => s.WriteAsync(bytes, 0, bytes.Length, It.IsAny<CancellationToken>()), Times.Once);
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public async Task OwinResponseMapper_MapAsync_SetResponseHeaders()
+        public async Task OwinResponseMapper_MapAsync_BodyAsJson()
-    {
-        // Arrange
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>> { { "h", new WireMockList<string>("x", "y") } }
+            // Arrange
-        };
+            var json = new { t = "x", i = (string?)null };
+            var responseMessage = new ResponseMessage
+            {
+                Headers = new Dictionary<string, WireMockList<string>>(),
+                BodyData = new BodyData { DetectedBodyType = BodyType.Json, BodyAsJson = json, BodyAsJsonIndented = false }
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
+            _stream.Verify(s => s.WriteAsync(new byte[] { 123, 34, 116, 34, 58, 34, 120, 34, 125 }, 0, 9, It.IsAny<CancellationToken>()), Times.Once);
+        }
+
+        [Fact]
+        public async Task OwinResponseMapper_MapAsync_SetResponseHeaders()
+        {
+            // Arrange
+            var responseMessage = new ResponseMessage
+            {
+                Headers = new Dictionary<string, WireMockList<string>> { { "h", new WireMockList<string>("x", "y") } }
+            };
+
+            // Act
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+
+            // Assert
 #if NET452
-        _headers.Verify(h => h.AppendValues("h", new string[] { "x", "y" }), Times.Once);
+            _headers.Verify(h => h.AppendValues("h", new string[] { "x", "y" }), Times.Once);
 #else
-        var v = new StringValues();
+            var v = new StringValues();
-        _headers.Verify(h => h.TryGetValue("h", out v), Times.Once);
+            _headers.Verify(h => h.TryGetValue("h", out v), Times.Once);
 #endif
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public void OwinResponseMapper_MapAsync_BodyAsFile_ThrowsException()
+        public void OwinResponseMapper_MapAsync_BodyAsFile_ThrowsException()
-    {
-        // Arrange
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>>(),
+            // Arrange
-            BodyData = new BodyData { DetectedBodyType = BodyType.File, BodyAsFile = string.Empty }
+            var responseMessage = new ResponseMessage
-        };
+            {
-        _fileSystemHandlerMock.Setup(f => f.ReadResponseBodyAsFile(It.IsAny<string>())).Throws<FileNotFoundException>();
+                Headers = new Dictionary<string, WireMockList<string>>(),
+                BodyData = new BodyData { DetectedBodyType = BodyType.File, BodyAsFile = string.Empty }
+            };
+            _fileSystemHandlerMock.Setup(f => f.ReadResponseBodyAsFile(It.IsAny<string>())).Throws<FileNotFoundException>();
 
-        // Act
+            // Act
-        Func<Task> action = () => _sut.MapAsync(responseMessage, _responseMock.Object);
+            Func<Task> action = () => _sut.MapAsync(responseMessage, _responseMock.Object);
 
-        // Assert
+            // Assert
-        action.Should().ThrowAsync<FileNotFoundException>();
+            action.Should().ThrowAsync<FileNotFoundException>();
-    }
+        }
 
-    [Fact]
+        [Fact]
-    public async Task OwinResponseMapper_MapAsync_WithFault_EMPTY_RESPONSE()
+        public async Task OwinResponseMapper_MapAsync_WithFault_EMPTY_RESPONSE()
-    {
-        // Arrange
-        string body = "abc";
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>>(),
+            // Arrange
-            BodyData = new BodyData { DetectedBodyType = BodyType.String, BodyAsString = body },
+            string body = "abc";
-            FaultType = FaultType.EMPTY_RESPONSE
+            var responseMessage = new ResponseMessage
-        };
+            {
+                Headers = new Dictionary<string, WireMockList<string>>(),
+                BodyData = new BodyData { DetectedBodyType = BodyType.String, BodyAsString = body },
+                FaultType = FaultType.EMPTY_RESPONSE
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _stream.Verify(s => s.WriteAsync(EmptyArray<byte>.Value, 0, 0, It.IsAny<CancellationToken>()), Times.Once);
+            _stream.Verify(s => s.WriteAsync(EmptyArray<byte>.Value, 0, 0, It.IsAny<CancellationToken>()), Times.Once);
-    }
+        }
 
-    [Theory]
+        [Theory]
-    [InlineData("abcd", BodyType.String)]
+        [InlineData("abcd", BodyType.String)]
-    [InlineData("", BodyType.String)]
+        [InlineData("", BodyType.String)]
-    [InlineData(null, BodyType.None)]
+        [InlineData(null, BodyType.None)]
-    public async Task OwinResponseMapper_MapAsync_WithFault_MALFORMED_RESPONSE_CHUNK(string body, BodyType detected)
+        public async Task OwinResponseMapper_MapAsync_WithFault_MALFORMED_RESPONSE_CHUNK(string body, BodyType detected)
-    {
-        // Arrange
-        var responseMessage = new ResponseMessage
         {
-            Headers = new Dictionary<string, WireMockList<string>>(),
+            // Arrange
-            BodyData = new BodyData { DetectedBodyType = detected, BodyAsString = body },
+            var responseMessage = new ResponseMessage
-            StatusCode = 100,
+            {
-            FaultType = FaultType.MALFORMED_RESPONSE_CHUNK
+                Headers = new Dictionary<string, WireMockList<string>>(),
-        };
+                BodyData = new BodyData { DetectedBodyType = detected, BodyAsString = body },
+                StatusCode = 100,
+                FaultType = FaultType.MALFORMED_RESPONSE_CHUNK
+            };
 
-        // Act
+            // Act
-        await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
+            await _sut.MapAsync(responseMessage, _responseMock.Object).ConfigureAwait(false);
 
-        // Assert
+            // Assert
-        _responseMock.VerifySet(r => r.StatusCode = 100, Times.Once);
+            _responseMock.VerifySet(r => r.StatusCode = 100, Times.Once);
-        _stream.Verify(s => s.WriteAsync(It.IsAny<byte[]>(), 0, It.Is<int>(count => count >= 0), It.IsAny<CancellationToken>()), Times.Once);
+            _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")]