]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/solver_git.t
tests: no IPv6 on old Net::NNTP, Mail::IMAPClient, HTTP::Tiny
[public-inbox.git] / t / solver_git.t
index 99ffb9e353cb691c125e5866ebfdfc8474e84f90..958af065509d1aa660990ea727dcf4dd56085a4d 100644 (file)
@@ -1,11 +1,12 @@
 #!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;
 use PublicInbox::TestCommon;
 use Cwd qw(abs_path);
 require_git(2.6);
+use PublicInbox::ContentHash qw(git_sha);
 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)});
@@ -16,17 +17,129 @@ chomp $git_dir;
 $git_dir = abs_path($git_dir);
 
 use_ok "PublicInbox::$_" for (qw(Inbox V2Writable Git SolverGit WWW));
+my $patch2 = eml_load 't/solve/0002-rename-with-modifications.patch';
+my $patch2_oid = git_sha(1, $patch2)->hexdigest;
 
 my ($tmpdir, $for_destroy) = tmpdir();
 my $ibx = create_inbox 'v2', version => 2,
                        indexlevel => 'medium', sub {
        my ($im) = @_;
        $im->add(eml_load 't/solve/0001-simple-mod.patch') or BAIL_OUT;
-       $im->add(eml_load 't/solve/0002-rename-with-modifications.patch') or
-               BAIL_OUT;
+       $im->add($patch2) or BAIL_OUT;
 };
+my $md = "$tmpdir/md";
+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_tag = 'cb7c42b1e15577ed2215356a2bf925aef59cdd8d';
 my $v1_0_0_tag_short = substr($v1_0_0_tag, 0, 16);
+my $expect = '69df7d565d49fbaaeb0a067910f03dc22cd52bd0';
+my $non_existent = 'ee5e32211bf62ab6531bdf39b84b6920d0b6775a';
+
+test_lei({tmpdir => "$tmpdir/blob"}, sub {
+       lei_ok('blob', '--mail', $patch2_oid, '-I', $ibx->{inboxdir},
+               \'--mail works for existing oid');
+       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');
+       my $prev = $lei_out;
+       lei_ok(qw(blob --no-mail 69df7d5 -I), $ibx->{inboxdir});
+       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},
+                       "--git-dir=$git_dir");
+               is($lei_out, $prev, '--git-dir works');
+
+               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
+       lei_ok('blob', $v1_0_0_tag, '-I', $ibx->{inboxdir});
+       lei_ok('blob', $v1_0_0_tag_short, '-I', $ibx->{inboxdir});
+});
+
+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 {
+       lei_ok(qw(index), $md);
+       lei_ok(qw(blob 69df7d5)); # hits LeiSearch->smsg_eml -> lms->local_blob
+});
+
 my $git = PublicInbox::Git->new($git_dir);
 $ibx->{-repo_objs} = [ $git ];
 my $res;
@@ -38,7 +151,6 @@ $solver->solve($psgi_env, $log, '69df7d5', {});
 ok($res, 'solved a blob!');
 my $wt_git = $res->[0];
 is(ref($wt_git), 'PublicInbox::Git', 'got a git object for the blob');
-my $expect = '69df7d565d49fbaaeb0a067910f03dc22cd52bd0';
 is($res->[1], $expect, 'resolved blob to unabbreviated identifier');
 is($res->[2], 'blob', 'type specified');
 is($res->[3], 4405, 'size returned');
@@ -148,7 +260,6 @@ EOF
        close $cfgfh or die;
        my $cfg = PublicInbox::Config->new($cfgpath);
        my $www = PublicInbox::WWW->new($cfg);
-       my $non_existent = 'ee5e32211bf62ab6531bdf39b84b6920d0b6775a';
        my $client = sub {
                my ($cb) = @_;
                my $mid = '20190401081523.16213-1-BOFH@YHBT.net';
@@ -176,13 +287,19 @@ 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');
        };
        test_psgi(sub { $www->call(@_) }, $client);
        SKIP: {
@@ -193,8 +310,20 @@ EOF
                my $cmd = [ qw(-httpd -W0), "--stdout=$out", "--stderr=$err" ];
                my $td = start_script($cmd, $env, { 3 => $sock });
                my ($h, $p) = tcp_host_port($sock);
-               local $ENV{PLACK_TEST_EXTERNALSERVER_URI} = "http://$h:$p";
+               my $url = "http://$h:$p";
+               local $ENV{PLACK_TEST_EXTERNALSERVER_URI} = $url;
                Plack::Test::ExternalServer::test_psgi(client => $client);
+               require_cmd('curl', 1) or skip 'no curl', 1;
+
+               mkdir "$tmpdir/ext" // xbail "mkdir $!";
+               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');
+               });
        }
 }