unofficial mirror of meta@public-inbox.org
 help / color / mirror / Atom feed
From: Eric Wong <e@80x24.org>
To: meta@public-inbox.org
Subject: [PATCH] lei-sigpipe: update and move test from xt => t
Date: Tue, 20 Apr 2021 07:17:26 -0200	[thread overview]
Message-ID: <20210420091726.69835-1-e@80x24.org> (raw)

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.
---
 MANIFEST         |  2 +-
 t/lei-sigpipe.t  | 43 +++++++++++++++++++++++++++++++++++
 xt/lei-sigpipe.t | 58 ------------------------------------------------
 3 files changed, 44 insertions(+), 59 deletions(-)
 create mode 100644 t/lei-sigpipe.t
 delete mode 100644 xt/lei-sigpipe.t

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;

                 reply	other threads:[~2021-04-20  9:17 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://public-inbox.org/README

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210420091726.69835-1-e@80x24.org \
    --to=e@80x24.org \
    --cc=meta@public-inbox.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).