Skip to content
New issue

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

JsonPathMatcher always convert to JArray before matching #307

Closed
mapjaha opened this issue Jul 30, 2019 · 10 comments
Closed

JsonPathMatcher always convert to JArray before matching #307

mapjaha opened this issue Jul 30, 2019 · 10 comments
Assignees
Labels

Comments

@mapjaha
Copy link

mapjaha commented Jul 30, 2019

No description provided.

@mapjaha mapjaha changed the title JsonPathMatcher always convert to JArray before maching JsonPathMatcher always convert to JArray before matching Jul 30, 2019
@mapjaha
Copy link
Author

mapjaha commented Jul 30, 2019

I have query with this body { "x": "42" }
To check with JsonPathMatcher if x property exist i need this pattern: "$[0].x".
According to your documentation it should be "$.x".
this is my configuration for Wiremock:

  server
                .Given(
                    Request.Create().WithPath("/jmespath_example").UsingPost()
                        .WithBody(new JsonPathMatcher("$[0].x"))
                )
                .RespondWith(Response.Create().WithBody("Hello"));

@StefH StefH self-assigned this Aug 2, 2019
@StefH StefH added the bug label Aug 2, 2019
@StefH
Copy link
Collaborator

StefH commented Aug 2, 2019

#311

@StefH
Copy link
Collaborator

StefH commented Aug 2, 2019

I think I fixed it, can you try this WireMock.Net.1.0.26-ci-11553 from MyGet , https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions

@mapjaha
Copy link
Author

mapjaha commented Aug 5, 2019

Maybe yes, but as far as i can see your PR is open.
When it will be released?

@StefH
Copy link
Collaborator

StefH commented Aug 5, 2019

You can try the preview version which is available at MyGet:

WireMock.Net.1.0.26-ci-11553 from MyGet , https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions

@StefH
Copy link
Collaborator

StefH commented Sep 1, 2019

@mapjaha Did you have time to test this?

@mapjaha
Copy link
Author

mapjaha commented Sep 1, 2019 via email

@StefH
Copy link
Collaborator

StefH commented Sep 23, 2019

@mapjaha Did you have time to test?

@mapjaha
Copy link
Author

mapjaha commented Oct 9, 2019

@StefH
Hi Steff,
sorry for no reply so long.
I have checked changes from this PR #311 and it works.
I can confirm that changes solve my issue.

@StefH
Copy link
Collaborator

StefH commented Oct 9, 2019

Thank you for testing.

A new NuGet will be released in few days.

@StefH StefH closed this as completed Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants