Bug #59246 for WWW-Mechanize-Shell: testsuite fails in check-dumpresponse

Sat Jul 10 04:14:23 2010 http://www.google.com/profiles/reltub.oel - Ticket created

Subject:	testsuite fails in check-dumpresponse
MIME-Version:	1.0
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
X-RT-Original-Encoding:	utf-8
Content-Type:	multipart/mixed; boundary="----------=_1278749663-2371-12"
Content-Length:	0

Content-Type:	text/plain; charset="UTF-8"
Content-Disposition:	inline
Content-Transfer-Encoding:	binary
Content-Length:	381

Data: WWW::Mechanize::Shell v 0.48 WWW::Mechanize v 1.64 LWP v 5.836 perl --version This is perl, v5.10.1 (*) built for i486-linux-gnu-thread-multi (with 45 registered patches, see perl -V for more detail) uname -a Linux squeeeze 2.6.32-3-686 #1 SMP Thu Feb 25 06:14:20 UTC 2010 i686 GNU/Linux See the attached file for the complete test output, including where the test fails.

Subject:	www-mechanize-shell-bug-report
MIME-Version:	1.0
Content-Type:	application/octet-stream; name="www-mechanize-shell-bug-report"
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline; filename="www-mechanize-shell-bug-report"
Content-Transfer-Encoding:	base64
Content-Length:	4328

www-mechanize-shell-bug-report

Message body not shown because it is not plain text.

Sun Aug 01 07:00:18 2010 CARNIL [...] cpan.org - Cc CARNIL [...] cpan.org added

Thu Aug 05 07:38:28 2010 ANSGAR [...] cpan.org - Correspondence added

From ansgar@43-1.org Thu Aug 5 07:	36:20 2010
MIME-Version:	1.0
X-Spam-Status:	No, score=-9.999 tagged_above=-99.9 required=10 tests=[AWL=-0.086, BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, SPF_NEUTRAL=0.686] autolearn=ham
X-Spam-Flag:	NO
X-Virus-Checked:	Checked by ClamAV on 16.mx.develooper.com
X-Virus-Scanned:	Debian amavisd-new at bestpractical.com
Message-ID:	<87pqxx5ly1.fsf@marvin.43-1.org>
Content-Type:	multipart/mixed; boundary="=-=-="
X-Spam-Score:	-9.999
Received:	from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id E794E240A82 for <cpan-bug+WWW-Mechanize-Shell@hipster.bestpractical.com>; Thu, 5 Aug 2010 07:36:20 -0400 (EDT)
Received:	from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6yxUgjOOBLCG for <cpan-bug+WWW-Mechanize-Shell@hipster.bestpractical.com>; Thu, 5 Aug 2010 07:36:19 -0400 (EDT)
Received:	from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id B23AF240A7F for <bug-WWW-Mechanize-Shell@rt.cpan.org>; Thu, 5 Aug 2010 07:36:18 -0400 (EDT)
Received:	(qmail 6475 invoked by uid 103); 5 Aug 2010 11:38:24 -0000
Received:	from x16.dev (10.0.100.26) by x1.dev with QMQP; 5 Aug 2010 11:38:24 -0000
Received:	from tsukuyomi.43-1.org (HELO tsukuyomi.43-1.org) (188.40.248.50) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Thu, 05 Aug 2010 04:38:23 -0700
Received:	from marvin.43-1.org (202-71-92-76.ap-w01.canvas.ne.jp [202.71.92.76]) by tsukuyomi.43-1.org (Postfix) with ESMTPS id 8BE7820D21C4 for <bug-WWW-Mechanize-Shell@rt.cpan.org>; Thu, 5 Aug 2010 13:38:17 +0200 (CEST)
Received:	from ansgar by marvin.43-1.org with local (Exim 4.72) (envelope-from <ansgar@43-1.org>) id 1Ogym6-0001l6-QJ for bug-WWW-Mechanize-Shell@rt.cpan.org; Thu, 05 Aug 2010 20:38:14 +0900
Delivered-To:	cpan-bug+WWW-Mechanize-Shell@hipster.bestpractical.com
Subject:	[rt.cpan.org #59246] [PATCH] testsuite fails in check-dumpresponse
User-Agent:	Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux)
Return-Path:	<ansgar@43-1.org>
X-Spam-Check-BY:	16.mx.develooper.com
X-Original-To:	cpan-bug+WWW-Mechanize-Shell@hipster.bestpractical.com
X-RT-Mail-Extension:	www-mechanize-shell
Date:	Thu, 05 Aug 2010 20:38:14 +0900
X-Spam-Level:
To:	bug-WWW-Mechanize-Shell@rt.cpan.org
From:	Ansgar Burchardt <ANSGAR@cpan.org>
RT-Message-ID:	<rt-3.8.HEAD-2998-1281008308-1960.59246-0-0@rt.cpan.org>
Content-Length:	0

Content-Type:	text/plain; charset="utf-8"
X-RT-Original-Encoding:	utf-8
Content-Length:	460

Hi, I found out why the test fails: LWP::UserAgent::request seems to be called in list context now. $_[-1] as passed to the post wrapper function by Hook::LexWrapper then contains an ArrayRef instead of the object. I prepared a patch to change the response_dumper function to also work when passed an array. Please consider applying it for the next release. There is also a second patch fixing a smell spelling error in the documentation. Regards, Ansgar

Content-Type:	text/x-diff
content-disposition:	attachment; filename="response-dumper.patch"
Content-Length:	795

response-dumper.patch

Message body is not shown because sender requested not to inline it.

Content-Type:	text/x-diff
content-disposition:	attachment; filename="spelling.patch"
Content-Length:	576

spelling.patch

Message body is not shown because sender requested not to inline it.

Thu Aug 05 07:38:29 2010 The RT System itself - Status changed from 'new' to 'open'

Thu Aug 05 07:52:30 2010 ANSGAR [...] cpan.org - Cc ANSGAR [...] cpan.org added

Tue Aug 17 16:32:49 2010 corion [...] cpan.org - Correspondence added

MIME-Version:	1.0
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline
Content-Type:	text/plain; charset="UTF-8"
Message-ID:	<rt-3.8.HEAD-11062-1282077169-513.59246-0-0@rt.cpan.org>
Content-Transfer-Encoding:	binary
X-RT-Original-Encoding:	utf-8
Content-Length:	49

Fixed with 0.49 Many thanks for the patch! -max

Tue Aug 17 16:32:50 2010 corion [...] cpan.org - Status changed from 'open' to 'resolved'