[PR #67] [MERGED] bug: fix supporting the Patch method and logging the body #754

Closed
opened 2025-12-29 15:33:03 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/67
Author: @alastairtree
Created: 11/20/2017
Status: Merged
Merged: 11/20/2017
Merged by: @StefH

Base: masterHead: feat/support-patch


📝 Commits (3)

  • 87ac0af bug: fix supporting the Patch method and logging the body
  • d50fa68 chore: typo fixed
  • 658289e Added / reordered tests for PATCH method

📊 Changes

7 files changed (+182 additions, -144 deletions)

View changed files

📝 src/WireMock.Net/Client/IFluentMockServerAdmin.cs (+0 -1)
📝 src/WireMock.Net/Owin/OwinRequestMapper.cs (+2 -1)
📝 src/WireMock.Net/RequestBuilders/IMethodRequestBuilder.cs (+16 -8)
📝 src/WireMock.Net/RequestBuilders/Request.cs (+22 -47)
📝 test/WireMock.Net.Tests/FluentMockServerTests.cs (+28 -0)
test/WireMock.Net.Tests/RequestTests.PathAndMethod.cs (+113 -0)
📝 test/WireMock.Net.Tests/RequestTests.cs (+1 -87)

📄 Description

If you need to mock an API designed to the JsonAPI specification you need to work with PATCH requests with a http body. This fix and tests should allow support of PATCH with body and ensure the correct information gets logged.

  • Patch requests not configured to support Body
  • Add a unit test covering a patch request and logging of results

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/wiremock/WireMock.Net/pull/67 **Author:** [@alastairtree](https://github.com/alastairtree) **Created:** 11/20/2017 **Status:** ✅ Merged **Merged:** 11/20/2017 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `feat/support-patch` --- ### 📝 Commits (3) - [`87ac0af`](https://github.com/wiremock/WireMock.Net/commit/87ac0afb4751c80af469e0dff757ed3cee3008d0) bug: fix supporting the Patch method and logging the body - [`d50fa68`](https://github.com/wiremock/WireMock.Net/commit/d50fa68bd900a973cea7c0307f0e7ebef45bbb8e) chore: typo fixed - [`658289e`](https://github.com/wiremock/WireMock.Net/commit/658289e371af110b64758fdb6b0777358882dfbc) Added / reordered tests for PATCH method ### 📊 Changes **7 files changed** (+182 additions, -144 deletions) <details> <summary>View changed files</summary> 📝 `src/WireMock.Net/Client/IFluentMockServerAdmin.cs` (+0 -1) 📝 `src/WireMock.Net/Owin/OwinRequestMapper.cs` (+2 -1) 📝 `src/WireMock.Net/RequestBuilders/IMethodRequestBuilder.cs` (+16 -8) 📝 `src/WireMock.Net/RequestBuilders/Request.cs` (+22 -47) 📝 `test/WireMock.Net.Tests/FluentMockServerTests.cs` (+28 -0) ➕ `test/WireMock.Net.Tests/RequestTests.PathAndMethod.cs` (+113 -0) 📝 `test/WireMock.Net.Tests/RequestTests.cs` (+1 -87) </details> ### 📄 Description If you need to mock an API designed to the JsonAPI specification you need to work with PATCH requests with a http body. This fix and tests should allow support of PATCH with body and ensure the correct information gets logged. - Patch requests not configured to support Body - Add a unit test covering a patch request and logging of results --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 15:33:03 +01:00
adam closed this issue 2025-12-29 15:33:03 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/WireMock.Net-wiremock#754