Skip Menu |

This queue is for tickets about the POE CPAN distribution.

Report information
The Basics
Id: 33718
Status: resolved
Worked: 20 min
Priority: 0/
Queue: POE

People
Owner: Nobody in particular
Requestors: pravus [...] cpan.org
Cc:
AdminCc:

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



Subject: Patch to re-work exit in t/90_regression/cfedde-filter-httpd.t
t/90_regression/cfedde-filter-httpd.t attempts to exit within a delayed event. This patch replaces that exit by shutting down the necessary components. I'm not sure what this will fix, but it seems more proper as Test::More appears to override the exit status.
Subject: cfedde-filter-httpd.t.patch
Index: t/90_regression/cfedde-filter-httpd.t =================================================================== --- t/90_regression/cfedde-filter-httpd.t (revision 2275) +++ t/90_regression/cfedde-filter-httpd.t (working copy) @@ -32,6 +32,7 @@ # POE::Component::Server::TCP->new( + Alias => 's0', Port => 0, ClientFilter => 'POE::Filter::HTTPD', Started => sub { @@ -48,6 +49,7 @@ ); POE::Component::Client::TCP->new ( + Alias => 'c0', RemoteAddress => '127.0.0.1', RemotePort => $port, ServerInput => sub { @@ -56,6 +58,7 @@ ); POE::Component::Client::TCP->new ( + Alias => 'c1', RemoteAddress => '127.0.0.1', RemotePort => $port, Connected => sub { @@ -73,7 +76,8 @@ $_[KERNEL]->delay_add( done => 3 ); }, done => sub { - exit 1; + $_[KERNEL]->post( $_ => 'shutdown' ) + for qw/ s0 c0 c1 /; } } );
committed this patch to https://poe.svn.sourceforge.net/svnroot/poe/trunk/poe at Revision: 2275
See previous comment.