Skip Menu |
 

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

Report information
The Basics
Id: 103382
Status: open
Priority: 0/
Queue: Perl-Critic-Dynamic

People
Owner: Nobody in particular
Requestors: ppisar [...] redhat.com
Cc:
AdminCc:

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

Attachments
Perl-Critic-Dynamic-0.05-test_AUTOLOAD_on_private_module.patch



Subject: t/20_policies.t tests fail with CGI 4.14
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.18-21253-1428493845-90.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 638
Download (untitled) / with headers
text/plain 638b
After upgrading CGI from 4.13 to 4.14, t/20_policies.t tests fail: ok 28 - Dynamic::ValidateAgainstSymbolTable - line 329 - at_inc_suffix configuration not ok 29 - Dynamic::ValidateAgainstSymbolTable - line 338 - AUTOLOADers ignored by default # Failed test 'Dynamic::ValidateAgainstSymbolTable - line 338 - AUTOLOADers ignored by default' # at t/20_policies.t line 33. # Expected 0 violations, got 1. Found violations follow... # Found violation: Subroutine "CGI::FooBar" does not appear to be defined at line 4, column 1. (CGI::FooBar();) ok 30 - Dynamic::ValidateAgainstSymbolTable - line 348 - AUTOLOADers inspected on request
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-21253-1428493845-90.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
X-RT-Interface: API
References: <rt-4.0.18-21253-1428493845-90.0-0-0 [...] rt.cpan.org>
Content-Type: multipart/mixed; boundary="----------=_1428497212-23368-8"
Message-ID: <rt-4.0.18-23368-1428497212-1410.0-0-0 [...] rt.cpan.org>
Message-ID: <rt-4.0.18-23368-1428497212-1438.103382-0-0 [...] rt.cpan.org>
X-RT-Original-Encoding: utf-8
From: ppisar [...] redhat.com
Content-Length: 0
Content-Disposition: inline
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 185
Download (untitled) / with headers
text/plain 185b
Dne St 08.dub.2015 07:50:45, ppisar napsal(a): Show quoted text
> After upgrading CGI from 4.13 to 4.14, t/20_policies.t tests fail: >
CGI-4.14 stopped using AUTOLOAD. Attached patch fixes it. -- Petr
MIME-Version: 1.0
Subject: Perl-Critic-Dynamic-0.05-test_AUTOLOAD_on_private_module.patch
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Type: text/x-patch; name="Perl-Critic-Dynamic-0.05-test_AUTOLOAD_on_private_module.patch"
Content-Disposition: inline; filename="Perl-Critic-Dynamic-0.05-test_AUTOLOAD_on_private_module.patch"
Content-Transfer-Encoding: binary
Content-Length: 1559
Test AUTOLOAD on private module CGI 4.14 stopped to use AUTOLOAD which caused failing tests. This patch provides private module with AUTOLOAD instead of relying on CGI. CPAN RT#103382 Index: t/Dynamic/ValidateAgainstSymbolTable.run =================================================================== --- t/Dynamic/ValidateAgainstSymbolTable.run (revision 4222) +++ t/Dynamic/ValidateAgainstSymbolTable.run (working copy) @@ -336,23 +336,24 @@ #----------------------------------------------------------------------------- ## name AUTOLOADers ignored by default +## parms { at_inc => 'tlib' } ## failures 0 ## cut -use CGI; # Has 'sub AUTOLOAD {...}' +use TestAutoload; # Has 'sub AUTOLOAD {...}' -CGI::FooBar(); +TestAutoload::FooBar(); #----------------------------------------------------------------------------- ## name AUTOLOADers inspected on request -## parms { inspect_autoloaders => 1 } +## parms { at_inc => 'tlib', inspect_autoloaders => 1 } ## failures 1 ## cut -use CGI; # Has 'sub AUTOLOAD {...}' +use TestAutoload; # Has 'sub AUTOLOAD {...}' -CGI::FooBar(); +TestAutoload::FooBar(); #----------------------------------------------------------------------------- Index: tlib/TestAutoload.pm =================================================================== --- tlib/TestAutoload.pm (revision 0) +++ tlib/TestAutoload.pm (working copy) @@ -0,0 +1,12 @@ +package TestAutoload; + +sub AUTOLOAD { + print "Autoloading <$AUTOLOAD>\n"; + goto &foo; +} + +sub foo { + print "foo() called\n"; +} + +1;
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-21253-1428493845-90.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-21253-1428493845-90.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-10580-1435445722-1955.103382-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 121
Download (untitled) / with headers
text/plain 121b
Additionally, CGI.pm is no longer in core, so relying on it without declaring it as a dependency breaks the test as well.


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.