prickle icon indicating copy to clipboard operation
prickle copied to clipboard

play-json

Open antonkulaga opened this issue 11 years ago • 3 comments

It would be nice to see play-json pickler (maybe as separate subproject) like it is in scala-js-picklers

antonkulaga avatar Sep 27 '14 21:09 antonkulaga

That shouldn't be very difficult. I'll try to add this at some point.

On Sun, Sep 28, 2014 at 7:29 AM, Anton Kulaga [email protected] wrote:

It would be nice to see play-json pickler (maybe as separate subproject) like it is in scala-js-picklers

— Reply to this email directly or view it on GitHub https://github.com/benhutchison/prickle/issues/2.

benhutchison avatar Sep 30 '14 22:09 benhutchison

I started coding play-json support but had to pause due to more importants todos. But what I undestood from my attempt is that generic test suites are missing, so everyone can make a subproject for his/her own favourite json lib and extend some generic test suite to test if everything is ok with readers/writers.

antonkulaga avatar Feb 16 '15 00:02 antonkulaga

IIRC supporting another json framework should involve just a new PBuilder and PReader:

https://github.com/benhutchison/prickle/blob/master/shared/src/main/scala/prickle/PConfig.scala#L38

Currently the tests dont really focus on this in isolation, partly because its lower risk relative to the main un/pickling logic. Some explicit contract testing along these interfaces would be valuable and not especially difficult, and could be made generic.

On Mon, Feb 16, 2015 at 11:33 AM, Anton Kulaga [email protected] wrote:

I started coding play-json support but had to pause due to more importants todos. But what I undestood from my attempt is that generic test suites are missing, so everyone can make a subproject for his/her own favourite json lib and extend some generic test suite to test if everything is ok with readers/writers.

— Reply to this email directly or view it on GitHub https://github.com/benhutchison/prickle/issues/2#issuecomment-74446278.

benhutchison avatar Feb 16 '15 01:02 benhutchison