lib/PublicInbox/V2Writable.pm | 1 - t/v2writable.t | 6 ++++-- diff --git a/lib/PublicInbox/V2Writable.pm b/lib/PublicInbox/V2Writable.pm index e0c8ac37916f629b6f028d8edfd8222d00086c23..832384601625fcbb5bca8cfa374e47d05d72d46b 100644 --- a/lib/PublicInbox/V2Writable.pm +++ b/lib/PublicInbox/V2Writable.pm @@ -309,7 +309,6 @@ $orig = undef; foreach my $idx (@$parts) { $idx->remote_remove($oid, $mid); } - $self->{over}->remove_oid($oid, $mid); } $self->barrier; } diff --git a/t/v2writable.t b/t/v2writable.t index e49c06b98c0efde79faf596f22891bddc0cdc084..ab85e9af86f73a1501a225b2950e3c5015237d44 100644 --- a/t/v2writable.t +++ b/t/v2writable.t @@ -218,7 +218,7 @@ my @after = $git0->qx(qw(log --pretty=oneline)); $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'); @@ -226,8 +226,10 @@ is($ibx->mm->mid_for($smsg->num), undef, 'no longer in Msgmap by num'); 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) {