Skip Menu |

This queue is for tickets about the Mojo-JSON-Any CPAN distribution.

Report information
The Basics
Id: 80160
Status: resolved
Priority: 0/
Queue: Mojo-JSON-Any

People
Owner: Nobody in particular
Requestors: davido [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)



Subject: Won't pass Mojo::JSON's tests.
As a near drop-in replacement for Mojo::JSON, Mojo::JSON::Any (with Mojo::JSON::XS) should be able to pass the Mojo::JSON test suite. Currently it fails tests #17, 19, 68, 72, 73, and 81. Each of those seem to be related to the fact that Mojo/JSON.t is expecting "1" (a string) wheras Mojo::JSON::XS is returning 1 (an integer). Tests 86 and 87 are also failing "properly escaped" tests. Attached is a copy of Mojo/JSON.t, modified to instantiate and test Mojo::JSON::Any instead of Mojo::JSON. Whether or not Mojo::JSON::Any passes (or fails) Mojolicious's t/Mojo/JSON.t tests should be a strong indicator of whether or not regressions are introduced by the XS version.
Subject: json.t

Message body is not shown because it is too large.

I've fixed issues with boolean values and escaping of Unicode whitespaces. Other issues connected with blessed references without TO_JSON method and with error messages are described in BUGS AND LIMITATIONS section.