Included JSON matching for body

[pkalbrun-goeuro]

Feature request:
Include JSON patch matching for request body in the withBody() matcher.
Like https://github.com/spotify/java-hamcrest#json-matchers

[PawelAdamski]

I guess you mean JSON path :)

HttpClientMock in current version works fine with json-matchers

  HttpClientMock mock = new HttpClientMock();
        mock.onGet("/aaa").withBody(isJsonStringMatching(
            jsonObject().where("foo", is(jsonInt(1)))
            )
        );

Do you need something more?

[pkalbrun-goeuro]

Yes, it works perfectly fine. My point was to include it inside library to make it more smooth :)

[jarst]

I've added a PR adding dependency to hamcrest-jackson.
However, the more I think about this "improvement", the less appealing whole idea seems to me...

[PawelAdamski]

Hi @jarst
Thanks for contributing to HttpClientMock.
I've looked at your PR. I agree that adding dependency is not a big improvement.
TBH I rather thought about adding a method like:
withBodyHavingJsonField('foo.bar', equal('some test'))
Before you start implementation please reach me and consult the proposition.