Bug #116753 for WebService-Freshservice: System perl used in tests

Sat Aug 06 04:50:15 2016 SREZIC [...] cpan.org - Ticket created

Subject:

System perl used in tests

Tests fail on many smoker systems, see http://matrix.cpantesters.org/?dist=WebService-Freshservice%200.001 One reason seems to be that the system perl is used for running some test scripts, not $^X. Example: ... Can't locate Dancer2.pm in @INC (you may need to install the Dancer2 module) (@INC contains: ./t/lib/ /tmpfs/.cpan-build-cpansand/2016080606/WebService-Freshservice-0.001-0/blib/lib /tmpfs/.cpan-build-cpansand/2016080606/WebService-Freshservice-0.001-0/blib/arch /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.20.2 /usr/local/share/perl/5.20.2 /usr/lib/x86_64-linux-gnu/perl5/5.20 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl/5.20 /usr/share/perl/5.20 /usr/local/lib/site_perl .) at t/bin/cached_api.pl line 5. BEGIN failed--compilation aborted at t/bin/cached_api.pl line 5. # No tests run! # Failed test 'No tests run for subtest "Create Requester - Minimal Options"' # at t/WebService/Freshservice.t line 80. API failed - error: 'Can't connect to localhost:3001' at /tmpfs/.cpan-build-cpansand/2016080606/WebService-Freshservice-0.001-0/blib/lib/WebService/Freshservice.pm line 141. # Tests were run but no plan was declared and done_testing() was not seen. # Looks like your test exited with 255 just after 5. t/WebService/Freshservice.t ................... Dubious, test returned 255 (wstat 65280, 0xff00) Failed 1/5 subtests ...

Sun Aug 07 22:34:10 2016 techman [...] cpan.org - Taken

Sun Aug 07 22:36:23 2016 techman [...] cpan.org - Correspondence added

Hi, Thanks for the notification, I've altered the #! and pushed a new release. Regards, Leon

Sun Aug 07 22:36:24 2016 The RT System itself - Status changed from 'new' to 'open'

Mon Aug 08 01:58:38 2016 SREZIC [...] cpan.org - Correspondence added

This does not help, I see the same errors with 0.002. This would fix the problem: diff --git a/t/lib/WebService/Freshservice/Test.pm b/t/lib/WebService/Freshservice/Test.pm index b5b5fed..9ca2ffb 100644 --- a/t/lib/WebService/Freshservice/Test.pm +++ b/t/lib/WebService/Freshservice/Test.pm @@ -54,7 +54,7 @@ method test_with_dancer($test, $number_tests) { my $pid = fork(); if (!$pid) { - exec("t/bin/cached_api.pl"); + exec($^X, "t/bin/cached_api.pl"); } # Allow some time for the instance to spawn. TODO: Make this smarter On 2016-08-07 22:36:23, TECHMAN wrote: Show quoted text

> Hi, > > Thanks for the notification, I've altered the #! and pushed a new release. > > Regards, > > Leon

Mon Aug 08 02:11:53 2016 techman [...] cpan.org - Correspondence added

Aha, that's rather neat! I've pushed a release with that change. Thanks, Leon

Mon Aug 08 23:29:05 2016 techman [...] cpan.org - Correspondence added

This failure scenario has been fixed.

Mon Aug 08 23:29:05 2016 techman [...] cpan.org - Status changed from 'open' to 'resolved'

Mon Aug 08 23:29:10 2016 techman [...] cpan.org - Fixed in 0.003 added