Fix WithPath to handle URL-encoded characters

[Copilot]

WithPath() now correctly matches requests containing special characters like | by decoding the path before comparing it to the pattern, using the same approach as WithQuery().

Changes Made

• Path Matching: Modified RequestMock.cs to decode Uri.AbsolutePath using WebUtility.UrlDecode() before matching against the pattern
• Consistent Approach: Both path and query now use the same decode-before-match strategy for handling URL-encoded characters
• Tests Added: Added tests in HttpMockSpecs.BasicUsage:
  • Can_match_path_with_pipe_character() - validates path matching with | character
  • Can_match_query_with_pipe_character() - validates query matching with | character

Before

var key = $"{Guid.NewGuid()}|{Guid.NewGuid()}";
mock.ForDelete()
    .WithPath($"IncomeRelations/{key}".Replace("|", "%7C"))  // Manual encoding required
    .RespondsWithStatus(HttpStatusCode.OK);

After

var key = $"{Guid.NewGuid()}|{Guid.NewGuid()}";
mock.ForDelete()
    .WithPath($"IncomeRelations/{key}")  // Just works
    .RespondsWithStatus(HttpStatusCode.OK);

Testing

• ✅ All 106 tests pass
• ✅ No security vulnerabilities detected
• ✅ Both path and query handle special characters consistently

The solution is simple and maintainable, requiring only a single line change to decode the path before matching, making it consistent with how query strings are already handled.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>WithPath and WithQuery should URL-encode</issue_title>
<issue_description>Given the following test

    [Fact]
    public async Task An_income_relationship_rule_is_deleted()
    {
        // Arrange
        var incomeRelationshipRuleKey = $"{Guid.NewGuid()}|{Guid.NewGuid()}";

        var capturedRequests = new RequestCollection();

        httpMock.ForDelete()
            .WithPath($"IncomeRelations/{incomeRelationshipRuleKey}".Replace("|", "%7C"))
            .CollectingRequestsIn(capturedRequests)
            .RespondsWithStatus(HttpStatusCode.OK);

        // Act
        await provider.DeleteSectorAssignmentRule(incomeRelationshipRuleKey);

        // Assert
        capturedRequests.Count.Should().Be(1);
    }

This only succeeds when .Replace("|", "%7C") is added as the HttpClient will URL-encode the actual request, but the mock is comparing the encoded | to the actual unencoded |</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes WithPath and WithQuery should URL-encode #50

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[dennisdoomen]

@copilot please address the comments

[github-actions]

Test Results

  3 files  ±0    3 suites  ±0   12s ⏱️ -5s
112 tests +2  112 ✅ +2  0 💤 ±0  0 ❌ ±0 
218 runs  +4  218 ✅ +4  0 💤 ±0  0 ❌ ±0 

Results for commit 5fd787c. ± Comparison against base commit 04828cb.

♻️ This comment has been updated with latest results.

[coveralls]

Pull Request Test Coverage Report for Build 20614151351

Details

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 83.668%

---

Totals
Change from base Build 20590170682:	0.0%
Covered Lines:	1098
Relevant Lines:	1265

---

💛 - Coveralls