about summary refs log tree commit homepage
path: root/t/v2writable.t
diff options
context:
space:
mode:
authorEric Wong (Contractor, The Linux Foundation) <e@80x24.org>2018-04-06 21:44:38 +0000
committerEric Wong (Contractor, The Linux Foundation) <e@80x24.org>2018-04-06 21:45:03 +0000
commitfad9acd35e56a289ade90a62d056b2a6663d448c (patch)
tree74d3203af13690a521d4de54ca2733223e2c389e /t/v2writable.t
parent8f2999546c9447ce2aed48ba4d1192e0058e28a2 (diff)
downloadpublic-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.t18
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');