Bug #22619 for XML-Twig: [PATCH] test_3_26.t fails without LWP::Simple installed

Thu Oct 26 14:54:53 2006 ernest.argetsinger [...] omnimedix.org - Ticket created

Subject:

[PATCH] test_3_26.t fails without LWP::Simple installed

When running make test without LWP installed, get the following failure: t/test_3_26.........................ok 11/13Undefined subroutine &XML::Twig::get called at <path to Show quoted text

build>/XML-Twig-3.26/blib/lib/XML/Twig.pm line 793.

Failure is due to LWP::Simple being required for parse_uri function. Patch attached will match this test file behavior to other test files, and skip the test if LWP::Simple is not installed.

Subject:

test_3_26.t.patch_diff

Download test_3_26.t.patch_diff
application/octet-stream 620b

Message body not shown because it is not plain text.

Fri Oct 27 10:40:37 2006 mirod [...] xmltwig.com - Correspondence added

Subject:	Re: [rt.cpan.org #22619] [PATCH] test_3_26.t fails without LWP::Simple installed
Date:	Fri, 27 Oct 2006 16:40:41 +0200
To:	bug-XML-Twig [...] rt.cpan.org
From:	Michel Rodriguez <mirod [...] xmltwig.com>

Ernest Argetsinger via RT wrote: Show quoted text

> Thu Oct 26 14:54:53 2006: Request 22619 was acted upon. > Transaction: Ticket created by arget > Queue: XML-Twig > Subject: [PATCH] test_3_26.t fails without LWP::Simple installed > Broken in: 3.26 > Severity: Normal > Owner: Nobody > Requestors: ernest.argetsinger@omnimedix.org > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=22619 > > > > When running make test without LWP installed, get the following failure: > t/test_3_26.........................ok 11/13Undefined subroutine > &XML::Twig::get called at <path to

> build>/XML-Twig-3.26/blib/lib/XML/Twig.pm line 793.

> > Failure is due to LWP::Simple being required for parse_uri function. > Patch attached will match this test file behavior to other test files, > and skip the test if LWP::Simple is not installed.

I believe this is already fixed in the development version, you can get it at http://xmltwig.com/xmltwig/XML-Twig-3.27.tar.gz if you want to test it. Thanks -- mirod

Fri Oct 27 10:40:39 2006 The RT System itself - Status changed from 'new' to 'open'

Fri Oct 27 13:09:25 2006 ernest.argetsinger [...] omnimedix.org - Correspondence added

From:

ernest.argetsinger [...] omnimedix.org

I am still seeing failed tests in the 3.27 version. t/test_3_26.........................ok 16/18Undefined subroutine &XML::Twig::get called at <path to Show quoted text

build>/XML-Twig-3.27/blib/lib/XML/Twig.pm line 853.

parse uri requires LWP::Simple. On Fri Oct 27 10:40:37 2006, mirod@xmltwig.com wrote: Show quoted text

> > I believe this is already fixed in the development version, you can get > it at http://xmltwig.com/xmltwig/XML-Twig-3.27.tar.gz if you want to > test it. > > Thanks >

Fri Oct 27 16:25:27 2006 mirod [...] xmltwig.com - Correspondence added

Subject:	Re: [rt.cpan.org #22619] [PATCH] test_3_26.t fails without LWP::Simple installed
Date:	Fri, 27 Oct 2006 22:25:54 +0200
To:	bug-XML-Twig [...] rt.cpan.org
From:	Michel Rodriguez <mirod [...] xmltwig.com>

Ernest Argetsinger via RT wrote: Show quoted text

> Queue: XML-Twig > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=22619 > > > I am still seeing failed tests in the 3.27 version. > > t/test_3_26.........................ok 16/18Undefined subroutine > &XML::Twig::get called at <path to

> build>/XML-Twig-3.27/blib/lib/XML/Twig.pm line 853.

> > parse uri requires LWP::Simple.

Thanks, I'll fix that tomorrow. -- mirod

Tue Nov 07 04:53:37 2006 mirod [...] xmltwig.com - Correspondence added

Subject:	Re: [rt.cpan.org #22619] [PATCH] test_3_26.t fails without LWP::Simple installed
Date:	Tue, 07 Nov 2006 10:53:33 +0100
To:	bug-XML-Twig [...] rt.cpan.org
From:	Michel Rodriguez <mirod [...] xmltwig.com>

Ernest Argetsinger via RT wrote: Show quoted text

> Queue: XML-Twig > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=22619 > > > I am still seeing failed tests in the 3.27 version. > > t/test_3_26.........................ok 16/18Undefined subroutine > &XML::Twig::get called at <path to

> build>/XML-Twig-3.27/blib/lib/XML/Twig.pm line 853.

OK, let's try it again. Sorry for the delay. BTW, this is a problem with the test, not with the module, so you can ignore the error message and install anyway. -- mirod

Wed Nov 08 16:33:59 2006 ernest.argetsinger [...] omnimedix.org - Correspondence added

From:

ernest.argetsinger [...] omnimedix.org

Thanks! The latest 3.27 tests are working for me now. Having the tests pass helps with scripted/automated installs. I appreciate the change. e. On Tue Nov 07 04:53:37 2006, mirod@xmltwig.com wrote: Show quoted text

> > OK, let's try it again. Sorry for the delay. > > BTW, this is a problem with the test, not with the module, so you can > ignore the error message and install anyway. >

Mon Jan 22 04:46:14 2007 MIROD [...] cpan.org - Status changed from 'open' to 'resolved'

Mon Jan 22 04:46:15 2007 MIROD [...] cpan.org - Fixed in 3.28 added