]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/solver_git.t
No ext_urls
[public-inbox.git] / t / solver_git.t
index 44cbbfdb2adb04aa34744801d6da97ff490acc9d..c65d9785cf1f8f0fb0bd92f547aa52b0f0ab37c2 100644 (file)
@@ -1,5 +1,5 @@
 #!perl -w
-# Copyright (C) 2019-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 v5.10.1;
@@ -7,9 +7,11 @@ use PublicInbox::TestCommon;
 use Cwd qw(abs_path);
 require_git(2.6);
 use PublicInbox::ContentHash qw(git_sha);
-use PublicInbox::Spawn qw(popen_rd which);
+use PublicInbox::Spawn qw(popen_rd);
 require_mods(qw(DBD::SQLite Search::Xapian Plack::Util));
-my $git_dir = xqx([qw(git rev-parse --git-dir)], undef, {2 => \(my $null)});
+my $rdr = { 2 => \(my $null) };
+my $git_dir = xqx([qw(git rev-parse --git-common-dir)], undef, $rdr);
+$git_dir = xqx([qw(git rev-parse --git-dir)], undef, $rdr) if $? != 0;
 $? == 0 or plan skip_all => "$0 must be run from a git working tree";
 chomp $git_dir;
 
@@ -32,6 +34,7 @@ File::Path::mkpath([map { $md.$_ } (qw(/ /cur /new /tmp))]);
 symlink(abs_path('t/solve/0001-simple-mod.patch'), "$md/cur/foo:2,") or
        xbail "symlink: $!";
 
+my $v1_0_0_rev = '8a918a8523bc9904123460f85999d75f6d604916';
 my $v1_0_0_tag = 'cb7c42b1e15577ed2215356a2bf925aef59cdd8d';
 my $v1_0_0_tag_short = substr($v1_0_0_tag, 0, 16);
 my $expect = '69df7d565d49fbaaeb0a067910f03dc22cd52bd0';
