about summary refs log tree commit homepage
diff options
context:
space:
mode:
-rw-r--r--MANIFEST1
-rw-r--r--t/imap.t20
-rw-r--r--t/imapd.t49
-rw-r--r--xt/perf-imap-list.t35
4 files changed, 57 insertions, 48 deletions
diff --git a/MANIFEST b/MANIFEST
index 0803c365..2e4ec915 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -353,6 +353,7 @@ xt/git_async_cmp.t
 xt/mem-msgview.t
 xt/msgtime_cmp.t
 xt/nntpd-validate.t
+xt/perf-imap-list.t
 xt/perf-msgview.t
 xt/perf-nntpd.t
 xt/perf-threading.t
diff --git a/t/imap.t b/t/imap.t
index c435d365..9b64f164 100644
--- a/t/imap.t
+++ b/t/imap.t
@@ -1,9 +1,29 @@
 #!perl -w
 # Copyright (C) 2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+# unit tests (no network) for IMAP, see t/imapd.t for end-to-end tests
 use strict;
 use Test::More;
 use PublicInbox::IMAP;
+use PublicInbox::IMAPD;
+
+{ # make sure we get '%' globbing right
+        my @n = map { { newsgroup => $_ } } (qw(x.y.z x.z.y));
+        my $self = { imapd => { grouplist => \@n } };
+        PublicInbox::IMAPD::refresh_inboxlist($self->{imapd});
+        my $res = PublicInbox::IMAP::cmd_list($self, 'tag', 'x', '%');
+        is(scalar($$res =~ tr/\n/\n/), 2, 'only one result');
+        like($$res, qr/ x\r\ntag OK/, 'saw expected');
+        $res = PublicInbox::IMAP::cmd_list($self, 'tag', 'x.', '%');
+        is(scalar($$res =~ tr/\n/\n/), 3, 'only one result');
+        is(scalar(my @x = ($$res =~ m/ x\.[zy]\r\n/g)), 2, 'match expected');
+
+        $res = PublicInbox::IMAP::cmd_list($self, 't', 'x.(?{die "RCE"})', '%');
+        like($$res, qr/\At OK /, 'refname does not match attempted RCE');
+        $res = PublicInbox::IMAP::cmd_list($self, 't', '', '(?{die "RCE"})%');
+        like($$res, qr/\At OK /, 'wildcard does not match attempted RCE');
+}
+
 {
         my $partial_prepare = \&PublicInbox::IMAP::partial_prepare;
         my $x = {};
diff --git a/t/imapd.t b/t/imapd.t
index a63be0fd..59b95a6b 100644
--- a/t/imapd.t
+++ b/t/imapd.t
@@ -1,6 +1,7 @@
 #!perl -w
 # Copyright (C) 2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+# end-to-end IMAP tests, see unit tests in t/imap.t, too
 use strict;
 use Test::More;
 use Time::HiRes ();
@@ -99,54 +100,6 @@ like($raw[0], qr/^\* LIST \(.*?\) "\." inbox/,
                 'got an inbox.i1');
 like($raw[-1], qr/^\S+ OK /, 'response ended with OK');
 
-{ # make sure we get '%' globbing right
-        my @n = map { { newsgroup => $_ } } (qw(x.y.z x.z.y));
-        my $self = { imapd => { grouplist => \@n } };
-        PublicInbox::IMAPD::refresh_inboxlist($self->{imapd});
-        my $res = PublicInbox::IMAP::cmd_list($self, 'tag', 'x', '%');
-        is(scalar($$res =~ tr/\n/\n/), 2, 'only one result');
-        like($$res, qr/ x\r\ntag OK/, 'saw expected');
-        $res = PublicInbox::IMAP::cmd_list($self, 'tag', 'x.', '%');
-        is(scalar($$res =~ tr/\n/\n/), 3, 'only one result');
-        is(scalar(my @x = ($$res =~ m/ x\.[zy]\r\n/g)), 2, 'match expected');
-
-        $res = PublicInbox::IMAP::cmd_list($self, 't', 'x.(?{die "RCE"})', '%');
-        like($$res, qr/\At OK /, 'refname does not match attempted RCE');
-        $res = PublicInbox::IMAP::cmd_list($self, 't', '', '(?{die "RCE"})%');
-        like($$res, qr/\At OK /, 'wildcard does not match attempted RCE');
-}
-
-if ($ENV{TEST_BENCHMARK}) {
-        use Benchmark qw(:all);
-        my @n = map { { newsgroup => "inbox.comp.foo.bar.$_" } } (0..50000);
-        push @n, map { { newsgroup => "xobni.womp.foo.bar.$_" } } (0..50000);
-        my $self = { imapd => { grouplist => \@n } };
-        PublicInbox::IMAPD::refresh_inboxlist($self->{imapd});
-
-        my $n = scalar @n;
-        open my $null, '>', '/dev/null' or die;
-        my $ds = { sock => $null };
-        my $nr = 200;
-        diag "starting benchmark...";
-        my $t = timeit(1, sub {
-                for (0..$nr) {
-                        my $res = PublicInbox::IMAP::cmd_list($self, 'tag',
-                                                                '', '*');
-                        PublicInbox::DS::write($ds, $res);
-                }
-        });
-        diag timestr($t). "list all for $n inboxes $nr times";
-        $nr = 20;
-        $t = timeit(1, sub {
-                for (0..$nr) {
-                        my $res = PublicInbox::IMAP::cmd_list($self, 'tag',
-                                                                'inbox.', '%');
-                        PublicInbox::DS::write($ds, $res);
-                }
-        });
-        diag timestr($t). "list partial for $n inboxes $nr times";
-}
-
 my $ret = $mic->search('all') or BAIL_OUT "SEARCH FAIL $@";
 is_deeply($ret, [ 1 ], 'search all works');
 $ret = $mic->search('uid 1') or BAIL_OUT "SEARCH FAIL $@";
diff --git a/xt/perf-imap-list.t b/xt/perf-imap-list.t
new file mode 100644
index 00000000..37640a90
--- /dev/null
+++ b/xt/perf-imap-list.t
@@ -0,0 +1,35 @@
+#!perl -w
+# Copyright (C) 2020 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+use Test::More;
+use_ok 'PublicInbox::IMAP';
+use_ok 'PublicInbox::IMAPD';
+use PublicInbox::DS;
+use Benchmark qw(:all);
+my @n = map { { newsgroup => "inbox.comp.foo.bar.$_" } } (0..50000);
+push @n, map { { newsgroup => "xobni.womp.foo.bar.$_" } } (0..50000);
+my $self = { imapd => { grouplist => \@n } };
+PublicInbox::IMAPD::refresh_inboxlist($self->{imapd});
+
+my $n = scalar @n;
+open my $null, '>', '/dev/null' or BAIL_OUT "open: $!";
+my $ds = { sock => $null };
+my $nr = 200;
+diag "starting benchmark...";
+my $cmd_list = \&PublicInbox::IMAP::cmd_list;
+my $t = timeit(1, sub {
+        for (0..$nr) {
+                my $res = $cmd_list->($self, 'tag', '', '*');
+                PublicInbox::DS::write($ds, $res);
+        }
+});
+diag timestr($t). "list all for $n inboxes $nr times";
+$nr = 20;
+$t = timeit(1, sub {
+        for (0..$nr) {
+                my $res = $cmd_list->($self, 'tag', 'inbox.', '%');
+                PublicInbox::DS::write($ds, $res);
+        }
+});
+diag timestr($t). "list partial for $n inboxes $nr times";
+done_testing;