Skip Menu |

This queue is for tickets about the Catalyst-Action-REST CPAN distribution.

Report information
The Basics
Id: 101854
Status: resolved
Priority: 0/
Queue: Catalyst-Action-REST

People
Owner: Nobody in particular
Requestors: emmanuel [...] seyman.fr
Cc:
AdminCc:

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



Subject: t/catalyst-action-rest.t fails
Date: Sun, 1 Feb 2015 10:01:20 +0100
To: bug-Catalyst-Action-REST [...] rt.cpan.org
From: Emmanuel Seyman <emmanuel [...] seyman.fr>
I'm trying to update Catalyst-Action-REST's package in Fedora's rawhide branch and I'm seeing one test fail: t/catalyst-action-rest.t ................................ ok # Failed test 'GET the serialized request succeeded' # at t/catalyst-action-serialize-accept.t line 47. malformed JSON string, neither tag, array, object, number, string or atom, at character offset 0 (before "Content-Type applica...") at t/catalyst-action-serialize-accept.t line 48. # Tests were run but no plan was declared and done_testing() was not seen. # Looks like your test exited with 2 just after 5. t/catalyst-action-serialize-accept.t .................... Dubious, test returned 2 (wstat 512, 0x200) Failed 1/5 subtests Emmanuel
On 2015-02-01 01:01:34, emmanuel@seyman.fr wrote: Show quoted text
> > I'm trying to update Catalyst-Action-REST's package in Fedora's > rawhide branch and I'm seeing one test fail: > > t/catalyst-action-rest.t ................................ ok > > # Failed test 'GET the serialized request succeeded' > # at t/catalyst-action-serialize-accept.t line 47. > malformed JSON string, neither tag, array, object, number, string or > atom, at character offset 0 (before "Content-Type applica...") at > t/catalyst-action-serialize-accept.t line 48. > # Tests were run but no plan was declared and done_testing() was not > seen. > # Looks like your test exited with 2 just after 5. > t/catalyst-action-serialize-accept.t .................... > Dubious, test returned 2 (wstat 512, 0x200) > Failed 1/5 subtests > > Emmanuel
What versions do you have installed for: JSON::PP JSON JSON::XS Cpanel::JSON::XS Could you also please run the test on its own, like this (it will require installing Devel::Confess, if you do not have it), and include the output here: cd Catalyst-Action-Rest-1.18 perl Makefile.PL make perl -Mblib -d:Confess t/catalyst-action-serialize-accept.t
Subject: Re: [rt.cpan.org #101854] t/catalyst-action-rest.t fails
Date: Sun, 1 Feb 2015 23:49:08 +0100
To: Karen Etheridge via RT <bug-Catalyst-Action-REST [...] rt.cpan.org>
From: Emmanuel Seyman <emmanuel [...] seyman.fr>
FWIW, a log of the build is here: https://kojipkgs.fedoraproject.org//work/tasks/586/8790586/root.log https://kojipkgs.fedoraproject.org//work/tasks/586/8790586/build.log Show quoted text
> What versions do you have installed for: > JSON::PP
JSON-PP-2.27300 Show quoted text
> JSON
JSON-2.90 Show quoted text
> JSON::XS
JSON-XS-3.01 Show quoted text
> Cpanel::JSON::XS
Cpanel-JSON-XS-3.0114 Show quoted text
> Could you also please run the test on its own, like this (it will require installing Devel::Confess, if you do not have it), and include the output here:
Sorry, Devel::Confess isn't packaged in Fedora. :-( Emmanuel
On 2015-02-01 14:49:22, emmanuel@seyman.fr wrote: Show quoted text
> > Could you also please run the test on its own, like this (it will > > require installing Devel::Confess, if you do not have it), and > > include the output here:
> > Sorry, Devel::Confess isn't packaged in Fedora. :-(
what about Carp::Always? perl -Mblib -MCarp::Always t/...
On 2015-02-01 14:49:22, emmanuel@seyman.fr wrote: Show quoted text
The second log looks truncated - it cuts off after the tests have started, and doesn't include the failing test.
Subject: Re: [rt.cpan.org #101854] t/catalyst-action-rest.t fails
Date: Mon, 2 Feb 2015 02:43:14 +0100
To: Karen Etheridge via RT <bug-Catalyst-Action-REST [...] rt.cpan.org>
From: Emmanuel Seyman <emmanuel [...] seyman.fr>
* Karen Etheridge via RT [01/02/2015 13:32] : Show quoted text
> > What versions do you have installed for: > JSON::PP > JSON > JSON::XS > Cpanel::JSON::XS
Playing around with the buildroot, I realized that the tests fails when JSON::MaybeXS is not present. If it is available, the test passes just fine. I apologize for the invalid report. Emmanuel
On 2015-02-01 17:43:25, emmanuel@seyman.fr wrote: Show quoted text
> * Karen Etheridge via RT [01/02/2015 13:32] :
> > > > What versions do you have installed for: > > JSON::PP > > JSON > > JSON::XS > > Cpanel::JSON::XS
> > Playing around with the buildroot, I realized that the tests fails when > JSON::MaybeXS is not present. If it is available, the test passes just fine.
This sounds like a valid issue. JSON::MaybeXS isn't listed as a dependency, so either it should be, or something else upstream should be doing so.
Fixed in repo, will release shortly. Thanks all!