X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=t%2F07_bugreport.t;h=5dfca0507772dd0c916a50dfbbf7224194bd5cd1;hb=07309ad3fbc3f3d29b0da8921716392b8e593eb0;hp=18ccbc5c9cc472dbf759da64d87e74bac9416d41;hpb=343464a69a83b5b2f1e956386482baaaccbc835c;p=debbugs.git diff --git a/t/07_bugreport.t b/t/07_bugreport.t index 18ccbc5..5dfca05 100644 --- a/t/07_bugreport.t +++ b/t/07_bugreport.t @@ -1,7 +1,7 @@ # -*- mode: cperl;-*- -use Test::More tests => 3; +use Test::More tests => 18; use warnings; use strict; @@ -19,6 +19,7 @@ use Cwd qw(getcwd); use Debbugs::MIME qw(create_mime_message); use File::Basename qw(dirname basename); use Test::WWW::Mechanize; +use HTTP::Status qw(RC_NOT_MODIFIED); # The test functions are placed here to make things easier use lib qw(t/lib); use DebbugsTest qw(:all); @@ -58,11 +59,27 @@ EOF # start up an HTTP::Server::Simple my $bugreport_cgi_handler = sub { - # I do not understand why this is necessary. - $ENV{DEBBUGS_CONFIG_FILE} = "$config{config_dir}/debbugs_config"; - my $content = qx(perl -I. -T cgi/bugreport.cgi); - $content =~ s/^\s*Content-Type:[^\n]+\n*//si; - print $content; + # I do not understand why this is necessary. + $ENV{DEBBUGS_CONFIG_FILE} = "$config{config_dir}/debbugs_config"; + my $fh; + open($fh,'-|',-e './cgi/version.cgi'? 'perl -I. -T ./cgi/bugreport.cgi' : 'perl -I. -T ../cgi/bugreport.cgi'); + my $headers; + my $status = 200; + while (<$fh>) { + if (/^\s*$/ and $status) { + print "HTTP/1.1 $status OK\n"; + print $headers; + $status = 0; + print $_; + } elsif ($status) { + $headers .= $_; + if (/^Status:\s*(\d+)/i) { + $status = $1; + } + } else { + print $_; + } + } }; my $port = 11342; @@ -76,5 +93,71 @@ $mech->get_ok('http://localhost:'.$port.'/?bug=1', 'Page received ok'); ok($mech->content() =~ qr/\\#1.+Submitting a bug/i, 'Title of bug is submitting a bug'); +my $etag = $mech->response->header('Etag'); +$mech->get('http://localhost:'.$port.'/?bug=1', + 'If-None-Match' => $etag); +is($mech->res->code, RC_NOT_MODIFIED, + 'Not modified when the same ETag sent for bug'); + +$mech->get_ok('http://localhost:'.$port.'/?bug=1;mbox=yes', + 'Page received ok'); +ok($mech->content() =~ qr/Subject: Submitting a bug/i, + 'Subject of bug maibox is right'); +ok($mech->content() =~ qr/^From /m, + 'Starts with a From appropriately'); + +$mech->get_ok('http://localhost:'.$port.'/?bug=1;mboxmaint=yes', + 'Page received ok'); +ok($mech->content() !~ qr/[\x01\x02\x03\x05\x06\x07]/i, + 'No unescaped states'); +$etag = $mech->response->header('Etag'); +$mech->get('http://localhost:'.$port.'/?bug=1;mboxmaint=yes', + 'If-None-Match' => $etag); +is($mech->res->code, RC_NOT_MODIFIED, + 'Not modified when the same ETag sent for bug maintmbox'); + +# now test the output of some control commands +my @control_commands = + ( + reassign_foo => {command => 'reassign', + value => 'bar', + regex => qr{bug reassigned from package 'foo' to 'bar}, + }, + forwarded_foo => {command => 'forwarded', + value => 'https://foo.invalid/bugs?id=1', + regex => qr{Set bug forwarded-to-address to 'https://foo\.invalid/bugs\?id=1'\.}, + }, + forwarded_foo_2 => {command => 'forwarded', + value => 'https://foo.example/bugs?id=1', + regex => qr{Changed bug forwarded-to-address to 'https://foo\.example/bugs\?id=1' from 'https://foo\.invalid/bugs\?id=1'\.}, + }, + clone => {command => 'clone', + value => '-1', + regex => qr{Bug 1 cloned as bug 2}, + }, + ); + +while (my ($command,$control_command) = splice(@control_commands,0,2)) { + # just check to see that control doesn't explode + $control_command->{value} = " $control_command->{value}" if length $control_command->{value} + and $control_command->{value} !~ /^\s/; + send_message(to => 'control@bugs.something', + headers => [To => 'control@bugs.something', + From => 'foo@bugs.something', + Subject => "Munging a bug with $command", + ], + body => <{command} 1$control_command->{value} +thanks +EOF + ; + # Now test that the output has changed accordingly + $mech->get_ok('http://localhost:'.$port.'/?bug=1', + 'Page received ok'); + like($mech->content(), $control_command->{regex}, + 'Page matches regex'); +} # Other tests for bugs in the page should be added here eventually +