-#!perl\r
-\r
-# This test tests the standalone server's auto-restart feature.\r
-\r
-use strict;\r
-use warnings;\r
-\r
-use File::Path;\r
-use FindBin;\r
-use LWP::Simple;\r
-use IO::Socket;\r
-use Test::More;\r
-use Time::HiRes qw/sleep/;\r
-eval "use File::Copy::Recursive";\r
-\r
-plan skip_all => 'set TEST_HTTP to enable this test' unless $ENV{TEST_HTTP};\r
-plan skip_all => 'File::Copy::Recursive required' if $@;\r
-\r
-plan tests => 40;\r
-\r
-# clean up\r
-rmtree "$FindBin::Bin/../t/tmp" if -d "$FindBin::Bin/../t/tmp";\r
-\r
-# create a TestApp and copy the test libs into it\r
-mkdir "$FindBin::Bin/../t/tmp";\r
-chdir "$FindBin::Bin/../t/tmp";\r
-system\r
- "perl -I$FindBin::Bin/../lib $FindBin::Bin/../script/catalyst.pl TestApp";\r
-chdir "$FindBin::Bin/..";\r
-File::Copy::Recursive::dircopy( 't/lib', 't/tmp/TestApp/lib' );\r
-\r
-# remove TestApp's tests\r
-rmtree 't/tmp/TestApp/t';\r
-\r
-# spawn the standalone HTTP server\r
-my $port = 30000 + int rand( 1 + 10000 );\r
-my $pid = open my $server,\r
-"perl -I$FindBin::Bin/../lib $FindBin::Bin/../t/tmp/TestApp/script/testapp_server.pl -port $port -restart 2>&1 |"\r
- or die "Unable to spawn standalone HTTP server: $!";\r
-\r
-# wait for it to start\r
-print "Waiting for server to start...\n";\r
-while ( check_port( 'localhost', $port ) != 1 ) {\r
- sleep 1;\r
-}\r
-\r
-# change various files\r
-my @files = (\r
- "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp.pm",\r
- "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Action/Begin.pm",\r
-"$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Engine/Request/URI.pm",\r
-);\r
-\r
-# change some files and make sure the server restarts itself\r
-for ( 1 .. 20 ) {\r
- my $index = rand @files;\r
- open my $pm, '>>', $files[$index]\r
- or die "Unable to open $files[$index] for writing: $!";\r
- print $pm "\n";\r
- close $pm;\r
-\r
- # give the server time to notice the change and restart\r
- my $count = 0;\r
- sleep 1;\r
- while ( check_port( 'localhost', $port ) != 1 ) {\r
-\r
- # wait for it to restart\r
- sleep 0.1;\r
- die "Server appears to have died" if $count++ > 50;\r
- }\r
- my $response = get("http://localhost:$port/action/default");\r
- like( $response, qr/Catalyst::Request/, 'Non-error restart, request OK' );\r
-\r
- #print $server->getline;\r
-}\r
-\r
-# add errors to the file and make sure server does not die or restart\r
-for ( 1 .. 20 ) {\r
- my $index = rand @files;\r
- open my $pm, '>>', $files[$index]\r
- or die "Unable to open $files[$index] for writing: $!";\r
- print $pm "bleh";\r
- close $pm;\r
-\r
- # give the server time to notice the change\r
- sleep 1;\r
- if ( check_port( 'localhost', $port ) != 1 ) {\r
- die "Server appears to have died";\r
- }\r
- my $response = get("http://localhost:$port/action/default");\r
- like( $response, qr/Catalyst::Request/,\r
- 'Syntax error, no restart, request OK' );\r
-\r
- #print $server->getline;\r
-}\r
-\r
-# shut it down\r
-kill 'INT', $pid;\r
-close $server;\r
-\r
-# clean up\r
-rmtree "$FindBin::Bin/../t/tmp" if -d "$FindBin::Bin/../t/tmp";\r
-\r
-sub check_port {\r
- my ( $host, $port ) = @_;\r
-\r
- my $remote = IO::Socket::INET->new(\r
- Proto => "tcp",\r
- PeerAddr => $host,\r
- PeerPort => $port\r
- );\r
- if ($remote) {\r
- close $remote;\r
- return 1;\r
- }\r
- else {\r
- return 0;\r
- }\r
-}\r
+# This test tests the standalone server's auto-restart feature.
+
+use strict;
+use warnings;
+
+use Test::More;
+BEGIN {
+ plan skip_all => 'set TEST_HTTP to enable this test' unless $ENV{TEST_HTTP};
+}
+
+use File::Path;
+use FindBin;
+use LWP::Simple;
+use IO::Socket;
+use IPC::Open3;
+use Catalyst::Engine::HTTP::Restarter::Watcher;
+use Time::HiRes qw/sleep/;
+eval "use Catalyst::Devel 1.0;";
+
+plan skip_all => 'Catalyst::Devel required' if $@;
+plan skip_all => 'Catalyst::Devel >= 1.04 required' if $Catalyst::Devel::VERSION <= 1.03;
+eval "use File::Copy::Recursive";
+plan skip_all => 'File::Copy::Recursive required' if $@;
+
+plan tests => 120;
+
+my $tmpdir = "$FindBin::Bin/../t/tmp";
+
+# clean up
+rmtree $tmpdir if -d $tmpdir;
+
+# create a TestApp and copy the test libs into it
+mkdir $tmpdir;
+chdir $tmpdir;
+
+system( $^X, "-I$FindBin::Bin/../lib", "$FindBin::Bin/../script/catalyst.pl", 'TestApp' );
+
+chdir "$FindBin::Bin/..";
+File::Copy::Recursive::dircopy( 't/lib', 't/tmp/TestApp/lib' );
+
+# remove TestApp's tests
+rmtree 't/tmp/TestApp/t';
+
+# spawn the standalone HTTP server
+my $port = 30000 + int rand( 1 + 10000 );
+
+my( $server, $pid );
+$pid = open3( undef, $server, undef,
+ $^X, "-I$FindBin::Bin/../lib",
+ "$FindBin::Bin/../t/tmp/TestApp/script/testapp_server.pl", '-port',
+ $port, '-restart' )
+ or die "Unable to spawn standalone HTTP server: $!";
+
+# switch to non-blocking reads so we can fail
+# gracefully instead of just hanging forever
+
+$server->blocking( 0 );
+
+# wait for it to start
+print "Waiting for server to start...\n";
+while ( check_port( 'localhost', $port ) != 1 ) {
+ sleep 1;
+}
+
+# change various files
+my @files = (
+ "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp.pm",
+ "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Action/Begin.pm",
+ "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Immutable.pm",
+);
+
+push(@files, "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Immutable/HardToReload.pm")
+ if Catalyst::Engine::HTTP::Restarter::Watcher::DETECT_PACKAGE_COMPILATION();
+
+# change some files and make sure the server restarts itself
+NON_ERROR_RESTART:
+for ( 1 .. 20 ) {
+ my $index = rand @files;
+ open my $pm, '>>', $files[$index]
+ or die "Unable to open $files[$index] for writing: $!";
+ print $pm "\n";
+ close $pm;
+
+ # give the server time to notice the change and restart
+ my $count = 0;
+ my $line;
+
+ while ( ( $line || '' ) !~ /can connect/ ) {
+ # wait for restart message
+ $line = $server->getline;
+ sleep 0.1;
+ if ( $count++ > 100 ) {
+ fail "Server restarted";
+ SKIP: {
+ skip "Server didn't restart, no sense in checking response", 1;
+ }
+ next NON_ERROR_RESTART;
+ }
+ };
+ pass "Server restarted";
+
+ $count = 0;
+ while ( check_port( 'localhost', $port ) != 1 ) {
+ # wait for it to restart
+ sleep 0.1;
+ die "Server appears to have died" if $count++ > 100;
+ }
+ my $response = get("http://localhost:$port/action/default");
+ like( $response, qr/Catalyst::Request/, 'Non-error restart, request OK' );
+
+ # give the server some time to reindex its files
+ sleep 1;
+}
+
+# add errors to the file and make sure server does not die or restart
+NO_RESTART_ON_ERROR:
+for ( 1 .. 20 ) {
+ my $index = rand @files;
+ open my $pm, '>>', $files[$index]
+ or die "Unable to open $files[$index] for writing: $!";
+ print $pm "bleh";
+ close $pm;
+
+ my $count = 0;
+ my $line;
+
+ while ( ( $line || '' ) !~ /failed/ ) {
+ # wait for restart message
+ $line = $server->getline;
+ sleep 0.1;
+ if ( $count++ > 100 ) {
+ fail "Server restarted";
+ SKIP: {
+ skip "Server didn't restart, no sense in checking response", 1;
+ }
+ next NO_RESTART_ON_ERROR;
+ }
+ };
+
+ pass "Server refused to restart";
+
+ if ( check_port( 'localhost', $port ) != 1 ) {
+ die "Server appears to have died";
+ }
+ my $response = get("http://localhost:$port/action/default");
+ like( $response, qr/Catalyst::Request/,
+ 'Syntax error, no restart, request OK' );
+
+ # give the server some time to reindex its files
+ sleep 1;
+
+}
+
+# multiple restart directories
+
+# we need different options so we have to rebuild most
+# of the testing environment
+
+kill 'KILL', $pid;
+close $server;
+
+# pick next port because the last one might still be blocked from
+# previous server. This might fail if this port is unavailable
+# but picking the first one has the same problem so this is acceptable
+
+$port += 1;
+
+{ no warnings 'once'; $File::Copy::Recursive::RMTrgFil = 1; }
+File::Copy::Recursive::dircopy( 't/lib', 't/tmp/TestApp/lib' );
+
+# change various files
+@files = (
+ "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Action/Begin.pm",
+ "$FindBin::Bin/../t/tmp/TestApp/lib/TestApp/Controller/Engine/Request/URI.pm",
+);
+
+my $app_root = "$FindBin::Bin/../t/tmp/TestApp";
+my $restartdirs = join ' ', map{
+ "-restartdirectory $app_root/lib/TestApp/Controller/$_"
+} qw/Action Engine/;
+
+$pid = open3( undef, $server, undef,
+ $^X, "-I$FindBin::Bin/../lib",
+ "$FindBin::Bin/../t/tmp/TestApp/script/testapp_server.pl", '-port',
+ $port, '-restart', $restartdirs )
+ or die "Unable to spawn standalone HTTP server: $!";
+$server->blocking( 0 );
+
+
+# wait for it to start
+print "Waiting for server to start...\n";
+while ( check_port( 'localhost', $port ) != 1 ) {
+ sleep 1;
+}
+
+MULTI_DIR_RESTART:
+for ( 1 .. 20 ) {
+ my $index = rand @files;
+ open my $pm, '>>', $files[$index]
+ or die "Unable to open $files[$index] for writing: $!";
+ print $pm "\n";
+ close $pm;
+
+ # give the server time to notice the change and restart
+ my $count = 0;
+ my $line;
+
+ while ( ( $line || '' ) !~ /can connect/ ) {
+ # wait for restart message
+ $line = $server->getline;
+ sleep 0.1;
+ if ( $count++ > 100 ) {
+ fail "Server restarted";
+ SKIP: {
+ skip "Server didn't restart, no sense in checking response", 1;
+ }
+ next MULTI_DIR_RESTART;
+ }
+ };
+ pass "Server restarted with multiple restartdirs";
+
+ $count = 0;
+ while ( check_port( 'localhost', $port ) != 1 ) {
+ # wait for it to restart
+ sleep 0.1;
+ die "Server appears to have died" if $count++ > 100;
+ }
+ my $response = get("http://localhost:$port/action/default");
+ like( $response, qr/Catalyst::Request/, 'Non-error restart, request OK' );
+
+ # give the server some time to reindex its files
+ sleep 1;
+}
+
+# shut it down again
+
+kill 'KILL', $pid;
+close $server;
+
+# clean up
+rmtree "$FindBin::Bin/../t/tmp" if -d "$FindBin::Bin/../t/tmp";
+
+sub check_port {
+ my ( $host, $port ) = @_;
+
+ my $remote = IO::Socket::INET->new(
+ Proto => "tcp",
+ PeerAddr => $host,
+ PeerPort => $port
+ );
+ if ($remote) {
+ close $remote;
+ return 1;
+ }
+ else {
+ return 0;
+ }
+}