# Copyright (C) 2018-2021 all contributors
# License: AGPL-3.0+
use strict; use v5.10.1; use PublicInbox::TestCommon;
use PublicInbox::Eml;
use PublicInbox::ContentHash qw(content_digest);
use File::Path qw(remove_tree);
require_git(2.6);
require_mods(qw(DBD::SQLite Search::Xapian));
use_ok 'PublicInbox::V2Writable';
use_ok 'PublicInbox::OverIdx';
my ($inboxdir, $for_destroy) = tmpdir();
my $ibx_config = {
inboxdir => $inboxdir,
name => 'test-v2writable',
version => 2,
-primary_address => 'test@example.com',
indexlevel => 'full',
-no_fsync => 1,
};
my $agpl = do {
open my $fh, '<', 'COPYING' or die "can't open COPYING: $!";
local $/;
<$fh>;
};
my $phrase = q("defending all users' freedom");
my $mime = PublicInbox::Eml->new(<<'EOF'.$agpl);
From: a@example.com
To: test@example.com
Subject: this is a subject
Date: Fri, 02 Oct 1993 00:00:00 +0000
EOF
my $minmax;
my $msgmap;
my ($mark1, $mark2, $mark3, $mark4);
{
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx, {nproc => 1});
my $im0 = $im->importer(0);
foreach my $i (1..10) {
$mime->header_set('Message-Id', "<$i\@example.com>");
ok($im->add($mime), "message $i added");
if ($i == 4) {
$mark1 = $im0->get_mark($im0->{tip});
$im->remove($mime);
$mark2 = $im0->get_mark($im0->{tip});
}
}
if ('test remove later') {
$mark3 = $im0->get_mark($im0->{tip});
$mime->header_set('Message-Id', "<5\@example.com>");
$im->remove($mime);
$mark4 = $im0->get_mark($im0->{tip});
}
$im->done;
$minmax = [ $ibx->mm->minmax ];
ok(defined $minmax->[0] && defined $minmax->[1], 'minmax defined');
is_deeply($minmax, [ 1, 10 ], 'minmax as expected');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
my ($min, $max) = @$minmax;
$msgmap = $ibx->mm->msg_range(\$min, $max);
is_deeply($msgmap, [
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
[6, '6@example.com' ],
[7, '7@example.com' ],
[8, '8@example.com' ],
[9, '9@example.com' ],
[10, '10@example.com' ],
], 'msgmap as expected');
}
{
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx, 1);
eval { $im->index_sync({reindex => 1}) };
is($@, '', 'no error from reindexing');
$im->done;
delete $ibx->{mm};
is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
my ($min, $max) = $ibx->mm->minmax;
is_deeply($ibx->mm->msg_range(\$min, $max), $msgmap, 'msgmap unchanged');
}
my $xap = "$inboxdir/xap".PublicInbox::Search::SCHEMA_VERSION();
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed');
{
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx, 1);
eval { $im->index_sync({reindex => 1}) };
is($@, '', 'no error from reindexing');
$im->done;
ok(-d $xap, 'Xapian directories recreated');
delete $ibx->{mm};
is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
my ($min, $max) = $ibx->mm->minmax;
is_deeply($ibx->mm->msg_range(\$min, $max), $msgmap, 'msgmap unchanged');
}
ok(unlink "$inboxdir/msgmap.sqlite3", 'remove msgmap');
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed again');
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx, 1);
eval { $im->index_sync({reindex => 1}) };
is($@, '', 'no error from reindexing without msgmap');
is(scalar(@warn), 0, 'no warnings from reindexing');
$im->done;
ok(-d $xap, 'Xapian directories recreated');
delete $ibx->{mm};
is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
my ($min, $max) = $ibx->mm->minmax;
is_deeply($ibx->mm->msg_range(\$min, $max), $msgmap, 'msgmap unchanged');
}
my %sizes;
ok(unlink "$inboxdir/msgmap.sqlite3", 'remove msgmap');
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed again');
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx, 1);
eval { $im->index_sync({reindex => 1}) };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
ok(-d $xap, 'Xapian directories recreated');
delete $ibx->{mm};
is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
my $mset = $ibx->search->mset($phrase);
isnt($mset->size, 0, "phrase search succeeds on indexlevel=full");
for (glob("$xap/*/*")) { $sizes{$ibx->{indexlevel}} += -s _ if -f $_ }
my ($min, $max) = $ibx->mm->minmax;
is_deeply($ibx->mm->msg_range(\$min, $max), $msgmap, 'msgmap unchanged');
}
ok(unlink "$inboxdir/msgmap.sqlite3", 'remove msgmap');
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed again');
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
$config{indexlevel} = 'medium';
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync({reindex => 1}) };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
ok(-d $xap, 'Xapian directories recreated');
delete $ibx->{mm};
is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
if (0) {
# not sure why, but Xapian seems to fallback to terms and
# phrase searches still work
delete $ibx->{search};
my $mset = $ibx->search->mset($phrase);
is($mset->size, 0, 'phrase search does not work on medium');
}
my $words = $phrase;
$words =~ tr/"'//d;
my $mset = $ibx->search->mset($words);
isnt($mset->size, 0, "normal search works on indexlevel=medium");
for (glob("$xap/*/*")) { $sizes{$ibx->{indexlevel}} += -s _ if -f $_ }
ok($sizes{full} > $sizes{medium}, 'medium is smaller than full');
my ($min, $max) = $ibx->mm->minmax;
is_deeply($ibx->mm->msg_range(\$min, $max), $msgmap, 'msgmap unchanged');
}
ok(unlink "$inboxdir/msgmap.sqlite3", 'remove msgmap');
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed again');
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
$config{indexlevel} = 'basic';
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync({reindex => 1}) };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
ok(-d $xap, 'Xapian directories recreated');
delete $ibx->{mm};
is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
isnt($ibx->search, 'no search for basic');
for (glob("$xap/*/*")) { $sizes{$ibx->{indexlevel}} += -s _ if -f $_ }
ok($sizes{medium} > $sizes{basic}, 'basic is smaller than medium');
my ($min, $max) = $ibx->mm->minmax;
is_deeply($ibx->mm->msg_range(\$min, $max), $msgmap, 'msgmap unchanged');
}
# An incremental indexing test
ok(unlink "$inboxdir/msgmap.sqlite3", 'remove msgmap');
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed again');
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark1 4 simple additions in the same index_sync
$ibx->{ref_head} = $mark1;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 4, 'max as expected');
is($ibx->mm->num_highwater, 4, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
[4, '4@example.com' ],
], 'msgmap as expected' );
}
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark2 A delete separated from an add in the same index_sync
$ibx->{ref_head} = $mark2;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 3, 'max as expected');
is($ibx->mm->num_highwater, 4, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
], 'msgmap as expected' );
}
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark3 adds following the delete at mark2
$ibx->{ref_head} = $mark3;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 10, 'max as expected');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
[5, '5@example.com' ],
[6, '6@example.com' ],
[7, '7@example.com' ],
[8, '8@example.com' ],
[9, '9@example.com' ],
[10, '10@example.com' ],
], 'msgmap as expected' );
}
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark4 A delete of an older message
$ibx->{ref_head} = $mark4;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 10, 'max as expected');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
[6, '6@example.com' ],
[7, '7@example.com' ],
[8, '8@example.com' ],
[9, '9@example.com' ],
[10, '10@example.com' ],
], 'msgmap as expected' );
}
# Another incremental indexing test
ok(unlink "$inboxdir/msgmap.sqlite3", 'remove msgmap');
remove_tree($xap);
ok(!-d $xap, 'Xapian directories removed again');
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark2 an add and it's delete in the same index_sync
$ibx->{ref_head} = $mark2;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 3, 'max as expected');
is($ibx->mm->num_highwater, 4, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
], 'msgmap as expected' );
}
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark3 adds following the delete at mark2
$ibx->{ref_head} = $mark3;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 10, 'max as expected');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
[5, '5@example.com' ],
[6, '6@example.com' ],
[7, '7@example.com' ],
[8, '8@example.com' ],
[9, '9@example.com' ],
[10, '10@example.com' ],
], 'msgmap as expected' );
}
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
# mark4 A delete of an older message
$ibx->{ref_head} = $mark4;
my $im = PublicInbox::V2Writable->new($ibx);
eval { $im->index_sync() };
is($@, '', 'no error from reindexing without msgmap');
is_deeply(\@warn, [], 'no warnings');
$im->done;
my ($min, $max) = $ibx->mm->minmax;
is($min, 1, 'min as expected');
is($max, 10, 'max as expected');
is($ibx->mm->num_highwater, 10, 'num_highwater as expected');
is_deeply($ibx->mm->msg_range(\$min, $max),
[
[1, '1@example.com' ],
[2, '2@example.com' ],
[3, '3@example.com' ],
[6, '6@example.com' ],
[7, '7@example.com' ],
[8, '8@example.com' ],
[9, '9@example.com' ],
[10, '10@example.com' ],
], 'msgmap as expected' );
}
my $check_rethread = sub {
my ($desc) = @_;
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
my $ibx = PublicInbox::Inbox->new(\%config);
my $f = $ibx->over->{dbh}->sqlite_db_filename;
my $over = PublicInbox::OverIdx->new($f);
my $dbh = $over->dbh;
my $non_ghost_tids = sub {
$dbh->selectall_arrayref(<<'');
SELECT tid FROM over WHERE num > 0 ORDER BY tid ASC
};
my $before = $non_ghost_tids->();
# mess up threading:
my $tid = PublicInbox::OverIdx::get_counter($dbh, 'thread');
my $nr = $dbh->do('UPDATE over SET tid = ?', undef, $tid);
diag "messing up all threads with tid=$tid";
my $v2w = PublicInbox::V2Writable->new($ibx);
my @pr;
my $pr = sub { push @pr, @_ };
$v2w->index_sync({reindex => 1, rethread => 1, -progress => $pr});
# diag "@pr"; # nobody cares
is_deeply(\@warn, [], 'no warnings on reindex + rethread');
my @n = $dbh->selectrow_array(<();
ok($after->[0]->[0] > $before->[-1]->[0],
'all tids greater than before');
is(scalar @$after, scalar @$before, 'thread count unchanged');
};
$check_rethread->('no-monster');
# A real example from linux-renesas-soc on lore where a 3-headed monster
# of a message has 3 sets of common headers. Another normal message
# previously existed with a single Message-ID that conflicts with one
# of the Message-IDs in the 3-headed monster.
{
my @warn;
local $SIG{__WARN__} = sub { push @warn, @_ };
my %config = %$ibx_config;
$config{indexlevel} = 'medium';
my $ibx = PublicInbox::Inbox->new(\%config);
my $im = PublicInbox::V2Writable->new($ibx);
my $m3 = PublicInbox::Eml->new(<<'EOF');
Date: Tue, 24 May 2016 14:34:22 -0700 (PDT)
Message-Id: <20160524.143422.552507610109476444.d@example.com>
To: t@example.com
Cc: c@example.com
Subject: Re: [PATCH v2 2/2] uno
From:
In-Reply-To: <1463825855-7363-2-git-send-email-y@example.com>
References: <1463825855-7363-1-git-send-email-y@example.com>
<1463825855-7363-2-git-send-email-y@example.com>
Date: Wed, 25 May 2016 10:01:51 +0900
From: h@example.com
To: g@example.com
Cc: m@example.com
Subject: Re: [PATCH] dos
Message-ID: <20160525010150.GD7292@example.com>
References: <1463498133-23918-1-git-send-email-g+r@example.com>
In-Reply-To: <1463498133-23918-1-git-send-email-g+r@example.com>
From: s@example.com
To: h@example.com
Cc: m@example.com
Subject: [PATCH 12/13] tres
Date: Wed, 01 Jun 2016 01:32:35 +0300
Message-ID: <1923946.Jvi0TDUXFC@wasted.example.com>
In-Reply-To: <13205049.n7pM8utpHF@wasted.example.com>
References: <13205049.n7pM8utpHF@wasted.example.com>
Somehow we got a message with 3 sets of headers into one
message, could've been something broken on the archiver side.
EOF
my $m1 = PublicInbox::Eml->new(<<'EOF');
From: a@example.com
To: t@example.com
Subject: [PATCH 12/13]
Date: Wed, 01 Jun 2016 01:32:35 +0300
Message-ID: <1923946.Jvi0TDUXFC@wasted.example.com>
In-Reply-To: <13205049.n7pM8utpHF@wasted.example.com>
References: <13205049.n7pM8utpHF@wasted.example.com>
This is probably one of the original messages
EOF
$im->add($m1);
$im->add($m3);
$im->done;
remove_tree($xap);
eval { $im->index_sync() };
is($@, '', 'no error from initial indexing');
is_deeply(\@warn, [], 'no warnings from initial index');
eval { $im->index_sync({reindex=>1}) };
is($@, '', 'no error from reindexing after reused Message-ID (x3)');
is_deeply(\@warn, [], 'no warnings on reindex');
my %uniq;
for my $s (qw(uno dos tres)) {
my $mset = $ibx->search->mset("s:$s");
my $msgs = $ibx->search->mset_to_smsg($ibx, $mset);
is(scalar(@$msgs), 1, "only one result for `$s'");
$uniq{$msgs->[0]->{num}}++;
}
is_deeply([values %uniq], [3], 'search on different subjects');
}
# XXX: not deterministic when dealing with ambiguous messages, oh well
$check_rethread->('3-headed-monster once');
$check_rethread->('3-headed-monster twice');
my $rdr = { 2 => \(my $err = '') };
my $env = { PI_CONFIG => '/dev/null' };
ok(run_script([qw(-index --reindex --xapian-only), $inboxdir], $env, $rdr),
'--xapian-only works');
is($err, '', 'no errors from --xapian-only');
undef $for_destroy;
SKIP: {
skip 'only testing lsof(8) output on Linux', 1 if $^O ne 'linux';
my $lsof = require_cmd('lsof', 1) or skip 'no lsof in PATH', 1;
my $rdr = { 2 => \(my $null_err) };
my @d = grep(m!/xap[0-9]+/!, xqx([$lsof, '-p', $$], undef, $rdr));
is_deeply(\@d, [], 'no deleted index files') or diag explain(\@d);
}
done_testing();