X-Git-Url: http://www.git.stargrave.org/?p=public-inbox.git;a=blobdiff_plain;f=t%2Flei-q-save.t;h=3d09fe37cd108c666d94347fd2e10e241147d810;hp=eada2dd48a99c4cdbc23ef4aa414935db815c34b;hb=refs%2Fheads%2Fmaster;hpb=d15e80db44399867d8ca53528e38f70f54562b88 diff --git a/t/lei-q-save.t b/t/lei-q-save.t index eada2dd4..d09c8397 100644 --- a/t/lei-q-save.t +++ b/t/lei-q-save.t @@ -1,9 +1,10 @@ #!perl -w -# Copyright (C) 2021 all contributors +# Copyright (C) all contributors # License: AGPL-3.0+ 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: @@ -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,8 +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'); - like($lei_err, qr/--save was not used/, 'error noted --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); @@ -94,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'); @@ -107,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'); @@ -127,13 +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/--save was not used/, 'error noted --save'); + 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"; @@ -143,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); } @@ -153,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'); @@ -170,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'); @@ -200,16 +212,79 @@ test_lei(sub { ok($shared < $orig, 'fewer bytes stored with --shared') or diag "shared=$shared orig=$orig"; - lei_ok([qw(edit-search), $v2s], { VISUAL => 'cat', EDITOR => 'cat' }); + 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 --save -o ../s m:testmessage@example.com)); + 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;