diff --git a/src/WireMock.Net.Abstractions/IResponseMessage.cs b/src/WireMock.Net.Abstractions/IResponseMessage.cs index ebfd4b0b..416c339a 100644 --- a/src/WireMock.Net.Abstractions/IResponseMessage.cs +++ b/src/WireMock.Net.Abstractions/IResponseMessage.cs @@ -45,7 +45,18 @@ namespace WireMock /// object StatusCode { get; } - //void AddHeader(string name, params string[] values); - //void AddHeader(string name, string value); + /// + /// Adds the header. + /// + /// The name. + /// The value. + void AddHeader(string name, string value); + + /// + /// Adds the header. + /// + /// The name. + /// The values. + void AddHeader(string name, params string[] values); } } \ No newline at end of file diff --git a/src/WireMock.Net.Abstractions/Logging/ILogEntry.cs b/src/WireMock.Net.Abstractions/Logging/ILogEntry.cs index b6d17704..9e5559a6 100644 --- a/src/WireMock.Net.Abstractions/Logging/ILogEntry.cs +++ b/src/WireMock.Net.Abstractions/Logging/ILogEntry.cs @@ -3,78 +3,54 @@ using WireMock.Matchers.Request; namespace WireMock.Logging { + /// + /// ILogEntry + /// public interface ILogEntry { /// /// Gets the unique identifier. /// - /// - /// The unique identifier. - /// Guid Guid { get; } /// /// Gets the mapping unique identifier. /// - /// - /// The mapping unique identifier. - /// Guid? MappingGuid { get; } /// /// Gets the mapping unique title. /// - /// - /// The mapping unique title. - /// string MappingTitle { get; } /// /// Gets the partial mapping unique identifier. /// - /// - /// The mapping unique identifier. - /// Guid? PartialMappingGuid { get; } /// /// Gets the partial mapping unique title. /// - /// - /// The mapping unique title. - /// string PartialMappingTitle { get; } /// /// Gets the partial match result. /// - /// - /// The request match result. - /// IRequestMatchResult PartialMatchResult { get; } /// /// Gets the request match result. /// - /// - /// The request match result. - /// IRequestMatchResult RequestMatchResult { get; } /// /// Gets the request message. /// - /// - /// The request message. - /// IRequestMessage RequestMessage { get; } /// /// Gets the response message. /// - /// - /// The response message. - /// IResponseMessage ResponseMessage { get; } } } \ No newline at end of file diff --git a/src/WireMock.Net.Abstractions/Matchers/Request/MatchDetail.cs b/src/WireMock.Net.Abstractions/Matchers/Request/MatchDetail.cs index 1aa39c7a..31f987d7 100644 --- a/src/WireMock.Net.Abstractions/Matchers/Request/MatchDetail.cs +++ b/src/WireMock.Net.Abstractions/Matchers/Request/MatchDetail.cs @@ -2,10 +2,19 @@ namespace WireMock.Matchers.Request { + /// + /// MatchDetail + /// public class MatchDetail { + /// + /// Gets or sets the type of the matcher. + /// public Type MatcherType { get; set; } + /// + /// Gets or sets the score between 0.0 and 1.0 + /// public double Score { get; set; } } } \ No newline at end of file diff --git a/src/WireMock.Net/Owin/Mappers/OwinResponseMapper.cs b/src/WireMock.Net/Owin/Mappers/OwinResponseMapper.cs index 365d50bc..2d00a389 100644 --- a/src/WireMock.Net/Owin/Mappers/OwinResponseMapper.cs +++ b/src/WireMock.Net/Owin/Mappers/OwinResponseMapper.cs @@ -7,8 +7,6 @@ using System.Threading.Tasks; using Newtonsoft.Json; using RandomDataGenerator.FieldOptions; using RandomDataGenerator.Randomizers; -using WireMock.Exceptions; -using WireMock.Handlers; using WireMock.Http; using WireMock.ResponseBuilders; using WireMock.Types; diff --git a/src/WireMock.Net/Owin/WireMockMiddleware.cs b/src/WireMock.Net/Owin/WireMockMiddleware.cs index 2e02eba1..5047306c 100644 --- a/src/WireMock.Net/Owin/WireMockMiddleware.cs +++ b/src/WireMock.Net/Owin/WireMockMiddleware.cs @@ -2,9 +2,7 @@ using System; using System.Threading.Tasks; using WireMock.Logging; using System.Linq; -using System.Text.RegularExpressions; using WireMock.Matchers; -using Newtonsoft.Json; using WireMock.Http; using WireMock.Owin.Mappers; using WireMock.Serialization; diff --git a/src/WireMock.Net/Owin/WireMockMiddlewareOptions.cs b/src/WireMock.Net/Owin/WireMockMiddlewareOptions.cs index c213b4fd..e6b1b138 100644 --- a/src/WireMock.Net/Owin/WireMockMiddlewareOptions.cs +++ b/src/WireMock.Net/Owin/WireMockMiddlewareOptions.cs @@ -1,6 +1,5 @@ using System; using System.Collections.Concurrent; -using System.Collections.ObjectModel; using WireMock.Handlers; using WireMock.Logging; using WireMock.Matchers; diff --git a/src/WireMock.Net/ResponseBuilders/Response.cs b/src/WireMock.Net/ResponseBuilders/Response.cs index c791969a..ee6715b9 100644 --- a/src/WireMock.Net/ResponseBuilders/Response.cs +++ b/src/WireMock.Net/ResponseBuilders/Response.cs @@ -60,7 +60,7 @@ namespace WireMock.ResponseBuilders [PublicAPI] public static IResponseBuilder Create([CanBeNull] ResponseMessage responseMessage = null) { - var message = responseMessage ?? new ResponseMessage(); // { StatusCode = (int)HttpStatusCode.OK }; + var message = responseMessage ?? new ResponseMessage(); return new Response(message); } diff --git a/src/WireMock.Net/ResponseMessage.cs b/src/WireMock.Net/ResponseMessage.cs index 0c35a25d..0e68c7dc 100644 --- a/src/WireMock.Net/ResponseMessage.cs +++ b/src/WireMock.Net/ResponseMessage.cs @@ -35,21 +35,13 @@ namespace WireMock /// public double? FaultPercentage { get; set; } - /// - /// Adds the header. - /// - /// The name. - /// The value. + /// public void AddHeader(string name, string value) { Headers.Add(name, new WireMockList(value)); } - /// - /// Adds the header. - /// - /// The name. - /// The values. + /// public void AddHeader(string name, params string[] values) { Check.NotNullOrEmpty(values, nameof(values)); diff --git a/src/WireMock.Net/Serialization/MatcherMapper.cs b/src/WireMock.Net/Serialization/MatcherMapper.cs index 50c4bc56..86320fa4 100644 --- a/src/WireMock.Net/Serialization/MatcherMapper.cs +++ b/src/WireMock.Net/Serialization/MatcherMapper.cs @@ -95,21 +95,6 @@ namespace WireMock.Serialization } } - private ExactObjectMatcher CreateExactObjectMatcher(MatchBehaviour matchBehaviour, string stringPattern, bool throwException) - { - byte[] bytePattern; - try - { - bytePattern = Convert.FromBase64String(stringPattern); - } - catch - { - throw new ArgumentException($"Matcher 'ExactObjectMatcher' has invalid pattern. The pattern value '{stringPattern}' is not a Base64String.", nameof(stringPattern)); - } - - return new ExactObjectMatcher(matchBehaviour, bytePattern, throwException); - } - public MatcherModel[] Map([CanBeNull] IEnumerable matchers) { return matchers?.Select(Map).Where(m => m != null).ToArray(); @@ -154,5 +139,20 @@ namespace WireMock.Serialization Patterns = patterns.Length > 1 ? patterns : null }; } + + private ExactObjectMatcher CreateExactObjectMatcher(MatchBehaviour matchBehaviour, string stringPattern, bool throwException) + { + byte[] bytePattern; + try + { + bytePattern = Convert.FromBase64String(stringPattern); + } + catch + { + throw new ArgumentException($"Matcher 'ExactObjectMatcher' has invalid pattern. The pattern value '{stringPattern}' is not a Base64String.", nameof(stringPattern)); + } + + return new ExactObjectMatcher(matchBehaviour, bytePattern, throwException); + } } } \ No newline at end of file diff --git a/src/WireMock.Net/Server/WireMockServer.Admin.cs b/src/WireMock.Net/Server/WireMockServer.Admin.cs index 1a9d78ce..eb23b4a0 100644 --- a/src/WireMock.Net/Server/WireMockServer.Admin.cs +++ b/src/WireMock.Net/Server/WireMockServer.Admin.cs @@ -833,9 +833,9 @@ namespace WireMock.Server if (requestModel.Params != null) { - foreach (var paramModel in requestModel.Params.Where(c => c.Matchers != null)) + foreach (var paramModel in requestModel.Params.Where(p => p != null && p.Matchers != null)) { - bool ignoreCase = paramModel?.IgnoreCase ?? false; + bool ignoreCase = paramModel.IgnoreCase == true; requestBuilder = requestBuilder.WithParam(paramModel.Name, ignoreCase, paramModel.Matchers.Select(_matcherMapper.Map).OfType().ToArray()); } } diff --git a/src/WireMock.Net/Server/WireMockServer.cs b/src/WireMock.Net/Server/WireMockServer.cs index ffb87834..8702c338 100644 --- a/src/WireMock.Net/Server/WireMockServer.cs +++ b/src/WireMock.Net/Server/WireMockServer.cs @@ -184,6 +184,17 @@ namespace WireMock.Server ReadStaticMappings = true }); } + + /// + /// Initializes a new instance of the class. + /// + /// The settings. + /// + /// Service start failed with error: {_httpServer.RunningException.Message} + /// or + /// Service start failed with error: {startTask.Exception.Message} + /// + /// Service start timed out after {TimeSpan.FromMilliseconds(settings.StartTimeout)} protected WireMockServer(IWireMockServerSettings settings) { _settings = settings; @@ -422,7 +433,7 @@ namespace WireMock.Server { _options.Scenarios.Clear(); } - + /// [PublicAPI] public IWireMockServer WithMapping(params MappingModel[] mappings) diff --git a/src/WireMock.Net/Settings/IWebProxySettings.cs b/src/WireMock.Net/Settings/IWebProxySettings.cs index 3cdbe172..4c25e3f2 100644 --- a/src/WireMock.Net/Settings/IWebProxySettings.cs +++ b/src/WireMock.Net/Settings/IWebProxySettings.cs @@ -1,5 +1,8 @@ namespace WireMock.Settings { + /// + /// IWebProxySettings + /// public interface IWebProxySettings { /// diff --git a/src/WireMock.Net/Settings/WebProxySettings.cs b/src/WireMock.Net/Settings/WebProxySettings.cs index 1de8b380..96cf1b04 100644 --- a/src/WireMock.Net/Settings/WebProxySettings.cs +++ b/src/WireMock.Net/Settings/WebProxySettings.cs @@ -2,6 +2,7 @@ namespace WireMock.Settings { + /// public class WebProxySettings : IWebProxySettings { /// diff --git a/src/WireMock.Net/Transformers/IHandlebarsContextFactory.cs b/src/WireMock.Net/Transformers/IHandlebarsContextFactory.cs index c69372ab..73b730ff 100644 --- a/src/WireMock.Net/Transformers/IHandlebarsContextFactory.cs +++ b/src/WireMock.Net/Transformers/IHandlebarsContextFactory.cs @@ -1,9 +1,7 @@ -using HandlebarsDotNet; - -namespace WireMock.Transformers +namespace WireMock.Transformers { interface IHandlebarsContextFactory { IHandlebarsContext Create(); } -} +} \ No newline at end of file diff --git a/src/WireMock.Net/Validation/Check.cs b/src/WireMock.Net/Validation/Check.cs index bd42dd22..8aebecc0 100644 --- a/src/WireMock.Net/Validation/Check.cs +++ b/src/WireMock.Net/Validation/Check.cs @@ -5,7 +5,6 @@ using System; using System.Collections.Generic; using System.Diagnostics; using System.Linq; -using System.Reflection; using JetBrains.Annotations; // Copied from https://github.com/aspnet/EntityFramework/blob/dev/src/Shared/Check.cs @@ -96,19 +95,6 @@ namespace WireMock.Validation return value; } - //public static string NullButNotEmpty(string value, [InvokerParameterName, ValidatedNotNull, NotNull] string parameterName) - //{ - // if (!ReferenceEquals(value, null) - // && (value.Length == 0)) - // { - // NotNullOrEmpty(parameterName, nameof(parameterName)); - - // throw new ArgumentException(CoreStrings.ArgumentIsEmpty(parameterName)); - // } - - // return value; - //} - public static IList HasNoNulls(IList value, [InvokerParameterName, ValidatedNotNull, NotNull] string parameterName) where T : class { @@ -123,17 +109,5 @@ namespace WireMock.Validation return value; } - - //public static Type ValidEntityType(Type value, [InvokerParameterName, ValidatedNotNull, NotNull] string parameterName) - //{ - // if (!value.GetTypeInfo().IsClass) - // { - // NotNullOrEmpty(parameterName, nameof(parameterName)); - - // throw new ArgumentException(CoreStrings.InvalidEntityType(value, parameterName)); - // } - - // return value; - //} } } \ No newline at end of file