]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/v2reindex.t
t/v[12]reindex.t: Place expected second in Xapian tests
[public-inbox.git] / t / v2reindex.t
index 5bc307f1cac11ed34c3c3ec6e4c8258544c85652..67d8be787a782c084ac42e763a4a5f98378214af 100644 (file)
@@ -19,8 +19,8 @@ my $ibx_config = {
        name => 'test-v2writable',
        version => 2,
        -primary_address => 'test@example.com',
+       indexlevel => 'full',
 };
-my $ibx = PublicInbox::Inbox->new($ibx_config);
 my $mime = PublicInbox::MIME->create(
        header => [
                From => 'a@example.com',
@@ -31,39 +31,57 @@ my $mime = PublicInbox::MIME->create(
        body => "hello world\n",
 );
 local $ENV{NPROC} = 2;
-my $im = PublicInbox::V2Writable->new($ibx, 1);
-foreach my $i (1..10) {
-       $mime->header_set('Message-Id', "<$i\@example.com>");
-       ok($im->add($mime), "message $i added");
-       if ($i == 4) {
+my $minmax;
+{
+       my %config = %$ibx_config;
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx, 1);
+       foreach my $i (1..10) {
+               $mime->header_set('Message-Id', "<$i\@example.com>");
+               ok($im->add($mime), "message $i added");
+               if ($i == 4) {
+                       $im->remove($mime);
+               }
+       }
+
+       if ('test remove later') {
+               $mime->header_set('Message-Id', "<5\@example.com>");
                $im->remove($mime);
        }
-}
 
-if ('test remove later') {
-       $mime->header_set('Message-Id', "<5\@example.com>");
-       $im->remove($mime);
+       $im->done;
+       $minmax = [ $ibx->mm->minmax ];
+       ok(defined $minmax->[0] && defined $minmax->[1], 'minmax defined');
+       is_deeply($minmax, [ 1, 10 ], 'minmax as expected');
 }
 
-$im->done;
-my $minmax = [ $ibx->mm->minmax ];
-ok(defined $minmax->[0] && defined $minmax->[1], 'minmax defined');
-is_deeply($minmax, [ 1, 10 ], 'minmax as expected');
+{
+       my %config = %$ibx_config;
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx, 1);
+       eval { $im->index_sync({reindex => 1}) };
+       is($@, '', 'no error from reindexing');
+       $im->done;
 
-eval { $im->index_sync({reindex => 1}) };
-is($@, '', 'no error from reindexing');
-$im->done;
+       delete $ibx->{mm};
+       is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
+}
 
 my $xap = "$mainrepo/xap".PublicInbox::Search::SCHEMA_VERSION();
 remove_tree($xap);
 ok(!-d $xap, 'Xapian directories removed');
-eval { $im->index_sync({reindex => 1}) };
-is($@, '', 'no error from reindexing');
-$im->done;
-ok(-d $xap, 'Xapian directories recreated');
+{
+       my %config = %$ibx_config;
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx, 1);
+       eval { $im->index_sync({reindex => 1}) };
+       is($@, '', 'no error from reindexing');
+       $im->done;
+       ok(-d $xap, 'Xapian directories recreated');
 
-delete $ibx->{mm};
-is_deeply($minmax, [ $ibx->mm->minmax ], 'minmax unchanged');
+       delete $ibx->{mm};
+       is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
+}
 
 ok(unlink "$mainrepo/msgmap.sqlite3", 'remove msgmap');
 remove_tree($xap);
@@ -71,13 +89,70 @@ ok(!-d $xap, 'Xapian directories removed again');
 {
        my @warn;
        local $SIG{__WARN__} = sub { push @warn, @_ };
+       my %config = %$ibx_config;
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx, 1);
        eval { $im->index_sync({reindex => 1}) };
        is($@, '', 'no error from reindexing without msgmap');
        is(scalar(@warn), 0, 'no warnings from reindexing');
        $im->done;
        ok(-d $xap, 'Xapian directories recreated');
        delete $ibx->{mm};
-       is_deeply($minmax, [ $ibx->mm->minmax ], 'minmax unchanged');
+       is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
+}
+
+my %sizes;
+ok(unlink "$mainrepo/msgmap.sqlite3", 'remove msgmap');
+remove_tree($xap);
+ok(!-d $xap, 'Xapian directories removed again');
+{
+       my @warn;
+       local $SIG{__WARN__} = sub { push @warn, @_ };
+       my %config = %$ibx_config;
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx, 1);
+       eval { $im->index_sync({reindex => 1}) };
+       is($@, '', 'no error from reindexing without msgmap');
+       is_deeply(\@warn, [], 'no warnings');
+       $im->done;
+       ok(-d $xap, 'Xapian directories recreated');
+       delete $ibx->{mm};
+       is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
+       my $mset = $ibx->search->query('"hello world"', {mset=>1});
+       isnt($mset->size, 0, "phrase search succeeds on indexlevel=full");
+       for (<"$xap/*/*">) { $sizes{$ibx->{indexlevel}} += -s _ if -f $_ }
+}
+
+ok(unlink "$mainrepo/msgmap.sqlite3", 'remove msgmap');
+remove_tree($xap);
+ok(!-d $xap, 'Xapian directories removed again');
+{
+       my @warn;
+       local $SIG{__WARN__} = sub { push @warn, @_ };
+       my %config = %$ibx_config;
+       $config{indexlevel} = 'medium';
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx);
+       eval { $im->index_sync({reindex => 1}) };
+       is($@, '', 'no error from reindexing without msgmap');
+       is_deeply(\@warn, [], 'no warnings');
+       $im->done;
+       ok(-d $xap, 'Xapian directories recreated');
+       delete $ibx->{mm};
+       is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
+
+       if (0) {
+               # not sure why, but Xapian seems to fallback to terms and
+               # phrase searches still work
+               delete $ibx->{search};
+               my $mset = $ibx->search->query('"hello world"', {mset=>1});
+               is($mset->size, 0, 'phrase search does not work on medium');
+       }
+
+       my $mset = $ibx->search->query('hello world', {mset=>1});
+       isnt($mset->size, 0, "normal search works on indexlevel=medium");
+       for (<"$xap/*/*">) { $sizes{$ibx->{indexlevel}} += -s _ if -f $_ }
+       ok($sizes{full} > $sizes{medium}, 'medium is smaller than full');
 }
 
 ok(unlink "$mainrepo/msgmap.sqlite3", 'remove msgmap');
@@ -86,13 +161,21 @@ ok(!-d $xap, 'Xapian directories removed again');
 {
        my @warn;
        local $SIG{__WARN__} = sub { push @warn, @_ };
+       my %config = %$ibx_config;
+       $config{indexlevel} = 'basic';
+       my $ibx = PublicInbox::Inbox->new(\%config);
+       my $im = PublicInbox::V2Writable->new($ibx);
        eval { $im->index_sync({reindex => 1}) };
        is($@, '', 'no error from reindexing without msgmap');
        is_deeply(\@warn, [], 'no warnings');
        $im->done;
        ok(-d $xap, 'Xapian directories recreated');
        delete $ibx->{mm};
-       is_deeply($minmax, [ $ibx->mm->minmax ], 'minmax unchanged');
+       is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
+       my $mset = $ibx->search->query('hello', {mset=>1});
+       is($mset->size, 0, "search fails on indexlevel='basic'");
+       for (<"$xap/*/*">) { $sizes{$ibx->{indexlevel}} += -s _ if -f $_ }
+       ok($sizes{medium} > $sizes{basic}, 'basic is smaller than medium');
 }
 
 done_testing();