We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I'd expect this to pass if added to Tests.cs
https://github.com/TinyBlueRobots/Hornbill/blob/master/src/Tests.CSharp/Tests.cs
It resolves to 404
[Test] public void IgnoreQueryString() { using (var fakeService = new FakeService()) using (var httpClient = HttpClient(fakeService.Start())) { fakeService.AddResponse("/test", Method.GET, Response.WithStatusCode(418)); Assert.That((int)httpClient.GetAsync("/test?foo=bar").Result.StatusCode, Is.EqualTo(418)); Assert.That((int)httpClient.GetAsync("/test?foo=baz").Result.StatusCode, Is.EqualTo(418)); } }
The text was updated successfully, but these errors were encountered:
Although I did find that making it a regex with .* on the end will work
.*
Sorry, something went wrong.
My thinking was to force test cases to be explicit so there's no surprises. The pattern is always a regex, so we should put that in the docs.
What do you think @CraftyFella ?
This caught me out today too.. so possibly worth considering.
Adding to docs makes sense to me.. shall I do this?
No branches or pull requests
I'd expect this to pass if added to Tests.cs
https://github.com/TinyBlueRobots/Hornbill/blob/master/src/Tests.CSharp/Tests.cs
It resolves to 404
The text was updated successfully, but these errors were encountered: