diff options
author | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-04-06 21:44:38 +0000 |
---|---|---|
committer | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-04-06 21:45:03 +0000 |
commit | fad9acd35e56a289ade90a62d056b2a6663d448c (patch) | |
tree | 74d3203af13690a521d4de54ca2733223e2c389e /t/v2writable.t | |
parent | 8f2999546c9447ce2aed48ba4d1192e0058e28a2 (diff) | |
download | public-inbox-fad9acd35e56a289ade90a62d056b2a6663d448c.tar.gz |
Favor simpler internal APIs this time around, this cuts a fair amount of code out and takes another step towards removing Xapian as a dependency for v2 repos.
Diffstat (limited to 't/v2writable.t')
-rw-r--r-- | t/v2writable.t | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/t/v2writable.t b/t/v2writable.t index ab85e9af..4a42c016 100644 --- a/t/v2writable.t +++ b/t/v2writable.t @@ -108,14 +108,13 @@ if ('ensure git configs are correct') { $mime->header_set('References', '<zz-mid@b>'); ok($im->add($mime), 'message with multiple Message-ID'); $im->done; - my @found; my $srch = $ibx->search; - $srch->reopen->each_smsg_by_mid('abcde@1', sub { push @found, @_; 1 }); - is(scalar(@found), 1, 'message found by first MID'); - $srch->reopen->each_smsg_by_mid('abcde@2', sub { push @found, @_; 1 }); - is(scalar(@found), 2, 'message found by second MID'); - is($found[0]->{doc_id}, $found[1]->{doc_id}, 'same document'); - ok($found[1]->{doc_id} > 0, 'doc_id is positive'); + my $mset1 = $srch->reopen->query('m:abcde@1', { mset => 1 }); + is($mset1->size, 1, 'message found by first MID'); + my $mset2 = $srch->reopen->query('m:abcde@2', { mset => 1 }); + is($mset2->size, 1, 'message found by second MID'); + is((($mset1->items)[0])->get_docid, (($mset2->items)[0])->get_docid, + 'same document'); } SKIP: { @@ -224,9 +223,8 @@ EOF 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 $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'); + 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, 'removal propagated to Over DB'); |