sc

WireMock-Net · Dec 16, 2023 · 3f370f3 · 3f370f3
1 parent 124959b
commit 3f370f3
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 31 deletions.
diff --git a/src/WireMock.Net/Server/IRespondWithAProvider.cs b/src/WireMock.Net/Server/IRespondWithAProvider.cs
@@ -26,13 +26,27 @@ public interface IRespondWithAProvider
     /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
     IRespondWithAProvider WithGuid(Guid guid);
 
+    /// <summary>
+    /// Define a unique identifier for this mapping.
+    /// </summary>
+    /// <param name="guid">The unique identifier.</param>
+    /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
+    IRespondWithAProvider WithGuid(string guid);
+
     /// <summary>
     /// Define a unique identifier for this mapping.
     /// </summary>
     /// <param name="guid">The unique identifier.</param>
     /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
     IRespondWithAProvider DefineGuid(Guid guid);
 
+    /// <summary>
+    /// Define a unique identifier for this mapping.
+    /// </summary>
+    /// <param name="guid">The unique identifier.</param>
+    /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
+    IRespondWithAProvider DefineGuid(string guid);
+
     /// <summary>
     /// Define the TimeSettings for this mapping.
     /// </summary>
@@ -61,20 +75,6 @@ public interface IRespondWithAProvider
     /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
     IRespondWithAProvider WithPath(string path);
 
-    /// <summary>
-    /// Define a unique identifier for this mapping.
-    /// </summary>
-    /// <param name="guid">The unique identifier.</param>
-    /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
-    IRespondWithAProvider WithGuid(string guid);
-
-    /// <summary>
-    /// Define a unique identifier for this mapping.
-    /// </summary>
-    /// <param name="guid">The unique identifier.</param>
-    /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
-    IRespondWithAProvider DefineGuid(string guid);
-
     /// <summary>
     /// Define the priority for this mapping.
     /// </summary>

diff --git a/src/WireMock.Net/Server/RespondWithAProvider.cs b/src/WireMock.Net/Server/RespondWithAProvider.cs
@@ -153,12 +153,6 @@ public IRespondWithAProvider WithGuid(string guid)
         return WithGuid(Guid.Parse(guid));
     }
 
-    /// <inheritdoc />
-    public IRespondWithAProvider DefineGuid(string guid)
-    {
-        return WithGuid(guid);
-    }
-
     /// <inheritdoc />
     public IRespondWithAProvider WithGuid(Guid guid)
     {
@@ -173,6 +167,12 @@ public IRespondWithAProvider DefineGuid(Guid guid)
         return WithGuid(guid);
     }
 
+    /// <inheritdoc />
+    public IRespondWithAProvider DefineGuid(string guid)
+    {
+        return WithGuid(guid);
+    }
+
     /// <inheritdoc />
     public IRespondWithAProvider WithTitle(string title)
     {
@@ -189,7 +189,7 @@ public IRespondWithAProvider WithDescription(string description)
         return this;
     }
 
-    /// <see cref="IRespondWithAProvider.WithPath"/>
+    /// <inheritdoc />
     public IRespondWithAProvider WithPath(string path)
     {
         _path = path;

diff --git a/src/WireMock.Net/Server/WireMockServer.Fluent.cs b/src/WireMock.Net/Server/WireMockServer.Fluent.cs
@@ -27,7 +27,7 @@ public IRespondWithAProvider Given(IRequestMatcher requestMatcher, bool saveToFi
     /// <param name="saveToFile">Optional boolean to indicate if this mapping should be saved as static mapping file.</param>
     /// <returns>The <see cref="IRespondWithAProvider"/>.</returns>
     [PublicAPI]
-    public IRespondWithAProvider WithRequest(Action<IRequestBuilder> action, bool saveToFile = false)
+    public IRespondWithAProvider WhenRequest(Action<IRequestBuilder> action, bool saveToFile = false)
     {
         Guard.NotNull(action);
 

diff --git a/test/WireMock.Net.Tests/WireMockServerTests.WithParam.cs b/test/WireMock.Net.Tests/WireMockServerTests.WithParam.cs
@@ -27,7 +27,7 @@ public async Task WireMockServer_WithParam_QueryParameterMultipleValueSupport_No
         };
         var server = WireMockServer.Start(settings);
         server
-            .WithRequest(r => r
+            .WhenRequest(r => r
                 .UsingGet()
                 .WithPath("/foo")
                 .WithParam("query", queryValue)
@@ -53,12 +53,12 @@ public async Task WireMockServer_WithParam_MultiValueComma()
         var queryValue = "1,2,3";
         var server = WireMockServer.Start();
         server
-            .WithRequest(r => r
+            .WhenRequest(r => r
                 .UsingGet()
                 .WithPath("/foo")
                 .WithParam("query", "1", "2", "3")
             )
-            .RespondWithOK();
+            .RespondWithStatusCode(200);
 
         // Act
         var requestUri = new Uri($"http://localhost:{server.Port}/foo?query={queryValue}");
@@ -81,9 +81,7 @@ public async Task WireMockServer_WithParam_RejectOnMatch_OnNonMatchingParam_Shou
                 .WithParam("delta_from", MatchBehaviour.RejectOnMatch)
                 .UsingGet()
             )
-            .RespondWith(
-                Response.Create()
-            );
+            .RespondWithOK();
 
         // Act
         var requestUri = new Uri($"http://localhost:{server.Port}/v1/person/workers?showsourcesystem=true&count=700&page=1&sections=personal%2Corganizations%2Cemployment");
@@ -106,9 +104,7 @@ public async Task WireMockServer_WithParam_AcceptOnMatch_OnNonMatchingParam_Shou
                 .WithParam("delta_from")
                 .UsingGet()
             )
-            .RespondWith(
-                Response.Create()
-            );
+            .RespondWithStatusCode("300");
 
         // Act
         var requestUri = new Uri($"http://localhost:{server.Port}/v1/person/workers?showsourcesystem=true&count=700&page=1&sections=personal%2Corganizations%2Cemployment");