diff options
author | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-03-29 09:57:52 +0000 |
---|---|---|
committer | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-03-29 10:00:04 +0000 |
commit | 11707dae97d1f4638157cfee298464b2f2deeed4 (patch) | |
tree | a25db85d1ab0e5e4daeb099140d31f1c07fd0c5c /t/search.t | |
parent | 821ed7c40b7b50ceb1c942af5e14d168995d514e (diff) | |
download | public-inbox-11707dae97d1f4638157cfee298464b2f2deeed4.tar.gz |
Too many similar functions doing the same basic thing was redundant and misleading, especially since Message-ID is no longer treated as a truly unique identifier. For displaying threads in the HTML, this makes it clear that we favor the primary Message-ID mapped to an NNTP article number if a message cannot be found.
Diffstat (limited to 't/search.t')
-rw-r--r-- | t/search.t | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -89,7 +89,7 @@ sub filter_mids { { $rw_commit->(); $ro->reopen; - my $found = $ro->lookup_message('<root@s>'); + my $found = $ro->first_smsg_by_mid('root@s'); ok($found, "message found"); is($root_id, $found->{doc_id}, 'doc_id set correctly'); is($found->mid, 'root@s', 'mid set correctly'); @@ -264,7 +264,7 @@ sub filter_mids { ], body => "LOOP!\n")); ok($doc_id > 0, "doc_id defined with circular reference"); - my $smsg = $rw->lookup_message('circle@a'); + my $smsg = $rw->first_smsg_by_mid('circle@a'); is($smsg->references, '', "no references created"); my $msg = PublicInbox::SearchMsg->load_doc($smsg->{doc}); is($s, $msg->subject, 'long subject not rewritten'); @@ -281,7 +281,7 @@ sub filter_mids { my $mime = Email::MIME->new($str); my $doc_id = $rw->add_message($mime); ok($doc_id > 0, 'message indexed doc_id with UTF-8'); - my $smsg = $rw->lookup_message('testmessage@example.com'); + my $smsg = $rw->first_smsg_by_mid('testmessage@example.com'); my $msg = PublicInbox::SearchMsg->load_doc($smsg->{doc}); is($mime->header('Subject'), $msg->subject, 'UTF-8 subject preserved'); |