v2writable: remove redundant remove from Over DB

[public-inbox.git] / t / v2writable.t

diff --git a/t/v2writable.t b/t/v2writable.t
index 4a7cfb909ee6f3030d722a532a48e7174fad44d9..ab85e9af86f73a1501a225b2950e3c5015237d44 100644 (file)
--- a/t/v2writable.t
+++ b/t/v2writable.t
@@ -105,6 +105,7 @@ if ('ensure git configs are correct') {
 
 {
        $mime->header_set('Message-Id', '<abcde@1>', '<abcde@2>');
+       $mime->header_set('References', '<zz-mid@b>');
        ok($im->add($mime), 'message with multiple Message-ID');
        $im->done;
        my @found;
@@ -189,6 +190,22 @@ EOF
                is($nn{$mid}++, 0, "MID is unique in NEWNEWS");
        }
        is_deeply([sort keys %nn], [sort keys %uniq]);
+
+       my %lg;
+       foreach my $num (@{$n->listgroup($group)}) {
+               is($lg{$num}++, 0, "num is unique in LISTGROUP");
+       }
+       is_deeply([sort keys %lg], [sort keys %$x],
+               'XOVER and LISTGROUPS return the same article numbers');
+
+       my $xref = $n->xhdr('Xref', '1-');
+       is_deeply([sort keys %lg], [sort keys %$xref], 'Xref range OK');
+
+       my $mids = $n->xhdr('Message-ID', '1-');
+       is_deeply([sort keys %lg], [sort keys %$xref], 'Message-ID range OK');
+
+       my $rover = $n->xrover('1-');
+       is_deeply([sort keys %lg], [sort keys %$rover], 'XROVER range OK');
 };
 {
        local $ENV{NPROC} = 2;
@@ -201,7 +218,7 @@ EOF
        $im->done;
        my $tip = shift @after;
        like($tip, qr/\A[a-f0-9]+ test removal\n\z/s,
-               'commit message propaged to git');
+               'commit message propagated to git');
        is_deeply(\@after, \@before, 'only one commit written to git');
        is($ibx->mm->num_for($smsg->mid), undef, 'no longer in Msgmap by mid');
        like($smsg->num, qr/\A\d+\z/, 'numeric number in return message');
@@ -209,8 +226,10 @@ EOF
        my $srch = $ibx->search->reopen;
        my @found = ();
        $srch->each_smsg_by_mid($smsg->mid, sub { push @found, @_; 1 });
-       is(scalar(@found), 0, 'no longer found in Xapian skeleton');
+       is(scalar(@found), 0, 'no longer found in Xapian');
        my @log1 = qw(log -1 --pretty=raw --raw -r --no-abbrev --no-renames);
+       is($srch->{over_ro}->get_art($smsg->num), undef,
+               'removal propagated to Over DB');
 
        my $after = $git0->qx(@log1);
        if ($after =~ m!( [a-f0-9]+ )A\td$!m) {
@@ -231,7 +250,30 @@ EOF
 {
        ok($im->add($mime), 'add message to be purged');
        local $SIG{__WARN__} = sub {};
-       ok($im->purge($mime), 'purged message');
+       ok(my $cmts = $im->purge($mime), 'purged message');
+       like($cmts->[0], qr/\A[a-f0-9]{40}\z/, 'purge returned current commit');
+       $im->done;
+}
+
+{
+       my @warn;
+       my $x = 'x'x250;
+       my $y = 'y'x250;
+       local $SIG{__WARN__} = sub { push @warn, @_ };
+       $mime->header_set('Subject', 'long mid');
+       $mime->header_set('Message-ID', "<$x>");
+       ok($im->add($mime), 'add excessively long Message-ID');
+
+       $mime->header_set('Message-ID', "<$y>");
+       $mime->header_set('References', "<$x>");
+       ok($im->add($mime), 'add excessively long References');
+       $im->barrier;
+
+       my $msgs = $ibx->search->reopen->get_thread('x'x244);
+       is(2, scalar(@$msgs), 'got both messages');
+       is($msgs->[0]->{mid}, 'x'x244, 'stored truncated mid');
+       is($msgs->[1]->{references}, '<'.('x'x244).'>', 'stored truncated ref');
+       is($msgs->[1]->{mid}, 'y'x244, 'stored truncated mid(2)');
        $im->done;
 }