searchidx: regenerate and avoid article number gaps on full index

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

diff --git a/t/v2writable.t b/t/v2writable.t
index b543c53fa44e0e5a98b46527146791f92f334336..d37fb06eae1bdf08582db8ec5771d1841e68a793 100644 (file)
--- a/t/v2writable.t
+++ b/t/v2writable.t
@@ -68,7 +68,7 @@ if ('ensure git configs are correct') {
                [ $sec->header_obj->header_raw('Message-Id') ],
                'no new Message-Id added');
 
-       my $sane_mid = qr/\A<[\w\-]+\@localhost>\z/;
+       my $sane_mid = qr/\A<[\w\-\.]+\@\w+>\z/;
        @warn = ();
        $mime->header_set('Message-Id', '<a-mid@b>');
        $mime->body_set('different');
@@ -82,7 +82,8 @@ if ('ensure git configs are correct') {
        @warn = ();
        $mime->header_set('Message-Id', '<a-mid@b>');
        $mime->body_set('this one needs a random mid');
-       my $gen = PublicInbox::Import::digest2mid(content_digest($mime));
+       my $hdr = $mime->header_obj;
+       my $gen = PublicInbox::Import::digest2mid(content_digest($mime), $hdr);
        unlike($gen, qr![\+/=]!, 'no URL-unfriendly chars in Message-Id');
        my $fake = PublicInbox::MIME->new($mime->as_string);
        $fake->header_set('Message-Id', "<$gen>");
@@ -220,13 +221,14 @@ EOF
                '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');
-       is($ibx->mm->mid_for($smsg->num), undef, 'no longer in Msgmap by num');
+       my $num = $smsg->{num};
+       like($num, qr/\A\d+\z/, 'numeric number in return message');
+       is($ibx->mm->mid_for($num), undef, 'no longer in Msgmap by num');
        my $srch = $ibx->search->reopen;
        my $mset = $srch->query('m:'.$smsg->mid, { mset => 1});
        is($mset->size, 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,
+       is($srch->{over_ro}->get_art($num), undef,
                'removal propagated to Over DB');
 
        my $after = $git0->qx(@log1);