Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the PPI CPAN distribution.

Report information
The Basics
Id: 29636
Status: resolved
Priority: 0/
Queue: PPI

People
Owner: Nobody in particular
Requestors: chris+rt [...] chrisdolan.net
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.199_03
Fixed in: (no value)



Subject: [PATCH] whole bunch of failing regression tests
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
X-RT-Original-Encoding: utf-8
Content-Type: multipart/mixed; boundary="----------=_1190870486-24913-7"
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 373
Download (untitled) / with headers
text/plain 373b
The attached patch against PPI SVN r2135 adds a bunch of failing regressions that I've accumulated over the last 4 months. The changes include: * Unsupported magic vars %!, $^X and $^H * The "-1" bug from v1.115 has resurfaced as "@{$arr_ref}-1" * Readline after comma misinterpreted as less-than * Correct test numbering in t/08_regressions.t and t/lib/PPI.pm Chris
Subject: ppi-regressions.patch
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="----------=_1190870485-24913-6"
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 0
Content-Type: application/octet-stream; name="ppi-regressions.patch"
Content-Disposition: inline; filename="ppi-regressions.patch"
Content-Transfer-Encoding: base64
Content-Length: 5751
Index: t/lib/PPI.pm =================================================================== --- t/lib/PPI.pm (revision 2135) +++ t/lib/PPI.pm (working copy) @@ -118,7 +118,7 @@ ##################################################################### # Process a .code/.dump file pair -# plan: 2 + 11 * npairs +# plan: 2 + 14 * npairs sub run_testdir { my $pkg = shift; @@ -151,10 +151,10 @@ my $rv; local *CODEFILE; SKIP: { - skip "No Document to test", 7 unless $Document; + skip "No Document to test", 10 unless $Document; # Check standard things - object_ok( $Document ); # 5 tests contained within + object_ok( $Document ); # 6 tests contained within # Get the dump array ref for the Document object my $Dumper = PPI::Dumper->new( $Document ); Index: t/data/08_regression/30_hash_bang.code =================================================================== --- t/data/08_regression/30_hash_bang.code (revision 0) +++ t/data/08_regression/30_hash_bang.code (revision 0) @@ -0,0 +1 @@ +%! = () Property changes on: t/data/08_regression/17_scope.code ___________________________________________________________________ Name: svn:executable - * Index: t/data/08_regression/14b_minus.code =================================================================== --- t/data/08_regression/14b_minus.code (revision 0) +++ t/data/08_regression/14b_minus.code (revision 0) @@ -0,0 +1 @@ +@{$arr_ref}-1 Index: t/data/08_regression/30_hash_bang.dump =================================================================== --- t/data/08_regression/30_hash_bang.dump (revision 0) +++ t/data/08_regression/30_hash_bang.dump (revision 0) @@ -0,0 +1,8 @@ +PPI::Document + PPI::Statement + PPI::Token::Magic '%!' + PPI::Token::Whitespace ' ' + PPI::Token::Operator '=' + PPI::Token::Whitespace ' ' + PPI::Structure::List ( ... ) + PPI::Token::Whitespace '\n' Property changes on: t/data/08_regression/17_scope.dump ___________________________________________________________________ Name: svn:executable - * Index: t/data/08_regression/14b_minus.dump =================================================================== --- t/data/08_regression/14b_minus.dump (revision 0) +++ t/data/08_regression/14b_minus.dump (revision 0) @@ -0,0 +1,9 @@ +PPI::Document + PPI::Statement + PPI::Token::Cast '@' + PPI::Structure::Block { ... } + PPI::Statement + PPI::Token::Symbol '$arr_ref' + PPI::Token::Operator '-' + PPI::Token::Number '1' + PPI::Token::Whitespace '\n' Index: t/data/08_regression/29_magic_carat.code =================================================================== --- t/data/08_regression/29_magic_carat.code (revision 0) +++ t/data/08_regression/29_magic_carat.code (revision 0) @@ -0,0 +1 @@ +$^X = 1 Index: t/data/08_regression/32_readline.code =================================================================== --- t/data/08_regression/32_readline.code (revision 0) +++ t/data/08_regression/32_readline.code (revision 0) @@ -0,0 +1 @@ +@foo = (<$fh1>, <$fh2>) Index: t/data/08_regression/29_magic_carat.dump =================================================================== --- t/data/08_regression/29_magic_carat.dump (revision 0) +++ t/data/08_regression/29_magic_carat.dump (revision 0) @@ -0,0 +1,8 @@ +PPI::Document + PPI::Statement + PPI::Token::Magic '$^X' + PPI::Token::Whitespace ' ' + PPI::Token::Operator '=' + PPI::Token::Whitespace ' ' + PPI::Token::Number '1' + PPI::Token::Whitespace '\n' Property changes on: t/data/08_regression/27_constant_hash.code ___________________________________________________________________ Name: svn:executable - * Index: t/data/08_regression/31_hash_carat_H.code =================================================================== --- t/data/08_regression/31_hash_carat_H.code (revision 0) +++ t/data/08_regression/31_hash_carat_H.code (revision 0) @@ -0,0 +1 @@ +%^H=() Index: t/data/08_regression/32_readline.dump =================================================================== --- t/data/08_regression/32_readline.dump (revision 0) +++ t/data/08_regression/32_readline.dump (revision 0) @@ -0,0 +1,12 @@ +PPI::Document + PPI::Statement + PPI::Token::Symbol '@foo' + PPI::Token::Whitespace ' ' + PPI::Token::Operator '=' + PPI::Token::Whitespace ' ' + PPI::Structure::List ( ... ) + PPI::Statement::Expression + PPI::Token::QuoteLike::Readline '<$fh1>' + PPI::Token::Operator ',' + PPI::Token::Whitespace ' ' + PPI::Token::QuoteLike::Readline '<$fh2>' Property changes on: t/data/08_regression/27_constant_hash.dump ___________________________________________________________________ Name: svn:executable - * Index: t/data/08_regression/31_hash_carat_H.dump =================================================================== --- t/data/08_regression/31_hash_carat_H.dump (revision 0) +++ t/data/08_regression/31_hash_carat_H.dump (revision 0) @@ -0,0 +1,6 @@ +PPI::Document + PPI::Statement + PPI::Token::Magic '%^H' + PPI::Token::Operator '=' + PPI::Structure::List ( ... ) + PPI::Token::Whitespace '\n' Index: t/08_regression.t =================================================================== --- t/08_regression.t (revision 2135) +++ t/08_regression.t (working copy) @@ -30,12 +30,12 @@ # For each new item in t/data/08_regression add another 14 tests -use Test::More tests => 480; +use Test::More tests => 550; use t::lib::PPI; ##################################################################### # Code/Dump Testing -# ntests = 2 + 11 * nfiles +# ntests = 2 + 14 * nfiles t::lib::PPI->run_testdir( catdir( 't', 'data', '08_regression' ) );
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Disposition: inline
Message-Id: <rt-3.6.HEAD-12832-1191374877-420.29636-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Original-Encoding: utf-8
Content-Length: 44
I committed this patch myself. Thanks Adam!
MIME-Version: 1.0
X-Spam-Status: No, hits=-2.6 required=8.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VERIFIED,DK_SIGNED,HTML_MESSAGE,SPF_PASS
In-Reply-To: <rt-3.6.HEAD-12832-1191374877-420.29636-5-0 [...] rt.cpan.org>
References: <RT-Ticket-29636 [...] rt.cpan.org> <rt-3.6.HEAD-12832-1191374877-420.29636-5-0 [...] rt.cpan.org>
Content-Type: multipart/alternative; boundary="----=_Part_23780_788983.1191396687918"
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 8666A4D80DB for <bug-PPI [...] rt.cpan.org>; Wed, 3 Oct 2007 03:31:38 -0400 (EDT)
Received: (qmail 24530 invoked by alias); 3 Oct 2007 07:31:37 -0000
Received: from qb-out-0506.google.com (HELO qb-out-0506.google.com) (72.14.204.225) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Wed, 03 Oct 2007 00:31:32 -0700
Received: by qb-out-0506.google.com with SMTP id d7so1614688qbd for <bug-PPI [...] rt.cpan.org>; Wed, 03 Oct 2007 00:31:28 -0700 (PDT)
Received: by 10.114.190.6 with SMTP id n6mr372797waf.1191396687923; Wed, 03 Oct 2007 00:31:27 -0700 (PDT)
Received: by 10.114.121.13 with HTTP; Wed, 3 Oct 2007 00:31:27 -0700 (PDT)
Delivered-To: cpan-bug+ppi [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #29636] [PATCH] whole bunch of failing regression tests
Domainkey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; b=XoJTSMnGpbeUIk6tl3EhgxPKTIEBQfeq6cYe2uOWcOnBBsERT8FDyCCG1g/U3d8d9BagrChGTmSKj0+YF0bfbCsM7Ek9OnaYnKLNQSi/rToDuRCtddoqmiU8wlYMgS27P4E4AfJTHhs66uEbIZBVkyRRNkje+lj6bdTNjnNcJ5c=
Return-Path: <adamkennedybackup [...] gmail.com>
Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; bh=OjZ68b/cta0bvf1LFXT0I5kcKdOMv9pMIYItDzh+OvA=; b=rkg5VROXY3rhIdrWwMBWWR3/oNzPZBN+8LHWKWfgEqa9rcxQGSt20OldCzfgSK4ph/uaDB8WrUVOA/v2RH1NGEpLN4FA/fjjGln+8qQI3J8eZsWiX/ot2qp1VJ6t8Pf8/cGe8mu8Vg0TttyHETvsygoPrDmtxzl4GXfNQfA0YI4=
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: bug-PPI [...] rt.cpan.org
Date: Wed, 3 Oct 2007 17:31:27 +1000
Message-Id: <b8cb49a40710030031l3b3b6e12wb54eed367b06e13a [...] mail.gmail.com>
To: bug-PPI [...] rt.cpan.org
From: "Adam Kennedy" <adamkennedybackup [...] gmail.com>
RT-Message-ID: <rt-3.6.HEAD-12838-1191396706-1883.29636-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: 7bit
X-RT-Original-Encoding: ISO-8859-1
X-RT-Original-Encoding: utf-8
Content-Length: 389
Download (untitled) / with headers
text/plain 389b
Thanks for doing the commited. I guess it's just very lucky timing that my repository manager was ready to release about the same time, it's been about 6 months in the making. Adam K On 03/10/2007, Chris Dolan via RT <bug-PPI@rt.cpan.org> wrote: Show quoted text
> > > Queue: PPI > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=29636 > > > I committed this patch myself. Thanks Adam! >
Content-Type: text/html; charset=ISO-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 7bit
X-RT-Original-Encoding: ISO-8859-1
X-RT-Original-Encoding: ISO-8859-1
Content-Length: 799
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Disposition: inline
Message-Id: <rt-3.6.HEAD-11194-1192454370-344.29636-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Original-Encoding: utf-8
Content-Length: 43
Applied by Chris directly to the repository


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.