Update the OpenApiPathsMapper to handle Value/Wildcard (#691)

src/WireMock.Net.OpenApiParser/Extensions/DictionaryExtensions.cs

@@ -0,0 +1,21 @@
+#if NET46 || NETSTANDARD2_0
+using System.Collections.Generic;
+
+namespace WireMock.Net.OpenApiParser.Extensions
+{
+    internal static class DictionaryExtensions
+    {
+        public static bool TryAdd<TKey, TValue>(this Dictionary<TKey, TValue> dictionary, TKey key, TValue value)
+        {
+            if (dictionary is null || dictionary.ContainsKey(key))
+            {
+                return false;
+            }
+
+            dictionary[key] = value;
+
+            return true;
+        }
+    }
+}
+#endif

src/WireMock.Net.OpenApiParser/Mappers/OpenApiPathsMapper.cs

@@ -19,6 +19,8 @@ namespace WireMock.Net.OpenApiParser.Mappers
 {
     internal class OpenApiPathsMapper
     {
+        private const string HeaderContentType = "Content-Type";
+
         private readonly WireMockOpenApiParserSettings _settings;
         private readonly ExampleValueGenerator _exampleValueGenerator;
 
@@ -248,6 +250,7 @@ namespace WireMock.Net.OpenApiParser.Mappers
                 return new JProperty(key, _exampleValueGenerator.GetExampleValue(openApiSchema));
             }
         }
+
         private string MapPathWithParameters(string path, IEnumerable<OpenApiParameter> parameters)
         {
             if (parameters == null)
@@ -258,7 +261,8 @@ namespace WireMock.Net.OpenApiParser.Mappers
             string newPath = path;
             foreach (var parameter in parameters)
             {
-                newPath = newPath.Replace($"{{{parameter.Name}}}", GetExampleValue(parameter.Schema, _settings.PathPatternToUse));
+                var exampleMatcherModel = GetExampleMatcherModel(parameter.Schema, _settings.PathPatternToUse);
+                newPath = newPath.Replace($"{{{parameter.Name}}}", exampleMatcherModel.Pattern as string);
             }
 
             return newPath;
@@ -305,19 +309,12 @@ namespace WireMock.Net.OpenApiParser.Mappers
         {
             var mappedHeaders = headers.ToDictionary(
                 item => item.Key,
-                item => GetExampleValue(null, _settings.HeaderPatternToUse) as object
+                item => GetExampleMatcherModel(null, _settings.HeaderPatternToUse).Pattern
             );
 
             if (!string.IsNullOrEmpty(responseContentType))
             {
-                if (!mappedHeaders.ContainsKey("Content-Type"))
-                {
-                    mappedHeaders.Add("Content-Type", responseContentType);
-                }
-                else
-                {
-                    mappedHeaders["Content-Type"] = responseContentType;
-                }
+                mappedHeaders.TryAdd(HeaderContentType, responseContentType);
             }
 
             return mappedHeaders.Keys.Any() ? mappedHeaders : null;
@@ -331,11 +328,7 @@ namespace WireMock.Net.OpenApiParser.Mappers
                     Name = qp.Name,
                     Matchers = new[]
                     {
-                        new MatcherModel
-                        {
-                            Name = "ExactMatcher",
-                            Pattern = GetDefaultValueAsStringForSchemaType(qp.Schema)
-                        }
+                        GetExampleMatcherModel(qp.Schema, _settings.QueryParameterPatternToUse)
                     }
                 })
                 .ToList();
@@ -351,11 +344,7 @@ namespace WireMock.Net.OpenApiParser.Mappers
                     Name = qp.Name,
                     Matchers = new[]
                     {
-                        new MatcherModel
-                        {
-                            Name = "ExactMatcher",
-                            Pattern = GetDefaultValueAsStringForSchemaType(qp.Schema)
-                        }
+                        GetExampleMatcherModel(qp.Schema, _settings.HeaderPatternToUse)
                     }
                 })
                 .ToList();
