ShouldHaveCalled gives false negative when called URL ends with "?"

[cesit-nav]

I am creating unit testing for flur call with multipal SetQueryParam it is throwing error like below

Flurl.Http.Testing.HttpTestException: 'Expected any calls to be made with URL pattern https://stamped.io/api/widget/reviews?, but no matching calls were made.'

Please let me know what is going wrong here.
please check first screen shot is for actual call and second is for unit test

[tmenier]

Does PublicReviewsWithFilter actually contain a trailing ?? If so remove the ? from that constant. It's not necessary for SetQueryParam and it might be confusing ShouldHaveCalled, which has special logic for ignoring query strings.

[tmenier]

Confirmed there's a bug here. (As a side note, if you had provided a minimal, isolated example such as below, it would have helped me understand this much sooner. But I got there anyway. 😉)

[TestCase("https://api.com/foo?", "https://api.com/foo?")] // fail
[TestCase("https://api.com/foo", "https://api.com/foo?")]  // pass
[TestCase("https://api.com/foo?", "https://api.com/foo")]  // fail
public async Task bug_721(string actual, string expected) {
	using var httpTest = new HttpTest();
	var x = await actual.GetAsync();
	httpTest.ShouldHaveCalled(expected);
}

[tmenier]

This wasn't a bug actually a bug in HttpTest like I suspected but rather a side-effect of #734