Bug #27123 for Catalyst-Runtime: Certain tests require Catalyst::Devel

Mon May 14 11:37:24 2007 cweyl [...] alumni.drew.edu - Ticket created

Subject:	Certain tests require Catalyst::Devel
Date:	Mon, 14 May 2007 08:36:00 -0700
To:	bug-Catalyst-Runtime [...] rt.cpan.org
From:	"Chris Weyl" <cweyl [...] alumni.drew.edu>

Certain tests in the Catalyst::Runtime suite require the presence of Catalyst::Devel. Given that Catalyst::Devel requires Catalyst::Runtime, to fully test and avoid a circular dependency loop, would it make sense to move these tests to the Catalyst::Devel package? t/optional_http-server-restart..............................skipped all skipped: Catalyst::Devel required t/optional_http-server......................................skipped all skipped: Catalyst::Devel required t/optional_lighttpd-fastcgi-non-root........................skipped all skipped: Catalyst::Devel required t/optional_lighttpd-fastcgi.................................skipped all skipped: Catalyst::Devel required -- Chris Weyl Ex astris, scientia

Mon May 14 11:43:25 2007 AGRUNDMA [...] cpan.org - Correspondence added

From:

AGRUNDMA [...] cpan.org

Those tests are all optional and are intended only for developers, so it doesn't really matter that they require Catalyst::Devel.

Mon May 14 11:43:27 2007 The RT System itself - Status changed from 'new' to 'open'

Mon May 14 11:48:49 2007 cweyl [...] alumni.drew.edu - Correspondence added

Subject:	Re: [rt.cpan.org #27123] Certain tests require Catalyst::Devel
Date:	Mon, 14 May 2007 08:47:47 -0700
To:	bug-Catalyst-Runtime [...] rt.cpan.org
From:	"Chris Weyl" <cweyl [...] alumni.drew.edu>

On 5/14/07, via RT <bug-Catalyst-Runtime@rt.cpan.org> wrote: Show quoted text

> > <URL: http://rt.cpan.org/Ticket/Display.html?id=27123 > > > Those tests are all optional and are intended only for developers, so it > doesn't really matter that they require Catalyst::Devel.

Gotcha. So (just being pedantic here) they shouldn't be considered part of the "standard" test suite that should always be run on package build? Thanks- -Chris -- Chris Weyl Ex astris, scientia

Mon May 14 11:57:16 2007 AGRUNDMA [...] cpan.org - Correspondence added

Right, that's why there is code in each optional test to require an environment variable to run them.

Thu Sep 06 13:56:37 2007 bricas [...] cpan.org - Status changed from 'open' to 'resolved'