Speaking of jackson, anyone know why we’re not using v2.2.0 for the scala 
module?  This is a dependency of geode-web-api.

jackson.version = 2.2.0
jackson-module-scala_2.10.version = 2.1.5


Anthony

> On Sep 8, 2016, at 10:32 AM, John Blum <jb...@pivotal.io> wrote:
> 
> +1 to standardizing Geode on Jackson for JSON processing.
> 
> +1 to using JsonPath/json-path-assert for testing interactions in Geode on
> JSON docs.
> 
> On Thu, Sep 8, 2016 at 10:22 AM, Kirk Lund <kl...@apache.org> wrote:
> 
>> Yes, we should change all json parsing code to use jackson. However, we
>> still need to write unit tests and that's what json-path-assert is for.
>> I've researched all of the ways to perform json assertions for unit tests
>> and test-path-assert is currently the best. It's from com.jayway same as
>> Awaitility.
>> 
>> -Kirk
>> 
>> 
>> On Thu, Sep 8, 2016 at 10:06 AM, Jinmei Liao <jil...@pivotal.io> wrote:
>> 
>>> Can we get rid of TypedJson in favor of some standard json libraries?
>>> 
>>> On Thu, Sep 8, 2016 at 9:56 AM, Kirk Lund <kl...@apache.org> wrote:
>>> 
>>>> To support fixing up unit tests that involve json, I'd like to add in a
>>>> dependency on com.jayway.jsonpath:json-path-assert.
>>>> 
>>>> We already have a dependency on com.jayway.jsonpath:json-path (for
>>>> spring-hateoas). json-path-assert basically adds some new Hamcrest
>>> matchers
>>>> for clean assertions involving json.
>>>> 
>>>> I intend to use this in TypedJsonJUnitTest (GEODE-1842) because that
>>>> test doesn't currently have assertions(!).
>>>> 
>>>> -Kirk
>>>> 
>>> 
>>> 
>>> 
>>> --
>>> Cheers
>>> 
>>> Jinmei
>>> 
>> 
> 
> 
> 
> --
> -John
> 503-504-8657
> john.blum10101 (skype)

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to