]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/lei-q-save.t
No ext_urls
[public-inbox.git] / t / lei-q-save.t
index 694b33b2e1998914fc0e9272a9b69e8f8dd3327c..d09c839798325d8eb2c7576e00a59adc0bfc0485 100644 (file)
@@ -1,9 +1,10 @@
 #!perl -w
-# Copyright (C) 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 PublicInbox::Smsg;
 use List::Util qw(sum);
+use File::Path qw(remove_tree);
 
 my $doc1 = eml_load('t/plack-qp.eml');
 $doc1->header_set('Date', PublicInbox::Smsg::date({ds => time - (86400 * 5)}));
@@ -11,7 +12,7 @@ my $doc2 = eml_load('t/utf8.eml');
 $doc2->header_set('Date', PublicInbox::Smsg::date({ds => time - (86400 * 4)}));
 my $doc3 = eml_load('t/msg_iter-order.eml');
 $doc3->header_set('Date', PublicInbox::Smsg::date({ds => time - (86400 * 4)}));
-
+my $cat_env = { VISUAL => 'cat', EDITOR => 'cat' };
 my $pre_existing = <<'EOF';
 From x Mon Sep 17 00:00:00 2001
 Message-ID: <import-before@example.com>
@@ -25,7 +26,7 @@ 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', "MAILDIR:$home/md/";
+       lei_ok qw(q -q z:0.. d:last.week..), '-o', "MAILDIR:$home/md/";
        my %before = map { $_ => 1 } glob("$home/md/cur/*");
        my $f = (keys %before)[0] or xbail({before => \%before});
        is_deeply(eml_load($f), $doc1, 'doc1 matches');
