]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/indexlevels-mirror.t
remove most internal Email::MIME usage
[public-inbox.git] / t / indexlevels-mirror.t
index b685da145fefcbe49bfd490186364a2ded5832d1..704f7e1174e3198e6c0bb6e6b1fc5cc5c3a48fd6 100644 (file)
@@ -1,42 +1,33 @@
-# Copyright (C) 2019 all contributors <meta@public-inbox.org>
+# Copyright (C) 2019-2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 use strict;
 use warnings;
 use Test::More;
-use PublicInbox::MIME;
+use PublicInbox::Eml;
 use PublicInbox::Inbox;
 use PublicInbox::InboxWritable;
-use File::Temp qw/tempdir/;
 require PublicInbox::Admin;
-require './t/common.perl';
+use PublicInbox::TestCommon;
 my $PI_TEST_VERSION = $ENV{PI_TEST_VERSION} || 2;
 require_git('2.6') if $PI_TEST_VERSION == 2;
+require_mods(qw(DBD::SQLite));
 
-foreach my $mod (qw(DBD::SQLite)) {
-       eval "require $mod";
-       plan skip_all => "$mod missing for $0" if $@;
-}
-
-my $path = 'blib/script';
-my $index = "$path/public-inbox-index";
-my @xcpdb = ("$path/public-inbox-xcpdb", '-q');
+my $mime = PublicInbox::Eml->new(<<'EOF');
+From: a@example.com
+To: test@example.com
+Subject: this is a subject
+Date: Fri, 02 Oct 1993 00:00:00 +0000
 
