[PR #1248] [MERGED] Add exception message to logging when mapping fails due to an exception. #1299

Closed
opened 2025-12-29 09:19:13 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/1248
Author: @JvE-iO
Created: 1/29/2025
Status: Merged
Merged: 1/30/2025
Merged by: @StefH

Base: masterHead: patch-1


📝 Commits (4)

  • eb7cf46 Add exception message to logging when mapping fails due to an exception.
  • eafc42b Merge branch 'WireMock-Net:master' into patch-1
  • 4951ef3 Revert "Add exception message to logging when mapping fails due to an exception."
  • 2e247ff Fix loggers with improved exception logging.

📊 Changes

3 files changed (+7 additions, -7 deletions)

View changed files

📝 examples/WireMock.Net.WebApplication.NET6/WireMockService.cs (+2 -2)
📝 src/WireMock.Net.TUnit/TUnitWireMockLogger.cs (+2 -2)
📝 src/WireMock.Net.xUnit/TestOutputHelperWireMockLogger.cs (+3 -3)

📄 Description

I spent way too much time in the debugger trying to figure out why the mapping was not found.
It was found, but an exception occurred during the mapping, leading to a misleading message.

As an aside, the exception occurred because I called .WithMapping() before I called .AddProtoDefinition()...


🔄 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/1248 **Author:** [@JvE-iO](https://github.com/JvE-iO) **Created:** 1/29/2025 **Status:** ✅ Merged **Merged:** 1/30/2025 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (4) - [`eb7cf46`](https://github.com/wiremock/WireMock.Net/commit/eb7cf46c95a5b7e6744c7c9f5642878c5e49e1f7) Add exception message to logging when mapping fails due to an exception. - [`eafc42b`](https://github.com/wiremock/WireMock.Net/commit/eafc42b90b44c2c5ce3fc6f9c9f62c71fc1a3e87) Merge branch 'WireMock-Net:master' into patch-1 - [`4951ef3`](https://github.com/wiremock/WireMock.Net/commit/4951ef36dcfe5ca676a462ee68ff31e68a1c2b18) Revert "Add exception message to logging when mapping fails due to an exception." - [`2e247ff`](https://github.com/wiremock/WireMock.Net/commit/2e247ff2eabfc5d90ac24c309eeb5068ab13e378) Fix loggers with improved exception logging. ### 📊 Changes **3 files changed** (+7 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `examples/WireMock.Net.WebApplication.NET6/WireMockService.cs` (+2 -2) 📝 `src/WireMock.Net.TUnit/TUnitWireMockLogger.cs` (+2 -2) 📝 `src/WireMock.Net.xUnit/TestOutputHelperWireMockLogger.cs` (+3 -3) </details> ### 📄 Description I spent way too much time in the debugger trying to figure out why the mapping was not found. It was found, but an exception occurred during the mapping, leading to a misleading message. As an aside, the exception occurred because I called `.WithMapping()` before I called `.AddProtoDefinition()`... --- <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 09:19:13 +01:00
adam closed this issue 2025-12-29 09:19:13 +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#1299