search

spray / spray-json

A lightweight, clean and simple JSON implementation in Scala
Apache License 2.0
969 stars 190 forks source link

JsonParserSpecJvm is failing #315

Closed SethTisue closed 5 years ago

SethTisue commented 5 years ago

this turned up in the Scala 2.12 community build, but it's reproducible outside of dbuild, just clone the rep and run sbt sprayJsonJVM/test and you get

[info] JsonParserSpecJvm
[info] The JsonParser (on the JVM) should
[info]   + be reentrant
[error]   x fail gracefully for deeply nested structures
[error]    'stackoverflow' != 'nonfatal: JSON input nested too deeply:JSON input was nested more deeply than the configured limit of maxDepth = 200' (JsonParserSpecJvm.scala:67)
[error] Actual:   stackoverflow
[error] Expected: nonfatal: JSON input nested too deeply:JSON input was nested more deeply than the configured limit of maxDepth = 200
SethTisue commented 5 years ago

@jrudolph sequel to #304?

jrudolph commented 5 years ago

It doesn't fail on my machine. It failed on Travis so I adapted the bounds a bit so now it passed on Travis as well. I guess on some JVMs it might still fail. Let's make the bounds a little more lose again.

jrudolph commented 5 years ago

Hopefully fixed in #316.