@@ -43,6 +46,7 @@ test_lei({tmpdir => "$tmpdir/blob"}, sub {
        is($lei_out, $patch2->as_string, 'blob matches');
        ok(!lei('blob', '--mail', '69df7d5', '-I', $ibx->{inboxdir}),
                "--mail won't run solver");
+       like($lei_err, qr/\b69df7d5\b/, 'OID in error by git(1)');
 
        lei_ok('blob', '69df7d5', '-I', $ibx->{inboxdir});
        is(git_sha(1, \$lei_out)->hexdigest, $expect, 'blob contents output');
@@ -51,6 +55,8 @@ test_lei({tmpdir => "$tmpdir/blob"}, sub {
        is($lei_out, $prev, '--no-mail works');
        ok(!lei(qw(blob -I), $ibx->{inboxdir}, $non_existent),
                        'non-existent blob fails');
+       my $abbrev = substr($non_existent, 0, 7);
+       like($lei_err, qr/could not find $abbrev/, 'failed abbreviation noted');
        SKIP: {
                skip '/.git exists', 1 if -e '/.git';
                lei_ok(qw(-C / blob 69df7d5 -I), $ibx->{inboxdir},
@@ -59,9 +65,13 @@ test_lei({tmpdir => "$tmpdir/blob"}, sub {
 
                ok(!lei(qw(-C / blob --no-cwd 69df7d5 -I), $ibx->{inboxdir}),
                        '--no-cwd works');
+               like($lei_err, qr/no --git-dir to try/,
+                       'lack of --git-dir noted');
 
                ok(!lei(qw(-C / blob -I), $ibx->{inboxdir}, $non_existent),
                        'non-existent blob fails');
+               like($lei_err, qr/no --git-dir to try/,
+                       'lack of --git-dir noted');
        }
 
        # fallbacks
@@ -73,6 +83,59 @@ test_lei({tmpdir => "$tmpdir/rediff"}, sub {
        lei_ok(qw(rediff -q -U9 t/solve/0001-simple-mod.patch));
        like($lei_out, qr!^\Q+++\E b/TODO\n@@ -103,9 \+103,11 @@!sm,
                'got more context with -U9');
+
+       my (undef, $re) = split(/\n\n/, $lei_out, 2);
+       $re =~ s/^/> /sgm;
+       substr($re, 0, 0, <<EOM);
+From: me\@example.com
+Subject: Re: awesome advice
+
+WEB DESIGN EXPERT wrote:
+EOM
+       lei_ok([qw(rediff --full-index -U16 --drq)], undef,
+               { 0 => \$re, %$lei_opt });
+       my $exp = <<'EOM';
+From: me@example.com
+Subject: Re: awesome advice
+
+EOM
+       like($lei_out, qr/\Q$exp\E/, '--drq preserved header');
+
+       # n.b. --drq can requote the attribution line ("So-and-so wrote:"),
+       # but it's probably not worth preventing...
+
+       $exp = <<'EOM';
+> ---
+>  TODO | 2 ++
+>  Ω    | 5 --
+>  1 file changed, 2 insertions(+)
+>
+> diff --git a/TODO b/TODO
+> index 605013e4904baabecd4a0a55997aebd8e8477a8f..69df7d565d49fbaaeb0a067910f03dc22cd52bd0 100644
+> --- a/TODO
+> +++ b/TODO
+> @@ -96,16 +96,18 @@ all need to be considered for everything we introduce)
+EOM
+       $exp =~ s/^>$/> /sgm; # re-add trailing white space
+       like($lei_out, qr/\Q$exp\E/, '--drq diffstat + context');
+
+       lei_ok(qw(rediff -q --full-index -U9 t/solve/bare.patch));
+       $exp = <<'EOM';
+diff --git a/script/public-inbox-extindex b/script/public-inbox-extindex
+old mode 100644
+new mode 100755
+index 15ac20eb871bf47697377e58a27db23102a38fca..771486c425b315bae70fd8a82d62ab0331e0a827
+--- a/script/public-inbox-extindex
++++ b/script/public-inbox-extindex
+@@ -1,13 +1,12 @@
+ #!perl -w
+EOM
+       like($lei_out, qr/\Q$exp\E/,
+               'preserve mode, regen header + context from -U0 patch');
+       is($lei_err, '', 'no warnings from bare patch');
+       my $e = { GIT_DIR => "$ENV{HOME}/.local/share/lei/store/ALL.git" };
+       my @x = xqx([qw(git cat-file --batch-all-objects --batch-check)], $e);
+       is_deeply(\@x, [], 'no objects stored') or diag explain(\@x);
 });
 
 test_lei({tmpdir => "$tmpdir/index-eml-only"}, sub {
@@ -184,6 +247,8 @@ SKIP: {
        my $cfgpath = "$tmpdir/httpd-config";
        open my $cfgfh, '>', $cfgpath or die;
        print $cfgfh <<EOF or die;
+[coderepo]
+       snapshots = tar.gz
 [publicinbox "$name"]
        address = $ibx->{-primary_address}
        inboxdir = $ibx->{inboxdir}
@@ -198,6 +263,16 @@ SKIP: {
        cgiturl = http://example.com/binfoo
 EOF
        close $cfgfh or die;
+       my $exp_digest;
+       {
+               my $exp = xqx([qw(git archive --format=tar.gz
+                               --prefix=public-inbox-1.0.0/ v1.0.0)],
+                               { GIT_DIR => $git_dir });
+               is($?, 0, 'no error from git archive');
+               ok(length($exp) > 1024, 'expected archive generated');
+               $exp_digest = git_sha(256, \$exp)->hexdigest;
+       };
+
        my $cfg = PublicInbox::Config->new($cfgpath);
        my $www = PublicInbox::WWW->new($cfg);
        my $client = sub {
@@ -218,7 +293,7 @@ EOF
                is($res->code, 404, 'failure with null OID');
 
                $res = $cb->(GET("/$name/$non_existent/s/"));
-               is($res->code, 404, 'failure with null OID');
+               is($res->code, 404, 'failure with non-existent OID');
 
                $res = $cb->(GET("/$name/$v1_0_0_tag/s/"));
                is($res->code, 200, 'shows commit (unabbreviated)');
@@ -227,38 +302,132 @@ EOF
                while (my ($label, $size) = each %bin) {
                        $res = $cb->(GET("/$name/$oid{$label}/s/"));
                        is($res->code, 200, "$label binary file");
-                       ok(index($res->content, "blob $size bytes") >= 0,
+                       ok(index($res->content,
+                               "blob $oid{$label} $size bytes") >= 0,
                                "showed $label binary blob size");
                        $res = $cb->(GET("/$name/$oid{$label}/s/raw"));
                        is($res->code, 200, "$label raw binary download");
                        is($res->content, "\0" x $size,
                                "$label content matches");
                }
+               my $utf8 = 'e022d3377fd2c50fd9931bf96394728958a90bf3';
+               $res = $cb->(GET("/$name/$utf8/s/"));
+               is($res->code, 200, 'shows commit w/ utf8.eml');
+               like($res->content, qr/El&#233;anor/,
+                               'UTF-8 commit shown properly');
+
+               # WwwCoderepo
+               my $olderr;
+               if (defined $ENV{PLACK_TEST_EXTERNALSERVER_URI}) {
+                       ok(!-s "$tmpdir/stderr.log",
+                               'nothing in stderr.log, yet');
+               } else {
+                       open $olderr, '>&', \*STDERR or xbail "open: $!";
+                       open STDERR, '+>>', "$tmpdir/stderr.log" or
+                               xbail "open: $!";
+               }
+               $res = $cb->(GET('/binfoo/'));
+               defined($ENV{PLACK_TEST_EXTERNALSERVER_URI}) or
+                       open STDERR, '>&', $olderr or xbail "open: $!";
+               is($res->code, 200, 'coderepo summary (binfoo)');
+               ok(!-s "$tmpdir/stderr.log");
+               $res = $cb->(GET('/public-inbox/'));
+               is($res->code, 200, 'coderepo summary (public-inbox)');
+
+               my $tip = 'invalid-'.int(rand(0xdeadbeef));
+               $res = $cb->(GET('/public-inbox/?h='.$tip));
+               is($res->code, 200, 'coderepo summary on dead branch');
+               like($res->content, qr/no commits in `\Q$tip\E', yet/,
+                       'lack of commits noted');
+
+               $res = $cb->(GET('/public-inbox'));
+               is($res->code, 301, 'redirected');
+
+               my $fn = 'public-inbox-1.0.0.tar.gz';
+               $res = $cb->(GET("/public-inbox/snapshot/$fn"));
+               is($res->code, 200, 'tar.gz snapshot');
+               is($res->header('Content-Disposition'),
+                       qq'inline; filename="$fn"', 'c-d header');
+               is($res->header('ETag'), qq'"$v1_0_0_rev"', 'etag header');
+
+               my $got = $res->content;
+               is(git_sha(256, \$got)->hexdigest, $exp_digest,
+                       "content matches installed `git archive' output");
+               undef $got;
+
+               $fn = 'public-inbox-1.0.2.tar.gz';
+               $res = $cb->(GET("/public-inbox/snapshot/$fn"));
+               is($res->code, 404, '404 on non-existent tag');
+
+               $fn = 'public-inbox-1.0.0.tar.bz2';
+               $res = $cb->(GET("/public-inbox/snapshot/$fn"));
+               is($res->code, 404, '404 on unconfigured snapshot format');
+
+               $res = $cb->(GET('/public-inbox/atom/'));
+               is($res->code, 200, 'Atom feed');
+               SKIP: {
+                       require_mods('XML::TreePP', 1);
+                       my $t = XML::TreePP->new->parse($res->content);
+                       is(scalar @{$t->{feed}->{entry}}, 50,
+                               'got 50 entries');
+
+                       $res = $cb->(GET('/public-inbox/atom/COPYING'));
+                       is($res->code, 200, 'file Atom feed');
+                       $t = XML::TreePP->new->parse($res->content);
+                       ok($t->{feed}->{entry}, 'got entry');
+
+                       $res = $cb->(GET('/public-inbox/atom/README.md'));
+                       is($res->code, 404, '404 on missing file Atom feed');
+
+                       $res = $cb->(GET('/public-inbox/atom/?h=gone'));
+                       is($res->code, 404, '404 on missing Atom feed branch');
+               }
+
+               $res = $cb->(GET('/public-inbox/tree/'));
+               is($res->code, 200, 'got 200 for root listing');
+               $got = $res->content;
+               like($got, qr/\bgit ls-tree\b/, 'ls-tree help shown');
+
+               $res = $cb->(GET('/public-inbox/tree/README'));
+               is($res->code, 200, 'got 200 for regular file');
+               $got = $res->content;
+               like($got, qr/\bgit show\b/, 'git show help shown');
+
+               $res = $cb->(GET('/public-inbox/tree/Documentation'));
+               is($res->code, 200, 'got 200 for a directory');
+               $got = $res->content;
+               like($got, qr/\bgit ls-tree\b/, 'ls-tree help shown');
+
+               $res = $cb->(GET('/public-inbox/tree/?h=no-branch'));
+               is($res->code, 404, 'got 404 for non-existent ref root');
+               $res = $cb->(GET('/public-inbox/tree/README?h=no-file'));
+               is($res->code, 404, 'got 404 for non-existent ref README');
+               $res = $cb->(GET('/public-inbox/tree/Documentation?h=no-dir'));
+               is($res->code, 404, 'got 404 for non-existent ref directory');
+
+               $res = $cb->(GET('/public-inbox/tags.atom'));
+               is($res->code, 200, 'Atom feed');
+               SKIP: {
+                       require_mods('XML::TreePP', 1);
+                       my $t = XML::TreePP->new->parse($res->content);
+                       ok(scalar @{$t->{feed}->{entry}}, 'got tag entries');
+               }
        };
        test_psgi(sub { $www->call(@_) }, $client);
+       my $env = { PI_CONFIG => $cfgpath, TMPDIR => $tmpdir };
+       test_httpd($env, $client, 7, sub {
        SKIP: {
-               require_mods(qw(Plack::Test::ExternalServer), 7);
-               my $env = { PI_CONFIG => $cfgpath };
-               my $sock = tcp_server() or die;
-               my ($out, $err) = map { "$tmpdir/std$_.log" } qw(out err);
-               my $cmd = [ qw(-httpd -W0), "--stdout=$out", "--stderr=$err" ];
-               my $td = start_script($cmd, $env, { 3 => $sock });
-               my ($h, $p) = tcp_host_port($sock);
-               my $url = "http://$h:$p";
-               local $ENV{PLACK_TEST_EXTERNALSERVER_URI} = $url;
-               Plack::Test::ExternalServer::test_psgi(client => $client);
-               skip 'no curl', 1 unless which('curl');
-
+               require_cmd('curl', 1) or skip 'no curl', 1;
                mkdir "$tmpdir/ext" // xbail "mkdir $!";
+               my $rurl = "$ENV{PLACK_TEST_EXTERNALSERVER_URI}/$name";
                test_lei({tmpdir => "$tmpdir/ext"}, sub {
-                       my $rurl = "$url/$name";
                        lei_ok(qw(blob --no-mail 69df7d5 -I), $rurl);
                        is(git_sha(1, \$lei_out)->hexdigest, $expect,
                                'blob contents output');
                        ok(!lei(qw(blob -I), $rurl, $non_existent),
                                        'non-existent blob fails');
                });
-       }
+       }});
 }
 
 done_testing();