about summary refs log tree commit homepage
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-04-20 09:01:00 +0000
committerEric Wong <e@80x24.org>2021-04-20 09:15:17 +0000
commit420fa1eac8cf0c73b6b0512c32206dc0b904dd45 (patch)
tree63b1fbcb2e566a5a887d189eaa337400526c4b07
parent48a3de066689174f65d5e12ebaff8f03f97942fb (diff)
downloadpublic-inbox-420fa1eac8cf0c73b6b0512c32206dc0b904dd45.tar.gz
lei-sigpipe: update and move test from xt => t
We have "lei import" and better test infrastructure for lei,
now, so we can more easily test SIGPIPE without relying on
an already-configured instance.
-rw-r--r--MANIFEST2
-rw-r--r--t/lei-sigpipe.t43
-rw-r--r--xt/lei-sigpipe.t58
3 files changed, 44 insertions, 59 deletions
diff --git a/MANIFEST b/MANIFEST
index f35c514c..f4a55687 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -398,6 +398,7 @@ t/lei-q-kw.t
 t/lei-q-remote-import.t
 t/lei-q-save.t
 t/lei-q-thread.t
+t/lei-sigpipe.t
 t/lei-tag.t
 t/lei.t
 t/lei_dedupe.t
@@ -501,7 +502,6 @@ xt/httpd-async-stream.t
 xt/imapd-mbsync-oimap.t
 xt/imapd-validate.t
 xt/lei-auth-fail.t
-xt/lei-sigpipe.t
 xt/mem-imapd-tls.t
 xt/mem-msgview.t
 xt/msgtime_cmp.t
diff --git a/t/lei-sigpipe.t b/t/lei-sigpipe.t
new file mode 100644
index 00000000..f84d6d22
--- /dev/null
+++ b/t/lei-sigpipe.t
@@ -0,0 +1,43 @@
+#!perl -w
+# Copyright (C) 2021 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+use strict;
+use v5.10.1;
+use PublicInbox::TestCommon;
+use POSIX qw(WTERMSIG WIFSIGNALED SIGPIPE);
+test_lei(sub {
+        my $f = "$ENV{HOME}/big.eml";
+        my $imported;
+        for my $out ([], [qw(-f mboxcl2)]) {
+                pipe(my ($r, $w)) or BAIL_OUT $!;
+                my $size = 65536;
+                if ($^O eq 'linux' && fcntl($w, 1031, 4096)) {
+                        $size = 4096;
+                }
+                unless (-f $f) {
+                        open my $fh, '>', $f or xbail "open $f: $!";
+                        print $fh <<'EOM' or xbail;
+From: big@example.com
+Message-ID: <big@example.com>
+EOM
+                        print $fh 'Subject:';
+                        print $fh (' '.('x' x 72)."\n") x (($size / 73) + 1);
+                        print $fh "\nbody\n";
+                        close $fh or xbail "close: $!";
+                }
+
+                lei_ok(qw(import), $f) if $imported++ == 0;
+                open my $errfh, '>>', "$ENV{HOME}/stderr.log" or xbail $!;
+                my $opt = { run_mode => 0, 2 => $errfh, 1 => $w };
+                my $cmd = [qw(lei q -q -t), @$out, 'z:1..'];
+                my $tp = start_script($cmd, undef, $opt);
+                close $w;
+                is(sysread($r, my $buf, 1), 1, 'read one byte');
+                close $r; # trigger SIGPIPE
+                $tp->join;
+                ok(WIFSIGNALED($?), "signaled @$out");
+                is(WTERMSIG($?), SIGPIPE, "got SIGPIPE @$out");
+        }
+});
+
+done_testing;
diff --git a/xt/lei-sigpipe.t b/xt/lei-sigpipe.t
deleted file mode 100644
index 44020bad..00000000
--- a/xt/lei-sigpipe.t
+++ /dev/null
@@ -1,58 +0,0 @@
-#!perl -w
-# Copyright (C) 2021 all contributors <meta@public-inbox.org>
-# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
-use strict;
-use v5.10.1;
-use Test::More;
-use PublicInbox::TestCommon;
-use POSIX qw(WTERMSIG WIFSIGNALED SIGPIPE);
-require_mods(qw(json DBD::SQLite Search::Xapian));
-# XXX this needs an already configured lei instance with many messages
-
-my $do_test = sub {
-        my $env = shift // {};
-        for my $out ([], [qw(-f mboxcl2)]) {
-                pipe(my ($r, $w)) or BAIL_OUT $!;
-                open my $err, '+>', undef or BAIL_OUT $!;
-                my $opt = { run_mode => 0, 1 => $w, 2 => $err };
-                my $cmd = [qw(lei q -q -t), @$out, 'z:1..'];
-                my $tp = start_script($cmd, $env, $opt);
-                close $w;
-                sysread($r, my $buf, 1);
-                close $r; # trigger SIGPIPE
-                $tp->join;
-                ok(WIFSIGNALED($?), "signaled @$out");
-                is(WTERMSIG($?), SIGPIPE, "got SIGPIPE @$out");
-                seek($err, 0, 0);
-                my @err = grep(!m{mkdir /dev/null\b}, <$err>);
-                is_deeply(\@err, [], "no errors @$out");
-        }
-};
-
-my ($tmp, $for_destroy) = tmpdir();
-my $pid;
-my $opt = { run_mode => 0, 1 => \(my $out = '') };
-if (run_script([qw(lei daemon-pid)], undef, $opt)) {
-        chomp($pid = $out);
-        mkdir "$tmp/d" or BAIL_OUT $!;
-        local $ENV{TMPDIR} = "$tmp/d";
-        $do_test->();
-        $out = '';
-        ok(run_script([qw(lei daemon-pid)], undef, $opt), 'daemon-pid again');
-        chomp($out);
-        is($out, $pid, 'daemon-pid unchanged');
-        ok(kill(0, $pid), 'daemon still running');
-        $out = '';
-}
-{
-        mkdir "$tmp/1" or BAIL_OUT $!;
-        local $ENV{TMPDIR} = "$tmp/1";
-        $do_test->({XDG_RUNTIME_DIR => '/dev/null'});
-        is(unlink(glob("$tmp/1/*")), 0, 'nothing left over w/ oneshot');
-}
-
-# the one-shot test should be slow enough that the daemon has cleaned
-# up in the background:
-is_deeply([glob("$tmp/d/*")], [], 'nothing left over with daemon');
-
-done_testing;