Re: json-path-assert for json related unit tests

2016-09-08 Thread Jinmei Liao
I just made a change to upgrade jackson and scala module to 2.8.2 On Thu, Sep 8, 2016 at 12:30 PM, Anthony Baker wrote: > 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 >

Re: json-path-assert for json related unit tests

2016-09-08 Thread Anthony Baker
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 wrote: > > +1 to standardizing Geode

Re: json-path-assert for json related unit tests

2016-09-08 Thread John Blum
+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 wrote: > Yes, we should change all json parsing code to use jackson. However, we > still

Re: json-path-assert for json related unit tests

2016-09-08 Thread Kirk Lund
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

Re: json-path-assert for json related unit tests

2016-09-08 Thread Jinmei Liao
Can we get rid of TypedJson in favor of some standard json libraries? On Thu, Sep 8, 2016 at 9:56 AM, Kirk Lund 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