]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/v2mirror.t
No ext_urls
[public-inbox.git] / t / v2mirror.t
index b0075fcc88702b14c954bb606252a6bcd186a7f9..c1c66d45a41d4c4c8f2e2ca1cf048ad5b372ddc3 100644 (file)
@@ -1,19 +1,20 @@
-# Copyright (C) 2018-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 use strict;
-use warnings;
-use Test::More;
+use v5.10.1;
 use PublicInbox::TestCommon;
-use File::Path qw(remove_tree);
+use File::Path qw(remove_tree make_path);
 use Cwd qw(abs_path);
+use Carp ();
+use PublicInbox::Spawn qw(which);
 require_git(2.6);
+require_cmd('curl');
 local $ENV{HOME} = abs_path('t');
+use IO::Uncompress::Gunzip qw(gunzip $GunzipError);
 
 # Integration tests for HTTP cloning + mirroring
 require_mods(qw(Plack::Util Plack::Builder
                HTTP::Date HTTP::Status Search::Xapian DBD::SQLite));
-use IO::Socket;
-use POSIX qw(dup2);
 use_ok 'PublicInbox::V2Writable';
 use PublicInbox::InboxWritable;
 use PublicInbox::Eml;
@@ -25,7 +26,8 @@ my $pi_config = "$tmpdir/config";
        open my $fh, '>', $pi_config or die "open($pi_config): $!";
        print $fh <<"" or die "print $pi_config: $!";
 [publicinbox "v2"]
-       inboxdir = $tmpdir/in
+; using "mainrepo" rather than "inboxdir" for v1.1.0-pre1 WWW compat below
+       mainrepo = $tmpdir/in
        address = test\@example.com
 
        close $fh or die "close($pi_config): $!";
@@ -64,11 +66,11 @@ $v2w->done;
 }
 $ibx->cleanup;
 
-my $sock = tcp_server();
+local $ENV{TEST_IPV4_ONLY} = 1; # plackup (below) doesn't do IPv6
+my $rdr = { 3 => tcp_server() };
 my @cmd = ('-httpd', '-W0', "--stdout=$tmpdir/out", "--stderr=$tmpdir/err");
-my $td = start_script(\@cmd, undef, { 3 => $sock });
-my ($host, $port) = tcp_host_port($sock);
-$sock = undef;
+my $td = start_script(\@cmd, undef, $rdr);
+my ($host, $port) = tcp_host_port(delete $rdr->{3});
 
 @cmd = (qw(-clone -q), "http://$host:$port/v2/", "$tmpdir/m");
 run_script(\@cmd) or xbail '-clone';
@@ -99,13 +101,12 @@ $ibx->cleanup;
 
 my @new_epochs;
 my $fetch_each_epoch = sub {
-       my $mf = "$tmpdir/m/manifest.js.gz";
-       if (my @st = stat($mf)) {
-               utime($st[8], $st[9] - 1, $mf) or xbail "utime $mf: $!";
-       }
        my %before = map { $_ => 1 } glob("$tmpdir/m/git/*");
-       run_script([qw(-fetch -q)], undef, {-C => "$tmpdir/m"}) or
-               xbail '-fetch fail';
+       run_script([qw(-fetch --exit-code -q)], undef, {-C => "$tmpdir/m"}) or
+               xbail('-fetch fail ',
+                       [ xqx([which('find'), "$tmpdir/m", qw(-type f -ls) ]) ],
+                       Carp::longmess());
+       is($?, 0, '--exit-code 0 after fetch updated');
        my @after = grep { !$before{$_} } glob("$tmpdir/m/git/*");
        push @new_epochs, @after;
 };
@@ -234,10 +235,168 @@ EOF
        is(scalar($mset->items), 0, 'large message not re-indexed');
 }
 ok(scalar(@new_epochs), 'new epochs were created and fetched');