@@ -52,8 +53,8 @@ test_lei(sub {
        is_deeply(eml_load($f), $doc2, 'doc2 matches');
 
        # check stdin
-       lei_ok [qw(q --save - -o), "mboxcl2:mbcl2" ],
-               undef, { -C => $home, %$lei_opt, 0 => \'d:last.week..'};
+       lei_ok [qw(q - -o), "mboxcl2:mbcl2" ], undef,
+               { -C => $home, %$lei_opt, 0 => \'d:last.week..'};
        @s = glob("$home/.local/share/lei/saved-searches/mbcl2-*");
        $cfg = PublicInbox::Config->new("$s[0]/lei.saved-search");
        is_deeply $cfg->{'lei.q'}, 'd:last.week..',
@@ -68,7 +69,12 @@ test_lei(sub {
        lei_ok([qw(up mbcl2)], undef, { -C => $home, %$lei_opt });
        ok(-s "$home/mbcl2" > $size, 'size increased after up');
 
-       ok(!lei(qw(up -q), $home), 'up fails w/o --save');
+       ok(!lei(qw(up -q), $home), 'up fails on unknown dir');
+       like($lei_err, qr/--no-save was used/, 'error noted --no-save');
+
+       lei_ok(qw(q --no-save d:last.week.. -q -o), "$home/no-save");
+       ok(!lei(qw(up -q), "$home/no-save"), 'up fails on --no-save');
+       like($lei_err, qr/--no-save was used/, 'error noted --no-save');
 
        lei_ok qw(ls-search); my @d = split(/\n/, $lei_out);
        lei_ok qw(ls-search -z); my @z = split(/\0/, $lei_out);
@@ -93,7 +99,7 @@ test_lei(sub {
        open my $mb, '>', "$home/mbrd" or xbail "open $!";
        print $mb $pre_existing;
        close $mb or xbail "close: $!";
-       lei_ok(qw(q --save -o mboxrd:mbrd m:qp@example.com -C), $home);
+       lei_ok(qw(q -o mboxrd:mbrd m:qp@example.com -C), $home);
        open $mb, '<', "$home/mbrd" or xbail "open $!";
        is_deeply([grep(/pre-existing/, <$mb>)], [],
                'pre-existing messsage gone w/o augment');
@@ -106,7 +112,7 @@ test_lei(sub {
        open $mb, '>', "$home/mbrd-aug" or xbail "open $!";
        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);
+       lei_ok(qw(q -a -o mboxrd:mbrd-aug m:qp@example.com -C), $home);
        open $mb, '<', "$home/mbrd-aug" or xbail "open $!";
        $mb = do { local $/; <$mb> };
        like($mb, qr/pre-existing/, 'pre-existing message preserved w/ -a');
@@ -115,6 +121,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");
@@ -124,12 +132,13 @@ 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/--no-save was used/, 'error noted --no-save');
 
        # dedupe=mid
        my $o = "$home/dd-mid";
        $in = $doc2->as_string . "\n-------\nappended list sig\n";
        lei_ok [qw(import -q -F eml -)], undef, { 0 => \$in, %$lei_opt };
-       lei_ok(qw(q --dedupe=mid --save m:testmessage@example.com -o), $o);
+       lei_ok(qw(q --dedupe=mid m:testmessage@example.com -o), $o);
        my @m = glob("$o/cur/*");
        is(scalar(@m), 1, '--dedupe=mid w/ --save');
        $in = $doc2->as_string . "\n-------\nanother list sig\n";
@@ -139,8 +148,7 @@ test_lei(sub {
 
        for my $dd (qw(content)) {
                $o = "$home/dd-$dd";
-               lei_ok(qw(q --save m:testmessage@example.com -o), $o,
-                               "--dedupe=$dd");
+               lei_ok(qw(q m:testmessage@example.com -o), $o, "--dedupe=$dd");
                @m = glob("$o/cur/*");
                is(scalar(@m), 3, 'all 3 matches with dedupe='.$dd);
        }
@@ -149,7 +157,7 @@ test_lei(sub {
        $o = "$home/dd-oid";
        my $ibx = create_inbox 'ibx', indexlevel => 'medium',
                        tmpdir => "$home/v1", sub {};
-       lei_ok(qw(q --save --dedupe=oid m:qp@example.com -o), $o,
+       lei_ok(qw(q --dedupe=oid m:qp@example.com -o), $o,
                '-I', $ibx->{inboxdir});
        @m = glob("$o/cur/*");
        is(scalar(@m), 1, 'got first result');
@@ -166,13 +174,21 @@ test_lei(sub {
                symlink($o, "$home/ln -s") or
                        skip "symlinks not supported in $home?: $!", 1;
                lei_ok('up', "$home/ln -s");
+               lei_ok('forget-search', "$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');
+       like($lei_err, qr/^# ([1-9][0-9]*) written to \Q$v2\E/sm,
+               'non-zero write output to stderr');
+       lei_ok('-C', $v2, qw(q z:0.. -o), "mboxrd:$home/before",
+               '--only', '.', '-j1,1');
+       lei_ok(['edit-search', "$home/before"], $cat_env);
+       like($lei_out, qr/^\tonly = \Q$v2\E$/sm,
+               'relative --only saved to absolute path');
+
        open my $fh, '<', "$home/before";
        PublicInbox::MboxReader->mboxrd($fh, sub { push @before, $_[0] });
        isnt(scalar(@before), 0, 'initial v2 written');
@@ -195,5 +211,80 @@ test_lei(sub {
                        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], $cat_env);
+       like($lei_out, qr/^\[lei/sm, 'edit-search can cat');
+
+       lei_ok('-C', "$home/v2s", qw(q -q -o ../s m:testmessage@example.com));
+       lei_ok qw(ls-search);
+       unlike $lei_out, qr{/\.\./s$}sm, 'relative path not in ls-search';
+       like $lei_out, qr{^\Q$home\E/s$}sm,
+               'absolute path appears in ls-search';
+       lei_ok qw(up ../s -C), "$home/v2s", \'relative lei up';
+       lei_ok qw(up), "$home/s", \'absolute lei up';
+
+       # mess up a config file
+       my @lss = glob("$home/" .
+               '.local/share/lei/saved-searches/*/lei.saved-search');
+       my $out = xqx([qw(git config -f), $lss[0], 'lei.q.output']);
+       xsys($^X, qw(-i -p -e), "s/\\[/\\0/", $lss[0])
+               and xbail "-ipe $lss[0]: $?";
+       lei_ok qw(ls-search);
+       like($lei_err, qr/bad config line.*?\Q$lss[0]\E/,
+               'git config parse error shown w/ lei ls-search');
+       lei_ok qw(up --all), \'up works with bad config';
+       like($lei_err, qr/bad config line.*?\Q$lss[0]\E/,
+               'git config parse error shown w/ lei up');
+       xsys($^X, qw(-i -p -e), "s/\\0/\\[/", $lss[0])
+               and xbail "-ipe $lss[0]: $?";
+       lei_ok qw(ls-search);
+       is($lei_err, '', 'no errors w/ fixed config');
+
+       like($lei_out, qr!\Q$home/after\E!, "`after' in ls-search");
+       remove_tree("$home/after");
+       lei_ok qw(forget-search --prune);
+       lei_ok qw(ls-search);
+       unlike($lei_out, qr!\Q$home/after\E!, "`after' pruned");
+
+       my $d = "$home/d";
+       lei_ok [qw(import -q -F eml)], undef,
+               {0 => \"Subject: do not call\n\n"};
+       lei_ok qw(q -o), $d, 's:do not call';
+
+       my @orig = glob("$d/*/*");
+       is(scalar(@orig), 1, 'got one message via argv');
+       lei_ok [qw(import -q -Feml)], undef,
+               {0 => \"Subject: do not ever call\n\n"};
+       lei_ok 'up', $d;
+       is_deeply([glob("$d/*/*")], \@orig, 'nothing written');
+       lei_ok [qw(import -q -Feml)], undef,
+               {0 => \"Subject: do not call, ever\n\n"};
+       lei_ok 'up', $d;
+       @after = glob("$d/*/*");
+       is(scalar(@after), 2, '2 total, messages, now');
+       is_deeply([glob("$d/cur/*")], \@orig, 'cur untouched');
+       my @new = glob("$d/new/*");
+       is(scalar(@new), 1, "new message written to `new'");
+       is(eml_load($new[0])->header('Subject'), 'do not call, ever',
+               'up retrieved correct message');
+
+       $d = "$home/d-stdin";
+       lei_ok [ qw(q -q -o), $d ], undef, { 0 => \'s:"do not ever call"' };
+       @orig = glob("$d/*/*");
+       is(scalar(@orig), 1, 'got one message via stdin');
+
+       lei_ok [qw(import -q -Feml)], undef,
+               {0 => \"Subject: do not fall or ever call\n\n"};
+       lei_ok [qw(import -q -Feml)], undef,
+               {0 => \"Subject: do not ever call, again\n\n"};
+       lei_ok 'up', $d;
+       @new = glob("$d/new/*");
+       is(scalar(@new), 1, "new message written to `new'") or do {
+               for (@new) { diag "$_ ".eml_load($_)->header('Subject') }
+       };
+       is_deeply([glob("$d/cur/*")], \@orig, 'cur untouched');
+       is(eml_load($new[0])->header('Subject'), 'do not ever call, again',
+               'up retrieved correct message');
+
 });
 done_testing;