@@ -363,30 +352,26 @@ namespace WireMock.Net.OpenApiParser.Mappers
             return list.Any() ? list : null;
         }
 
-        private string GetDefaultValueAsStringForSchemaType(OpenApiSchema schema)
+        private MatcherModel GetExampleMatcherModel(OpenApiSchema schema, ExampleValueType type)
+        {
+            return type switch
+            {
+                ExampleValueType.Value => new MatcherModel { Name = "ExactMatcher", Pattern = GetExampleValueAsStringForSchemaType(schema) },
+
+                _ => new MatcherModel { Name = "WildcardMatcher", Pattern = "*" }
+            };
+        }
+
+        private string GetExampleValueAsStringForSchemaType(OpenApiSchema schema)
         {
             var value = _exampleValueGenerator.GetExampleValue(schema);
 
-            switch (value)
+            return value switch
             {
-                case string valueAsString:
-                    return valueAsString;
+                string valueAsString => valueAsString,
 
-                default:
-                    return value.ToString();
-            }
-        }
-
-        private string GetExampleValue(OpenApiSchema schema, ExampleValueType type)
-        {
-            switch (type)
-            {
-                case ExampleValueType.Value:
-                    return GetDefaultValueAsStringForSchemaType(schema);
-
-                default:
-                    return "*";
-            }
+                _ => value.ToString(),
+            };
         }
     }
 }

src/WireMock.Net.OpenApiParser/Settings/WireMockOpenApiParserSettings.cs

@@ -1,4 +1,4 @@
-using WireMock.Net.OpenApiParser.Types;
+using WireMock.Net.OpenApiParser.Types;
 
 namespace WireMock.Net.OpenApiParser.Settings
 {
@@ -22,10 +22,15 @@ namespace WireMock.Net.OpenApiParser.Settings
         /// </summary>
         public ExampleValueType HeaderPatternToUse { get; set; } = ExampleValueType.Value;
 
+        /// <summary>
+        /// The example value type to use when generating a Query Parameter
+        /// </summary>
+        public ExampleValueType QueryParameterPatternToUse { get; set; } = ExampleValueType.Value;
+
         /// <summary>
         /// The example values to use
         /// </summary>
-        public IWireMockOpenApiParserExampleValues ExampleValues { get; set; } = new WireMockOpenApiParserExampleValues();
+        public IWireMockOpenApiParserExampleValues ExampleValues { get; set; }
 
         /// <summary>
         /// Are examples generated dynamically?

src/WireMock.Net.OpenApiParser/Types/ExampleValueType.cs

@@ -1,4 +1,4 @@
-namespace WireMock.Net.OpenApiParser.Types
+namespace WireMock.Net.OpenApiParser.Types
 {
     /// <summary>
     /// The example value to use
@@ -6,7 +6,9 @@
     public enum ExampleValueType
     {
         /// <summary>
-        /// Use a generated example value based on the SchemaType (default).
+        /// 1. Use a generated example value based on the SchemaType (default).
+        /// 2. If there is no example value defined in the schema,
+        ///    then the <see cref="Settings.IWireMockOpenApiParserExampleValues"/> will be used (custom, fixed or dynamic).
         /// </summary>
         Value,
 

src/WireMock.Net.OpenApiParser/Utils/ExampleValueGenerator.cs

@@ -17,13 +17,17 @@ namespace WireMock.Net.OpenApiParser.Utils
         {
             _settings = Guard.NotNull(settings, nameof(settings));
 
-            if (_settings.DynamicExamples)
+            // Check if user provided an own implementation
+            if (settings.ExampleValues is null)
             {
-                _settings.ExampleValues = new WireMockOpenApiParserDynamicExampleValues();
-            }
-            else
-            {
-                _settings.ExampleValues = new WireMockOpenApiParserExampleValues();
+                if (_settings.DynamicExamples)
+                {
+                    _settings.ExampleValues = new WireMockOpenApiParserDynamicExampleValues();
+                }
+                else
+                {
+                    _settings.ExampleValues = new WireMockOpenApiParserExampleValues();
+                }
             }
         }