-my $mime = PublicInbox::MIME->create(
-       header => [
-               From => 'a@example.com',
-               To => 'test@example.com',
-               Subject => 'this is a subject',
-               Date => 'Fri, 02 Oct 1993 00:00:00 +0000',
-       ],
-       body => "hello world\n",
-);
+hello world
+EOF
 
 sub import_index_incremental {
-       my ($v, $level) = @_;
+       my ($v, $level, $mime) = @_;
        my $this = "pi-$v-$level-indexlevels";
-       my $tmpdir = tempdir("$this-tmp-XXXXXX", TMPDIR => 1, CLEANUP => 1);
+       my ($tmpdir, $for_destroy) = tmpdir();
+       local $ENV{PI_CONFIG} = "$tmpdir/config";
        my $ibx = PublicInbox::Inbox->new({
-               mainrepo => "$tmpdir/testbox",
+               inboxdir => "$tmpdir/testbox",
                name => $this,
                version => $v,
                -primary_address => 'test@example.com',
@@ -48,9 +39,10 @@ sub import_index_incremental {
        $im->done;
 
        # index master (required for v1)
-       is(system($index, $ibx->{mainrepo}, "-L$level"), 0, 'index master OK');
+       ok(run_script(['-index', $ibx->{inboxdir}, "-L$level"]),
+               'index master OK');
        my $ro_master = PublicInbox::Inbox->new({
-               mainrepo => $ibx->{mainrepo},
+               inboxdir => $ibx->{inboxdir},
                indexlevel => $level
        });
        my ($nr, $msgs) = $ro_master->recent;
@@ -61,26 +53,26 @@ sub import_index_incremental {
        my @cmd = (qw(git clone --mirror -q));
        my $mirror = "$tmpdir/mirror-$v";
        if ($v == 1) {
-               push @cmd, $ibx->{mainrepo}, $mirror;
+               push @cmd, $ibx->{inboxdir}, $mirror;
        } else {
-               push @cmd, "$ibx->{mainrepo}/git/0.git", "$mirror/git/0.git";
+               push @cmd, "$ibx->{inboxdir}/git/0.git", "$mirror/git/0.git";
        }
        my $fetch_dir = $cmd[-1];
-       is(system(@cmd), 0, "v$v clone OK");
+       is(xsys(@cmd), 0, "v$v clone OK");
 
        # inbox init
        local $ENV{PI_CONFIG} = "$tmpdir/.picfg";
-       @cmd = ("$path/public-inbox-init", '-L', $level,
+       @cmd = ('-init', '-L', $level,
                'mirror', $mirror, '//example.com/test', 'test@example.com');
        push @cmd, '-V2' if $v == 2;
-       is(system(@cmd), 0, "v$v init OK");
+       ok(run_script(\@cmd), "v$v init OK");
 
        # index mirror
-       is(system($index, $mirror), 0, "v$v index mirror OK");
+       ok(run_script(['-index', $mirror]), "v$v index mirror OK");
 
        # read-only access
        my $ro_mirror = PublicInbox::Inbox->new({
-               mainrepo => $mirror,
+               inboxdir => $mirror,
                indexlevel => $level,
        });
        ($nr, $msgs) = $ro_mirror->recent;
@@ -93,15 +85,16 @@ sub import_index_incremental {
        $im->done;
 
        # mirror updates
-       is(system('git', "--git-dir=$fetch_dir", qw(fetch -q)), 0, 'fetch OK');
-       is(system($index, $mirror), 0, "v$v index mirror again OK");
+       is(xsys('git', "--git-dir=$fetch_dir", qw(fetch -q)), 0, 'fetch OK');
+       ok(run_script(['-index', $mirror]), "v$v index mirror again OK");
        ($nr, $msgs) = $ro_mirror->recent;
        is($nr, 2, '2nd message seen in mirror');
        is_deeply([sort { $a cmp $b } map { $_->{mid} } @$msgs],
                ['m@1','m@2'], 'got both messages in mirror');
 
        # incremental index master (required for v1)
-       is(system($index, $ibx->{mainrepo}, "-L$level"), 0, 'index master OK');
+       ok(run_script(['-index', $ibx->{inboxdir}, "-L$level"]),
+               'index master OK');
        ($nr, $msgs) = $ro_master->recent;
        is($nr, 2, '2nd message seen in master');
        is_deeply([sort { $a cmp $b } map { $_->{mid} } @$msgs],
@@ -120,7 +113,7 @@ sub import_index_incremental {
        is_deeply(\@rw_nums, [1], 'unindex NNTP article'.$v.$level);
 
        if ($level ne 'basic') {
-               is(system(@xcpdb, $mirror), 0, "v$v xcpdb OK");
+               ok(run_script(['-xcpdb', '-q', $mirror]), "v$v xcpdb OK");
                is(PublicInbox::Admin::detect_indexlevel($ro_mirror), $level,
                   'indexlevel detectable by Admin after xcpdb v' .$v.$level);
                delete $ro_mirror->{$_} for (qw(over search));
@@ -129,15 +122,15 @@ sub import_index_incremental {
        }
 
        # sync the mirror
-       is(system('git', "--git-dir=$fetch_dir", qw(fetch -q)), 0, 'fetch OK');
-       is(system($index, $mirror), 0, "v$v index mirror again OK");
+       is(xsys('git', "--git-dir=$fetch_dir", qw(fetch -q)), 0, 'fetch OK');
+       ok(run_script(['-index', $mirror]), "v$v index mirror again OK");
        ($nr, $msgs) = $ro_mirror->recent;
        is($nr, 1, '2nd message gone from mirror');
        is_deeply([map { $_->{mid} } @$msgs], ['m@1'],
                'message unavailable in mirror');
 
        if ($v == 2 && $level eq 'basic') {
-               is_deeply([glob("$ibx->{mainrepo}/xap*/?/")], [],
+               is_deeply([glob("$ibx->{inboxdir}/xap*/?/")], [],
                         'no Xapian shard directories for v2 basic');
        }
        if ($level ne 'basic') {
@@ -154,8 +147,8 @@ sub import_index_incremental {
                push @expect, $i;
        }
        $im->done;
-       is(system('git', "--git-dir=$fetch_dir", qw(fetch -q)), 0, 'fetch OK');
-       is(system($index, '--reindex', $mirror), 0,
+       is(xsys('git', "--git-dir=$fetch_dir", qw(fetch -q)), 0, 'fetch OK');
+       ok(run_script(['-index', '--reindex', $mirror]),
                "v$v index --reindex mirror OK");
        @ro_nums = map { $_->{num} } @{$ro_mirror->over->query_ts(0, 0)};
        @rw_nums = map { $_->{num} } @{$ibx->over->query_ts(0, 0)};
@@ -167,13 +160,14 @@ sub import_index_incremental {
 }
 
 # we can probably cull some other tests
-import_index_incremental($PI_TEST_VERSION, 'basic');
+import_index_incremental($PI_TEST_VERSION, 'basic', $mime);
 
 SKIP: {
        require PublicInbox::Search;
-       PublicInbox::Search::load_xapian() or skip 'Search::Xapian missing', 2;
+       PublicInbox::Search::load_xapian() or
+               skip('Xapian perl binding missing', 2);
        foreach my $l (qw(medium full)) {
-               import_index_incremental($PI_TEST_VERSION, $l);
+               import_index_incremental($PI_TEST_VERSION, $l, $mime);
        }
 }