Skip Menu |

This queue is for tickets about the Perl-Critic-Pulp CPAN distribution.

Report information
The Basics
Id: 100298
Status: resolved
Priority: 0/
Queue: Perl-Critic-Pulp

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

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



Subject: Tests failing under PPI 1.220 and Perl::Critic 1.123
From CPAN shell: Test Summary Report ------------------- t/ConstantBeforeLt.t (Wstat: 256 Tests: 49 Failed: 2) Failed tests: 29-30 Non-zero exit status: 1 t/NotWithCompare.t (Wstat: 256 Tests: 116 Failed: 1) Failed test: 104 Non-zero exit status: 1 I expect that the ConstantBeforeLt needs to be changed in response to the PPI 1.220 interface change to PPI::Statement::Sub::prototype https://github.com/adamkennedy/PPI/commit/18adbca35b4cba94988027a26ab350bc528104a3 Details: KRYDE/Perl-Critic-Pulp-85.tar.gz /bin/make -- OK Running make test PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/ConstantBeforeLt.t ............... 1/49 Use of uninitialized value in string eq at /root/.local/share/.cpan/build/Perl-Critic-Pulp-85-cCiVoV/blib/lib/Perl/Critic/Policy/ValuesAndExpressions/ConstantBeforeLt.pm line 114. Use of uninitialized value in string eq at /root/.local/share/.cpan/build/Perl-Critic-Pulp-85-cCiVoV/blib/lib/Perl/Critic/Policy/ValuesAndExpressions/ConstantBeforeLt.pm line 114. # Failed test 'sub FOO () { 123; }' # at t/ConstantBeforeLt.t line 99. # Structures begin differing at: # $got->[0] = Does not exist # $expected->[0] = 'FOO' # Failed test 'sub FOO () { 123; };' # at t/ConstantBeforeLt.t line 99. # Structures begin differing at: # $got->[0] = Does not exist # $expected->[0] = 'FOO' t/ConstantBeforeLt.t ............... 33/49 # Saw 2 warning(s): # # Trace begun at t/MyTestHelpers.pm line 55 # MyTestHelpers::nowarnings_handler('Use of uninitialized value in string eq at /root/.local/share/.cpan/build/Perl-Critic-Pulp-85-cCiVoV/blib/lib/Perl/Critic/Policy/ValuesAndExpressions/ConstantBeforeLt.pm line 114.^J') called at /root/.local/share/.cpan/build/Perl-Critic-Pulp-85-cCiVoV/blib/lib/Perl/Critic/Policy/ValuesAndExpressions/ConstantBeforeLt.pm line 114 # Perl::Critic::Policy::ValuesAndExpressions::ConstantBeforeLt::_use_constants('PPI::Statement::Sub=HASH(0x3009078)') called at t/ConstantBeforeLt.t line 98 # # # Trace begun at t/MyTestHelpers.pm line 55 # MyTestHelpers::nowarnings_handler('Use of uninitialized value in string eq at /root/.local/share/.cpan/build/Perl-Critic-Pulp-85-cCiVoV/blib/lib/Perl/Critic/Policy/ValuesAndExpressions/ConstantBeforeLt.pm line 114.^J') called at /root/.local/share/.cpan/build/Perl-Critic-Pulp-85-cCiVoV/blib/lib/Perl/Critic/Policy/ValuesAndExpressions/ConstantBeforeLt.pm line 114 # Perl::Critic::Policy::ValuesAndExpressions::ConstantBeforeLt::_use_constants('PPI::Statement::Sub=HASH(0x30116a0)') called at t/ConstantBeforeLt.t line 98 # # Exit code 1 for warnings # Looks like you failed 2 tests of 49. # Looks like your test exited with 1 just after 49. t/ConstantBeforeLt.t ............... Dubious, test returned 1 (wstat 256, 0x100) Failed 2/49 subtests and # Failed test 'str: sub FOO () { 123 } # ! FOO < 100 || $bar > 200' # at t/NotWithCompare.t line 220. # got: '0' # expected: '1' # Looks like you failed 1 test of 116. t/NotWithCompare.t ................. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/116 subtests
CC: bug-Perl-Critic-Pulp [...] rt.cpan.org
Subject: Re: [rt.cpan.org #100298] Tests failing under PPI 1.220 and Perl::Critic 1.123
Date: Thu, 13 Nov 2014 19:39:20 +1100
To: MOREGAN [...] cpan.org
From: Kevin Ryde <user42_kevin [...] yahoo.com.au>
" via RT" <bug-Perl-Critic-Pulp@rt.cpan.org> writes: Show quoted text
> > PPI 1.220 interface change
Oh a typo there, you meant incompatible change I expect :-). I'll upload shortly. -- The sigfile unexpected subcultures series: Did you know Cairns has a thriving Goth scene? It takes real dedication to maintain a pasty complexion under the tropical sun and wear long black overcoats in 95% humidity.
From: ppisar [...] redhat.com
Dne Čt 13.lis.2014 03:41:23, user42_kevin@yahoo.com.au napsal(a): Show quoted text
> " via RT" <bug-Perl-Critic-Pulp@rt.cpan.org> writes:
> > > > PPI 1.220 interface change
> > Oh a typo there, you meant incompatible change I expect :-). > I'll upload shortly. >
Is that what 86 version should be supposed to do? Because it still fails with PPI 1.220. -- Petr
On Tue Nov 18 07:11:35 2014, ppisar wrote: Show quoted text
> Dne Čt 13.lis.2014 03:41:23, user42_kevin@yahoo.com.au napsal(a):
> > " via RT" <bug-Perl-Critic-Pulp@rt.cpan.org> writes:
> > > > > > PPI 1.220 interface change
> > > > Oh a typo there, you meant incompatible change I expect :-). > > I'll upload shortly. > >
> Is that what 86 version should be supposed to do? Because it still > fails with PPI 1.220. > > -- Petr
FWIW, Changes in version 86 says: Version 87, November 2014 - ConstantBeforeLt allow for PPI incompatible change to ->prototype() release problem?
Subject: Re: [rt.cpan.org #100298] Tests failing under PPI 1.220 and Perl::Critic 1.123
Date: Thu, 20 Nov 2014 18:01:05 +1100
To: bug-Perl-Critic-Pulp [...] rt.cpan.org
From: Kevin Ryde <user42_kevin [...] yahoo.com.au>
Oops, tested properly this time. New version 88 might be better. The incompatibility is worse than how I first read it :-(.
From: paul [...] city-fan.org
On Thu Nov 20 02:05:20 2014, user42_kevin@yahoo.com.au wrote: Show quoted text
> Oops, tested properly this time. New version 88 might be better. > The incompatibility is worse than how I first read it :-(.
The new release does indeed fix the issue, but the release numbers seem to be out of step. The new release is 87 (tarball name, version numbers of modules), not 88, though the Changes file references version 88. Version 86 included a Changes entry for version 87 as noted earlier in this ticket.
Latest version should have scotched it.