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>
Download (untitled) / with headers
text/plain 722b
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
Download (untitled) / with headers
text/plain 1.1k
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>
Download (untitled) / with headers
text/plain 552b
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
Download (untitled) / with headers
text/plain 333b
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/...
Download (untitled) / with headers
text/plain 343b
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>
Download (untitled) / with headers
text/plain 336b
* 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
Download (untitled) / with headers
text/plain 506b
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!


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.