]> Sergey Matveev's repositories - public-inbox.git/commitdiff
t/cgi.t: eliminate some cruft and unnecessary tests
authorEric Wong <e@80x24.org>
Fri, 4 Jan 2019 08:51:20 +0000 (08:51 +0000)
committerEric Wong <e@80x24.org>
Fri, 4 Jan 2019 08:51:20 +0000 (08:51 +0000)
More of this test will be, we use PSGI nowadays; and
most of these tests can be ported over to use PSGI and
not fork+exec as much.

t/cgi.t

diff --git a/t/cgi.t b/t/cgi.t
index d92749b3d98ec52053dab204134207ff210c4c6c..ac2c69fc8425485525e92f91ae81e90d0849eda4 100644 (file)
--- a/t/cgi.t
+++ b/t/cgi.t
@@ -48,9 +48,8 @@ my $im = PublicInbox::Import->new($git, 'test', $addr);
 {
        local $ENV{HOME} = $home;
 
-       # ensure successful message delivery
-       {
-               my $mime = Email::MIME->new(<<EOF);
+       # inject some messages:
+       my $mime = Email::MIME->new(<<EOF);
 From: Me <me\@example.com>
 To: You <you\@example.com>
 Cc: $addr
@@ -60,15 +59,10 @@ Date: Thu, 01 Jan 1970 00:00:00 +0000
 
 zzzzzz
 EOF
-               $im->add($mime);
-               $im->done;
-               my $rev = `git --git-dir=$maindir rev-list HEAD`;
-               like($rev, qr/\A[a-f0-9]{40}/, "good revision committed");
-       }
+       $im->add($mime);
 
        # deliver a reply, too
-       {
-               my $reply = Email::MIME->new(<<EOF);
+       my $reply = Email::MIME->new(<<EOF);
 From: You <you\@example.com>
 To: Me <me\@example.com>
 Cc: $addr
@@ -82,12 +76,8 @@ Me wrote:
 
 what?
 EOF
-               $im->add($reply);
-               $im->done;
-               my $rev = `git --git-dir=$maindir rev-list HEAD`;
-               like($rev, qr/\A[a-f0-9]{40}/, "good revision committed");
-       }
-
+       $im->add($reply);
+       $im->done;
 }
 
 # obvious failures, first
@@ -118,7 +108,7 @@ EOF
        like($res->{head}, qr/Status:\s*206/i, "info/refs partial past end OK");
        is($res->{body}, substr($orig, 5), 'partial body OK past end');
 }
-use Data::Dumper;
+
 # atom feeds
 {
        local $ENV{HOME} = $home;