[PR #413] [MERGED] Fix new Delete with body missing from IWireMockAdminApi interface #901

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

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/413
Author: @NoahLerner
Created: 2/2/2020
Status: Merged
Merged: 2/5/2020
Merged by: @StefH

Base: masterHead: Fix_DeleteWithBodyMissingFromInterface


📝 Commits (10+)

  • 60e06f0 Fix new Delete with body missing from IWireMockAdminApi interface
  • c98878c Admin Delete with mappings in body (#409)
  • 2bff182 Add unit test for delete with body
  • d6dbd6e change order of checks for readability. looks nicer.
  • e4148e0 Allow body in DELETE requests
  • b60b544 Fix unit tests according to DELETE supporting body
  • fdbfe34 Re-run CI
  • 4869b30 Fix DELETE with body unit test
  • 15eb6a7 Fix ElementAt index in mappings list
  • 479c9c9 Fix DELETE with body unit test

📊 Changes

6 files changed (+131 additions, -10 deletions)

View changed files

📝 src/WireMock.Net.RestClient/IWireMockAdminApi.cs (+8 -0)
📝 src/WireMock.Net/Server/WireMockServer.Admin.cs (+56 -4)
📝 src/WireMock.Net/Util/BodyParser.cs (+1 -1)
📝 test/WireMock.Net.Tests/Util/BodyParserTests.cs (+1 -1)
📝 test/WireMock.Net.Tests/WireMockServer.Admin.cs (+62 -1)
📝 test/WireMock.Net.Tests/WireMockServerTests.cs (+3 -3)

📄 Description

As requested.


🔄 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/413 **Author:** [@NoahLerner](https://github.com/NoahLerner) **Created:** 2/2/2020 **Status:** ✅ Merged **Merged:** 2/5/2020 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `Fix_DeleteWithBodyMissingFromInterface` --- ### 📝 Commits (10+) - [`60e06f0`](https://github.com/wiremock/WireMock.Net/commit/60e06f0710bda3fc80e20c0a61070cc81418f8ee) Fix new Delete with body missing from IWireMockAdminApi interface - [`c98878c`](https://github.com/wiremock/WireMock.Net/commit/c98878c3cb2f471b37168359ce2784304538635f) Admin Delete with mappings in body (#409) - [`2bff182`](https://github.com/wiremock/WireMock.Net/commit/2bff1821f8816379303fe6c75244a8506870c4bc) Add unit test for delete with body - [`d6dbd6e`](https://github.com/wiremock/WireMock.Net/commit/d6dbd6ec6bab197e1eab7ab790b671c9f87a1020) change order of checks for readability. looks nicer. - [`e4148e0`](https://github.com/wiremock/WireMock.Net/commit/e4148e08ff11edc3ec5a3f39324a63b8c3a83b9e) Allow body in DELETE requests - [`b60b544`](https://github.com/wiremock/WireMock.Net/commit/b60b54421ec7798e5ce124a431d666221774f722) Fix unit tests according to DELETE supporting body - [`fdbfe34`](https://github.com/wiremock/WireMock.Net/commit/fdbfe34ef9c0194da42e34ec6711686c28edfccc) Re-run CI - [`4869b30`](https://github.com/wiremock/WireMock.Net/commit/4869b30c3fa9c9ec5703ee9a32ddfee7f9f9a3ae) Fix DELETE with body unit test - [`15eb6a7`](https://github.com/wiremock/WireMock.Net/commit/15eb6a79e8714061450f38f62a1ea39fd947843c) Fix ElementAt index in mappings list - [`479c9c9`](https://github.com/wiremock/WireMock.Net/commit/479c9c938f16bce77733d5aa89a3e27b802fd923) Fix DELETE with body unit test ### 📊 Changes **6 files changed** (+131 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `src/WireMock.Net.RestClient/IWireMockAdminApi.cs` (+8 -0) 📝 `src/WireMock.Net/Server/WireMockServer.Admin.cs` (+56 -4) 📝 `src/WireMock.Net/Util/BodyParser.cs` (+1 -1) 📝 `test/WireMock.Net.Tests/Util/BodyParserTests.cs` (+1 -1) 📝 `test/WireMock.Net.Tests/WireMockServer.Admin.cs` (+62 -1) 📝 `test/WireMock.Net.Tests/WireMockServerTests.cs` (+3 -3) </details> ### 📄 Description As requested. --- <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:34:06 +01:00
adam closed this issue 2025-12-29 15:34:06 +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#901