Skip Menu |

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

Report information
The Basics
Id: 85775
Status: resolved
Worked: 30 min
Priority: 0/
Queue: JSON-Tiny
People
Owner: Nobody in particular
Requestors: DOUGDUDE [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.27
Fixed in: (no value)

History Show all quoted text
  Fri May 31 14:56:46 2013 DOUGDUDE [...] cpan.org - Ticket created
Subject: Test fails on Windows Strawberry Perl
You may want to port the commit https://github.com/kraih/mojo/commit/caa1af3e2b86280f20c09d271c8cd0e95f3a83c4 from Mojolicious, which changes the inf and nan tests to not actually care about the representation. On this platform, the values are dumb (imo). # Failed test 'encode "inf"' # at t/20-mojo-json.t line 322. # got: '{"test":"1.#INF"}' # expected: '{"test":"inf"}' # Failed test 'encode "nan"' # at t/20-mojo-json.t line 323. # got: '{"test":"-1.#IND"}' # expected: '{"test":"nan"}' # Looks like you failed 2 tests of 130. t/20-mojo-json.t ..... Dubious, test returned 2 (wstat 512, 0x200)
  Fri May 31 23:49:52 2013 davido [...] cpan.org - Correspondence added 30 min
On Fri May 31 11:56:46 2013, DOUGDUDE wrote: Show quoted text
> You may want to port the commit > https://github.com/kraih/mojo/commit/caa1af3e2b86280f20c09d271c8cd0e95f3a83c4 > from Mojolicious, which changes the inf and nan tests to not > actually care about the representation. On this platform, the > values are dumb (imo).
All commits to kraih/mojo from May 15th that touch code relevant to this distribution have now been applied to JSON::Tiny. One of the patches specifically targets the tests in question to make them more portable. The issue you were seeing should be fixed in JSON::Tiny v0.28, released 5/13/2013. Thanks for bringing this to my attention. -- Dave
  Fri May 31 23:49:53 2013 The RT System itself - Status changed from 'new' to 'open'
  Fri May 31 23:49:53 2013 davido [...] cpan.org - Status changed from 'open' to 'resolved'