warn pre-start() errors
[perl/plp/.git] / t / 50-cgi.t
index 0de69407b3798b2ae275747195bf9cb8847bf6cf..bf60143f59edf38044528cefba344d8e80e27522 100644 (file)
@@ -1,39 +1,43 @@
 use strict;
 use warnings;
 
-use Cwd qw(cwd);
+use File::Spec;
 use Test::More;
 
 eval { require PerlIO::scalar };
 plan skip_all => "PerlIO required (perl 5.8) to test PLP" if $@;
 
-plan tests => 18;
+plan tests => 20;
 
 require_ok('PLP::Backend::CGI') or BAIL_OUT();
 
 $PLP::use_cache = 0 if $PLP::use_cache;
 #TODO: caching on (change file names)
 
-my $base = -w '/tmp' ? '/tmp' : cwd();
+my $base = File::Spec->tmpdir || File::Spec->curdir;
+-w $base or BAIL_OUT("$base not writable");
 my $testfile = 'testfile.plp';
 not -f "$base/$testfile" or BAIL_OUT("$testfile exists");
 
-open ORGOUT, '>&', STDOUT;
+open ORGOUT, '>&', *STDOUT;
 
 sub plp_is {
        my ($test, $plp, $expect) = @_;
        chomp $expect;
        local $Test::Builder::Level = $Test::Builder::Level + 1;
 
-       eval {
-               open my $testfh, '>', "$base/$testfile" or die $!;
-               print {$testfh} $plp or die $!;
-               close $testfh or die $!;
-       };
-       not $@ or fail("write $testfile"), diag("    Error: $@"), return;
+       if (defined $plp) {
+               eval {
+                       open my $testfh, '>', "$base/$testfile" or die $!;
+                       print {$testfh} $plp or die $!;
+                       close $testfh or die $!;
+               };
+               not $@ or fail("write $testfile"), diag("    Error: $@"), return;
+       }
 
        close STDOUT;
        open STDOUT, '>', \my $output;  # STDOUT buffered to scalar
+       select STDOUT;  # output before start() (which selects PLPOUT)
        eval {
                local $SIG{__WARN__} = sub { print $_[0] }; # enables warnings
                PLP::everything();
@@ -150,9 +154,6 @@ plp_is('$PLP::ERROR',
        qq{$HEAD\n Oh no: $INCWARN at $testfile line 1.\n\n}
 );
 
-#TODO: 404
-#TODO: 403
-
 plp_is('$PLP::DEBUG',
        '<: $PLP::DEBUG = 2; delete $header{x_plp_version} :>1<(missinginclude)>2',
        "Content-Type: text/plain\n\nContent-Type: text/html\n\n1"
@@ -174,6 +175,41 @@ my @envtest = (
 
 plp_is('%ENV (on apache)', @envtest);
 
+SKIP: {
+chmod 0244, $testfile or skip("changed permissions", 1);
+plp_is('permission denied', undef, <<TEST);
+Content-Type: text/html
+PLP: Can't read: $base/$testfile (/$testfile/test/123)
+Status: 403
+
+<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
+<html><head>
+<title>403 Forbidden</title>
+</head></body>
+<h1>Forbidden</h1>
+You don't have permission to access /$testfile/test/123 on this server.<p>
+<hr>
+</body></html>
+TEST
+chmod 0644, $testfile;
+}
+
+$ENV{PATH_TRANSLATED} = "$base/missinginclude/test/123";
+plp_is('not found', undef, <<TEST);
+Content-Type: text/html
+PLP: Not found: $base/missinginclude/test/123 (/$testfile/test/123)
+Status: 404
+
+<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
+<html><head>
+<title>404 Not Found</title>
+</head></body>
+<h1>Not Found</h1>
+The requested URL /$testfile/test/123 was not found on this server.<p>
+<hr>
+</body></html>
+TEST
+
 %ENV = (
        REQUEST_METHOD => 'GET',
        REQUEST_URI => "/$testfile/test/123",