diff options
author | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-04-01 23:15:04 +0000 |
---|---|---|
committer | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-04-01 23:25:04 +0000 |
commit | c34a83286234ea1e876ebdf92a33744272bb6f4e (patch) | |
tree | a04f43151284e6893e23780438773f9ea07c9fba /t/convert-compact.t | |
parent | 0321a1a9e7ae9c9d878d547ee67659ef8aa95689 (diff) | |
download | public-inbox-c34a83286234ea1e876ebdf92a33744272bb6f4e.tar.gz |
We need to ensure there is only one file in the top-level tree at any commit so the "add; remove; add;" sequence on the same message is detected properly. Otherwise, git will not detect the second "add" unless a second message is added to history. Deletes are now stored in "d" (and not "D" or "_/D") at the top-level, now. There's no need to have a "_" to reduce churn as "m" and "d" should never co-exist. It's now lowercased to make it easier-to-distinguish from "D" in git-log output.
Diffstat (limited to 't/convert-compact.t')
-rw-r--r-- | t/convert-compact.t | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/t/convert-compact.t b/t/convert-compact.t index e51eadcf..92a6a9c5 100644 --- a/t/convert-compact.t +++ b/t/convert-compact.t @@ -37,6 +37,8 @@ my $mime = PublicInbox::MIME->create( body => "hello world\n", ); ok($im->add($mime), 'added one message'); +ok($im->remove($mime), 'remove message'); +ok($im->add($mime), 'added message again'); $im->done; PublicInbox::SearchIdx->new($ibx, 1)->index_sync; @@ -77,6 +79,7 @@ $cmd = [ 'public-inbox-compact', "$tmpdir/v2" ]; my $env = { NPROC => 2 }; ok(PublicInbox::Import::run_die($cmd, $env, $rdr), 'v2 compact works'); $ibx->{mainrepo} = "$tmpdir/v2"; +$ibx->{version} = 2; my $v2w = PublicInbox::V2Writable->new($ibx); is($v2w->{partitions}, 1, "only one partition in compacted repo"); @@ -96,5 +99,8 @@ foreach (@xdir) { is($st[2] & 07777, -f _ ? 0444 : 0755, 'sharedRepository respected after v2 compact'); } +my $res = $ibx->recent({limit => 1000}); +is($res->{msgs}->[0]->{mid}, 'a-mid@b', 'message exists in history'); +is(scalar @{$res->{msgs}}, 1, 'only one message in history'); done_testing(); |