+for my $d (@new_epochs) {
+       is(xqx(['git', "--git-dir=$d", 'config', qw(include.path)]),
+               "../../all.git/config\n",
+               'include.path set');
+}
+
+if ('test read-only epoch dirs') {
+       my @git = ('git', "--git-dir=$new_epochs[0]");
+       my $get_objs = [@git,
+               qw(cat-file --buffer --batch-check --batch-all-objects)];
+       my $before = [sort xqx($get_objs)];
+
+       remove_tree(map { "$new_epochs[0]/$_" } qw(objects refs/heads));
+       chmod(0555, $new_epochs[0]) or xbail "chmod: $!";
+
+       # force a refetch
+       unlink("$tmpdir/m/manifest.js.gz") or xbail "unlink: $!";
+
+       run_script([qw(-fetch -q)], undef, {-C => "$tmpdir/m"}) or
+               xbail '-fetch failed';
+
+       ok(!-d "$new_epochs[0]/objects", 'no objects after fetch to R/O dir');
 
-ok($td->kill, 'killed httpd');
-$td->join;
+       chmod(0755, $new_epochs[0]) or xbail "chmod: $!";
+       mkdir("$new_epochs[0]/objects") or xbail "mkdir: $!";
+       mkdir("$new_epochs[0]/refs/heads") or xbail "mkdir: $!";
+
+       my $err = '';
+       run_script([qw(-fetch -q)], undef, {-C => "$tmpdir/m", 2 => \$err}) or
+               xbail '-fetch failed '.$err;
+       is_deeply([ sort xqx($get_objs) ], $before,
+               'fetch restored objects once GIT_DIR became writable');
+}
+
+{
+       my $dst = "$tmpdir/partial";
+       run_script([qw(-clone -q --epoch=~0), "http://$host:$port/v2/", $dst]);
+       is($?, 0, 'no error from partial clone');
+       my @g = glob("$dst/git/*.git");
+       my @w = grep { -w $_ } @g;
+       my @r = grep { ! -w $_ } @g;
+       if ($> == 0) {
+               @w = grep { (stat($_))[2] & 0200 } @g;
+               @r = grep { !((stat($_))[2] & 0200) } @g;
+       }
+       is(scalar(@w), 1, 'one writable directory');
+       my ($w) = ($w[0] =~ m!/([0-9]+)\.git\z!);
+       is((grep {
+               m!/([0-9]+)\.git\z! or xbail "no digit in $_";
+               $w > ($1 + 0)
+       } @r), scalar(@r), 'writable epoch # exceeds read-only ones');
+       run_script([qw(-fetch -q)], undef, { -C => $dst });
+       is($?, 0, 'no error from partial fetch');
+       remove_tree($dst);
 
-done_testing();
+       run_script([qw(-clone -q --epoch=~1..),
+                       "http://$host:$port/v2/", $dst]);
+       my @g2 = glob("$dst/git/*.git") ;
+       is_deeply(\@g2, \@g, 'cloned again');
+       is(scalar(grep { (stat($_))[2] & 0200 } @g2), scalar(@w) + 1,
+               'got one more cloned epoch');
+
+       # make 0.git writable and fetch into it, relies on culled manifest
+       chmod(0755, $g2[0]) or xbail "chmod: $!";
+       my @before = glob("$g2[0]/objects/*/*");
+       run_script([qw(-fetch -q)], undef, { -C => $dst });
+       is($?, 0, 'no error from partial fetch');
+       my @after = glob("$g2[0]/objects/*/*");
+       ok(scalar(@before) < scalar(@after), 'fetched after chmod 0755 0.git');
+
+       # ensure culled manifest is maintained after fetch
+       gunzip("$dst/manifest.js.gz" => \(my $m), MultiStream => 1) or
+               xbail "gunzip: $GunzipError";
+       $m = PublicInbox::Config->json->decode($m);
+       for my $k (keys %$m) { # /$name/git/$N.git
+               my ($nr) = ($k =~ m!/git/([0-9]+)\.git\z!);
+               ok(-w "$dst/git/$nr.git", "writable $nr.git in manifest");
+       }
+       for my $ro (grep { !-w $_ } @g2) {
+               my ($nr) = ($ro =~ m!/git/([0-9]+)\.git\z!);
+               is(grep(m!/git/$nr\.git\z!, keys %$m), 0,
+                       "read-only $nr.git not in manifest")
+                       or xbail([sort keys %$m]);
+       }
+}
+
+my $err = '';
+my $oldrev = '0b3e19584c90d958a723ac2d3dec3f84f5513688~1';
+# 3e0e596105198cfa (wwwlisting: allow hiding entries from manifest, 2019-06-09)
+$oldrev = xqx([qw(git rev-parse), $oldrev], undef, { 2 => \$err });
+SKIP: {
+       skip("no detected public-inbox GIT_DIR ($err)", 1) if $?;
+       require_mods('Email::MIME', 1); # for legacy revision
+       # using plackup to test old PublicInbox::WWW since -httpd from
+       # back then relied on some packages we no longer depend on
+       my $plackup = require_cmd('plackup', 1) or skip('no plackup in path', 1);
+       require PublicInbox::Lock;
+       chomp $oldrev;
+       my ($base) = ($0 =~ m!\b([^/]+)\.[^\.]+\z!);
+       my $wt = "t/data-gen/$base.pre-manifest-$oldrev";
+       my $lk = bless { lock_path => __FILE__ }, 'PublicInbox::Lock';
+       $lk->lock_acquire;
+       my $psgi = "$wt/app.psgi";
+       if (!-f $psgi) { # checkout a pre-manifest.js.gz version
+               my $t = File::Temp->new(TEMPLATE => 'g-XXXX', TMPDIR => 1);
+               my $env = { GIT_INDEX_FILE => $t->filename };
+               xsys([qw(git read-tree), $oldrev], $env) and xbail 'read-tree';
+               xsys([qw(git checkout-index -a), "--prefix=$wt/"], $env)
+                       and xbail 'checkout-index';
+               my $f = "$wt/app.psgi.tmp.$$";
+               open my $fh, '>', $f or xbail $!;
+               print $fh <<'EOM' or xbail $!;
+use Plack::Builder;
+use PublicInbox::WWW;
+my $www = PublicInbox::WWW->new;
+builder { enable 'Head'; sub { $www->call(@_) } }
+EOM
+               close $fh or xbail $!;
+               rename($f, $psgi) or xbail $!;
+       }
+       $lk->lock_release;
+
+       $rdr->{run_mode} = 0;
+       $rdr->{-C} = $wt;
+       my $cmd = [$plackup, qw(-Enone -Ilib), "--host=$host", "--port=$port"];
+       $td->join('TERM');
+       open $rdr->{2}, '>>', "$tmpdir/plackup.err.log" or xbail "open: $!";
+       open $rdr->{1}, '>>&', $rdr->{2} or xbail "open: $!";
+       my $env = { PERL5LIB => 'lib', PERL_INLINE_DIRECTORY => undef };
+       $td = start_script($cmd, $env, $rdr);
+       # wait for plackup socket()+bind()+listen()
+       my %opt = ( Proto => 'tcp', Type => Socket::SOCK_STREAM(),
+               PeerAddr => "$host:$port" );
+       for (0..100) {
+               tick();
+               last if IO::Socket::INET->new(%opt);
+       }
+       IO::Socket::INET->new(%opt) or xbail "connect $host:$port: $!";
+       my $dst = "$tmpdir/scrape";
+       @cmd = (qw(-clone -q), "http://$host:$port/v2", $dst);
+       run_script(\@cmd, undef, { 2 => \($err = '') });
+       is($?, 0, 'scraping clone on old PublicInbox::WWW')
+               or diag $err;
+       my @g_all = glob("$dst/git/*.git");
+       ok(scalar(@g_all) > 1, 'cloned multiple epochs');
+
+       remove_tree($dst);
+       @cmd = (qw(-clone -q --epoch=~0), "http://$host:$port/v2", $dst);
+       run_script(\@cmd, undef, { 2 => \($err = '') });
+       is($?, 0, 'partial scraping clone on old PublicInbox::WWW');
+       my @g_last = grep { (stat($_))[2] & 0200 } glob("$dst/git/*.git");
+       is_deeply(\@g_last, [ $g_all[-1] ], 'partial clone of ~0 worked');
+
+       chmod(0755, $g_all[0]) or xbail "chmod $!";
+       my @before = glob("$g_all[0]/objects/*/*");
+       run_script([qw(-fetch -v)], undef, { -C => $dst, 2 => \($err = '') });
+       is($?, 0, 'scraping fetch on old PublicInbox::WWW') or diag $err;
+       my @after = glob("$g_all[0]/objects/*/*");
+       ok(scalar(@before) < scalar(@after),
+               'fetched 0.git after enabling write-bit');
+
+       $td->join('TERM');
+}
 
-1;
+done_testing;