diff options
| author | Eric Wong <e@80x24.org> | 2021-10-18 05:09:05 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2021-10-18 05:15:15 +0000 |
| commit | 325fbe26c3e7731eb53bea7f2ce2627c8828e0d8 (patch) | |
| tree | 22a71079d086ce0a7c1a7caffc99eb930123c47f /t | |
| parent | c51814c647ec83a137be7de3a1baeb7336d318ea (diff) | |
| download | public-inbox-325fbe26c3e7731eb53bea7f2ce2627c8828e0d8.tar.gz | |
For odd messages with reused Message-IDs, the second message
showing up in a mirror (via git-fetch + -index) should never
clobber an entry with a different blob in over.
This is noticeable only if the messages arrive in-between
indexing runs.
Fixes: 4441a38481ed ("v2: index forwards (via `git log --reverse')")
Diffstat (limited to 't')
| -rw-r--r-- | t/v2index-late-dupe.t | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/t/v2index-late-dupe.t b/t/v2index-late-dupe.t new file mode 100644 index 00000000..c83e3409 --- /dev/null +++ b/t/v2index-late-dupe.t @@ -0,0 +1,37 @@ +# Copyright (C) all contributors <meta@public-inbox.org> +# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt> +# +# this simulates a mirror path: git fetch && -index +use strict; use v5.10.1; use PublicInbox::TestCommon; +use Test::More; # redundant, used for bisect +require_mods 'v2'; +require PublicInbox::Import; +require PublicInbox::Inbox; +require PublicInbox::Git; +my ($tmpdir, $for_destroy) = tmpdir(); +my $inboxdir = "$tmpdir/i"; +PublicInbox::Import::init_bare(my $e0 = "$inboxdir/git/0.git"); +open my $fh, '>', "$inboxdir/inbox.lock" or xbail $!; +my $git = PublicInbox::Git->new($e0); +my $im = PublicInbox::Import->new($git, qw(i i@example.com)); +$im->{lock_path} = undef; +$im->{path_type} = 'v2'; +my $eml = eml_load('t/plack-qp.eml'); +ok($im->add($eml), 'add original'); +$im->done; +run_script([qw(-index -Lbasic), $inboxdir]); +is($?, 0, 'basic index'); +my $ibx = PublicInbox::Inbox->new({ inboxdir => $inboxdir }); +my $orig = $ibx->over->get_art(1); + +my @mid = $eml->header_raw('Message-ID'); +$eml->header_set('Message-ID', @mid, '<extra@z>'); +ok($im->add($eml), 'add another'); +$im->done; +run_script([qw(-index -Lbasic), $inboxdir]); +is($?, 0, 'basic index again'); + +my $after = $ibx->over->get_art(1); +is_deeply($after, $orig, 'original unchanged') or note explain([$orig,$after]); + +done_testing; |