]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/lei-q-save.t
t/lei*: check error messages on failures
[public-inbox.git] / t / lei-q-save.t
index 170f7ce5642067d2cc8e2c974b3120a1b28cda00..b1ca4e92ab656ea1ff6792d05b57bf870e895814 100644 (file)
@@ -3,6 +3,8 @@
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 use strict; use v5.10.1; use PublicInbox::TestCommon;
 use PublicInbox::Smsg;
+use List::Util qw(sum);
+
 my $doc1 = eml_load('t/plack-qp.eml');
 $doc1->header_set('Date', PublicInbox::Smsg::date({ds => time - (86400 * 5)}));
 my $doc2 = eml_load('t/utf8.eml');
@@ -23,28 +25,31 @@ test_lei(sub {
        my $home = $ENV{HOME};
        my $in = $doc1->as_string;
        lei_ok [qw(import -q -F eml -)], undef, { 0 => \$in, %$lei_opt };
-       lei_ok qw(q -q --save z:0.. d:last.week..), '-o', "$home/md/";
+       lei_ok qw(q -q --save z:0.. d:last.week..), '-o', "MAILDIR:$home/md/";
        my %before = map { $_ => 1 } glob("$home/md/cur/*");
-       is_deeply(eml_load((keys %before)[0]), $doc1, 'doc1 matches');
+       my $f = (keys %before)[0] or xbail({before => \%before});
+       is_deeply(eml_load($f), $doc1, 'doc1 matches');
+       lei_ok qw(ls-mail-sync);
+       is($lei_out, "maildir:$home/md\n", 'canonicalized mail sync name');
 
        my @s = glob("$home/.local/share/lei/saved-searches/md-*");
        is(scalar(@s), 1, 'got one saved search');
        my $cfg = PublicInbox::Config->new("$s[0]/lei.saved-search");
+       is($cfg->{'lei.q.output'}, "maildir:$home/md", 'canonicalized output');
        is_deeply($cfg->{'lei.q'}, ['z:0..', 'd:last.week..'],
                'store relative time, not parsed (absolute) timestamp');
 
        # ensure "lei up" works, since it compliments "lei q --save"
        $in = $doc2->as_string;
        lei_ok [qw(import -q -F eml -)], undef, { 0 => \$in, %$lei_opt };
-       opendir my $dh, '.' or xbail "opendir .: $!";
        lei_ok qw(up -q md -C), $home;
        lei_ok qw(up -q . -C), "$home/md";
        lei_ok qw(up -q), "/$home/md";
-       chdir($dh) or xbail "fchdir . $!";
-       my %after = map { $_ => 1 } glob("$home/md/cur/*");
+       my %after = map { $_ => 1 } glob("$home/md/{new,cur}/*");
        is(delete $after{(keys(%before))[0]}, 1, 'original message kept');
        is(scalar(keys %after), 1, 'one new message added');
-       is_deeply(eml_load((keys %after)[0]), $doc2, 'doc2 matches');
+       $f = (keys %after)[0] or xbail({after => \%after});
+       is_deeply(eml_load($f), $doc2, 'doc2 matches');
 
        # check stdin
        lei_ok [qw(q --save - -o), "mboxcl2:mbcl2" ],
@@ -64,18 +69,19 @@ test_lei(sub {
        ok(-s "$home/mbcl2" > $size, 'size increased after up');
 
        ok(!lei(qw(up -q), $home), 'up fails w/o --save');
+       like($lei_err, qr/--save was not used/, 'error noted --save');
 
        lei_ok qw(ls-search); my @d = split(/\n/, $lei_out);
        lei_ok qw(ls-search -z); my @z = split(/\0/, $lei_out);
        is_deeply(\@d, \@z, '-z output matches non-z');
-       is_deeply(\@d, [ "$home/mbcl2", "$home/md/" ],
+       is_deeply(\@d, [ "$home/mbcl2", "$home/md" ],
                'ls-search output alphabetically sorted');
        lei_ok qw(ls-search -l);
        my $json = PublicInbox::Config->json->decode($lei_out);
        ok($json && $json->[0]->{output}, 'JSON has output');
        lei_ok qw(_complete lei up);
        like($lei_out, qr!^\Q$home/mbcl2\E$!sm, 'complete got mbcl2 output');
-       like($lei_out, qr!^\Q$home/md/\E$!sm, 'complete got maildir output');
+       like($lei_out, qr!^\Q$home/md\E$!sm, 'complete got maildir output');
 
        unlink("$home/mbcl2") or xbail "unlink $!";
        lei_ok qw(_complete lei up);
@@ -89,7 +95,6 @@ test_lei(sub {
        print $mb $pre_existing;
        close $mb or xbail "close: $!";
        lei_ok(qw(q --save -o mboxrd:mbrd m:qp@example.com -C), $home);
-       chdir($dh) or xbail "fchdir . $!";
        open $mb, '<', "$home/mbrd" or xbail "open $!";
        is_deeply([grep(/pre-existing/, <$mb>)], [],
                'pre-existing messsage gone w/o augment');
@@ -103,7 +108,6 @@ test_lei(sub {
        print $mb $pre_existing;
        close $mb or xbail "close: $!";
        lei_ok(qw(q -a --save -o mboxrd:mbrd-aug m:qp@example.com -C), $home);
-       chdir($dh) or xbail "fchdir . $!";
        open $mb, '<', "$home/mbrd-aug" or xbail "open $!";
        $mb = do { local $/; <$mb> };
        like($mb, qr/pre-existing/, 'pre-existing message preserved w/ -a');
@@ -112,6 +116,8 @@ test_lei(sub {
        lei_ok(qw(up --all=local));
 
        ok(!lei(qw(forget-search), "$home/bogus"), 'bogus forget');
+       like($lei_err, qr/--save was not used/, 'error noted --save');
+
        lei_ok qw(_complete lei forget-search);
        like($lei_out, qr/mbrd-aug/, 'forget-search completion');
        lei_ok(qw(forget-search -v), "$home/mbrd-aug");
@@ -121,6 +127,7 @@ test_lei(sub {
        unlike($lei_out, qr/mbrd-aug/,
                'forget-search completion cleared after forget');
        ok(!lei('up', "$home/mbrd-aug"), 'lei up fails after forget');
+       like($lei_err, qr/--save was not used/, 'error noted --save');
 
        # dedupe=mid
        my $o = "$home/dd-mid";
@@ -156,7 +163,7 @@ test_lei(sub {
        $im->add(PublicInbox::Eml->new($diff));
        $im->done;
        lei_ok('up', $o);
-       @m = glob("$o/cur/*");
+       @m = glob("$o/{new,cur}/*");
        is(scalar(@m), 2, 'got 2nd result due to different OID');
 
        SKIP: {
@@ -164,5 +171,36 @@ test_lei(sub {
                        skip "symlinks not supported in $home?: $!", 1;
                lei_ok('up', "$home/ln -s");
        };
+
+       my $v2 = "$home/v2"; # v2: as an output destination
+       my (@before, @after);
+       require PublicInbox::MboxReader;
+       lei_ok(qw(q z:0.. -o), "v2:$v2");
+       lei_ok(qw(q z:0.. -o), "mboxrd:$home/before", '--only', $v2, '-j1,1');
+       open my $fh, '<', "$home/before";
+       PublicInbox::MboxReader->mboxrd($fh, sub { push @before, $_[0] });
+       isnt(scalar(@before), 0, 'initial v2 written');
+       my $orig = sum(map { -f $_ ? -s _ : () } (
+                       glob("$v2/git/0.git/objects/*/*")));
+       lei_ok(qw(import t/data/0001.patch));
+       lei_ok 'up', $v2;
+       lei_ok(qw(q z:0.. -o), "mboxrd:$home/after", '--only', $v2, '-j1,1');
+       open $fh, '<', "$home/after";
+       PublicInbox::MboxReader->mboxrd($fh, sub { push @after, $_[0] });
+
+       my $last = shift @after;
+       $last->header_set('Status');
+       is_deeply($last, eml_load('t/data/0001.patch'), 'lei up worked on v2');
+       is_deeply(\@before, \@after, 'got same results');
+
+       my $v2s = "$home/v2s";
+       lei_ok(qw(q --shared z:0.. -o), "v2:$v2s");
+       my $shared = sum(map { -f $_ ? -s _ : () } (
+                       glob("$v2s/git/0.git/objects/*/*")));
+       ok($shared < $orig, 'fewer bytes stored with --shared') or
+               diag "shared=$shared orig=$orig";
+
+       lei_ok([qw(edit-search), $v2s], { VISUAL => 'cat', EDITOR => 'cat' });
+       like($lei_out, qr/^\[lei/sm, 'edit-search can cat');
 });
 done_testing;