]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/lei.t
www: drop --subject from "git send-email" instructions
[public-inbox.git] / t / lei.t
diff --git a/t/lei.t b/t/lei.t
index 8e771eb5e54490faad4b53c6fff2052de85f8b86..b10c9b59c72b8e29570b63dbf5853463a2c55cb7 100644 (file)
--- a/t/lei.t
+++ b/t/lei.t
@@ -3,76 +3,74 @@
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 use strict; use v5.10.1; use PublicInbox::TestCommon;
 use File::Path qw(rmtree);
-use PublicInbox::Spawn qw(which);
 
 # this only tests the basic help/config/init/completion bits of lei;
 # actual functionality is tested in other t/lei-*.t tests
-my $curl = which('curl');
 my $home;
 my $home_trash = [];
 my $cleanup = sub { rmtree([@$home_trash, @_]) };
 
 my $test_help = sub {
-       ok(!$lei->(), 'no args fails');
+       ok(!lei([]), 'no args fails');
        is($? >> 8, 1, '$? is 1');
        is($lei_out, '', 'nothing in stdout');
        like($lei_err, qr/^usage:/sm, 'usage in stderr');
 
        for my $arg (['-h'], ['--help'], ['help'], [qw(daemon-pid --help)]) {
-               ok($lei->($arg), "lei @$arg");
+               lei_ok($arg);
                like($lei_out, qr/^usage:/sm, "usage in stdout (@$arg)");
                is($lei_err, '', "nothing in stderr (@$arg)");
        }
 
        for my $arg ([''], ['--halp'], ['halp'], [qw(daemon-pid --halp)]) {
-               ok(!$lei->($arg), "lei @$arg");
+               ok(!lei($arg), "lei @$arg");
                is($? >> 8, 1, '$? set correctly');
                isnt($lei_err, '', 'something in stderr');
                is($lei_out, '', 'nothing in stdout');
        }
-       ok($lei->(qw(init -h)), 'init -h');
+       lei_ok(qw(init -h));
        like($lei_out, qr! \Q$home\E/\.local/share/lei/store\b!,
                'actual path shown in init -h');
-       ok($lei->(qw(init -h), { XDG_DATA_HOME => '/XDH' }),
-               'init with XDG_DATA_HOME');
+       lei_ok(qw(init -h), { XDG_DATA_HOME => '/XDH' },
+               \'init with XDG_DATA_HOME');
        like($lei_out, qr! /XDH/lei/store\b!, 'XDG_DATA_HOME in init -h');
        is($lei_err, '', 'no errors from init -h');
 
-       ok($lei->(qw(config -h)), 'config-h');
+       lei_ok(qw(config -h));
        like($lei_out, qr! \Q$home\E/\.config/lei/config\b!,
                'actual path shown in config -h');
-       ok($lei->(qw(config -h), { XDG_CONFIG_HOME => '/XDC' }),
-               'config with XDG_CONFIG_HOME');
+       lei_ok(qw(config -h), { XDG_CONFIG_HOME => '/XDC' },
+               \'config with XDG_CONFIG_HOME');
        like($lei_out, qr! /XDC/lei/config\b!, 'XDG_CONFIG_HOME in config -h');
        is($lei_err, '', 'no errors from config -h');
 };
 
 my $ok_err_info = sub {
        my ($msg) = @_;
-       is(grep(!/^I:/, split(/^/, $lei_err)), 0, $msg) or
+       is(grep(!/^#/, split(/^/, $lei_err)), 0, $msg) or
                diag "$msg: err=$lei_err";
 };
 
 my $test_init = sub {
        $cleanup->();
-       ok($lei->('init'), 'init w/o args');
+       lei_ok('init', \'init w/o args');
        $ok_err_info->('after init w/o args');
-       ok($lei->('init'), 'idempotent init w/o args');
+       lei_ok('init', \'idempotent init w/o args');
        $ok_err_info->('after idempotent init w/o args');
 
-       ok(!$lei->('init', "$home/x"), 'init conflict');
+       ok(!lei('init', "$home/x"), 'init conflict');
        is(grep(/^E:/, split(/^/, $lei_err)), 1, 'got error on conflict');
        ok(!-e "$home/x", 'nothing created on conflict');
        $cleanup->();
 
-       ok($lei->('init', "$home/x"), 'init conflict resolved');
+       lei_ok('init', "$home/x", \'init conflict resolved');
        $ok_err_info->('init w/ arg');
-       ok($lei->('init', "$home/x"), 'init idempotent w/ path');
+       lei_ok('init', "$home/x", \'init idempotent w/ path');
        $ok_err_info->('init idempotent w/ arg');
        ok(-d "$home/x", 'created dir');
        $cleanup->("$home/x");
 
-       ok(!$lei->('init', "$home/x", "$home/2"), 'too many args fails');
+       ok(!lei('init', "$home/x", "$home/2"), 'too many args fails');
        like($lei_err, qr/too many/, 'noted excessive');
        ok(!-e "$home/x", 'x not created on excessive');
        for my $d (@$home_trash) {
@@ -84,57 +82,90 @@ my $test_init = sub {
 
 my $test_config = sub {
        $cleanup->();
-       ok($lei->(qw(config a.b c)), 'config set var');
+       lei_ok(qw(config a.b c), \'config set var');
        is($lei_out.$lei_err, '', 'no output on var set');
-       ok($lei->(qw(config -l)), 'config -l');
+       lei_ok(qw(config -l), \'config -l');
        is($lei_err, '', 'no errors on listing');
        is($lei_out, "a.b=c\n", 'got expected output');
-       ok(!$lei->(qw(config -f), "$home/.config/f", qw(x.y z)),
+       ok(!lei(qw(config -f), "$home/.config/f", qw(x.y z)),
                        'config set var with -f fails');
        like($lei_err, qr/not supported/, 'not supported noted');
        ok(!-f "$home/config/f", 'no file created');
+
+       lei_ok(qw(-c imap.debug config --bool imap.debug));
+       is($lei_out, "true\n", "-c sets w/o value");
+       lei_ok(qw(-c imap.debug=1 config --bool imap.debug));
+       is($lei_out, "true\n", "-c coerces value");
+       lei_ok(qw(-c imap.debug=tr00 config imap.debug));
+       is($lei_out, "tr00\n", "-c string value passed as-is");
+       lei_ok(qw(-c imap.debug=a -c imap.debug=b config --get-all imap.debug));
+       is($lei_out, "a\nb\n", '-c and --get-all work together');
+
+       lei_ok([qw(config -e)], { VISUAL => 'cat', EDITOR => 'cat' });
+       is($lei_out, "[a]\n\tb = c\n", '--edit works');
 };
 
 my $test_completion = sub {
-       ok($lei->(qw(_complete lei)), 'no errors on complete');
+       lei_ok(qw(_complete lei), \'no errors on complete');
        my %out = map { $_ => 1 } split(/\s+/s, $lei_out);
        ok($out{'q'}, "`lei q' offered as completion");
        ok($out{'add-external'}, "`lei add-external' offered as completion");
 
-       ok($lei->(qw(_complete lei q)), 'complete q (no args)');
+       lei_ok(qw(_complete lei q), \'complete q (no args)');
        %out = map { $_ => 1 } split(/\s+/s, $lei_out);
        for my $sw (qw(-f --format -o --output --mfolder --augment -a
                        --mua --no-local --local --verbose -v
-                       --save-as --no-remote --remote --torsocks
+                       --save --no-save --no-remote --remote --torsocks
                        --reverse -r )) {
                ok($out{$sw}, "$sw offered as `lei q' completion");
        }
 
-       ok($lei->(qw(_complete lei q --form)), 'complete q --format');
+       lei_ok(qw(_complete lei q --form), \'complete q --format');
        is($lei_out, "--format\n", 'complete lei q --format');
        for my $sw (qw(-f --format)) {
-               ok($lei->(qw(_complete lei q), $sw), "complete q $sw ARG");
+               lei_ok(qw(_complete lei q), $sw);
                %out = map { $_ => 1 } split(/\s+/s, $lei_out);
                for my $f (qw(mboxrd mboxcl2 mboxcl mboxo json jsonl
                                concatjson maildir)) {
                        ok($out{$f}, "got $sw $f as output format");
                }
        }
-       ok($lei->(qw(_complete lei import)), 'complete import');
+       lei_ok(qw(_complete lei import));
        %out = map { $_ => 1 } split(/\s+/s, $lei_out);
-       for my $sw (qw(--flags --no-flags --no-kw --kw --no-keywords
-                       --keywords)) {
+       for my $sw (qw(--no-kw --kw)) {
                ok($out{$sw}, "$sw offered as `lei import' completion");
        }
 };
 
 my $test_fail = sub {
+       lei('q', 'whatever', '-C', '/dev/null');
+       is($? >> 8, 1, 'chdir at end fails to /dev/null');
+       lei('-C', '/dev/null', 'q', 'whatever');
+       is($? >> 8, 1, 'chdir at beginning fails to /dev/null');
+
+       lei_ok('q', "foo\n");
+       like($lei_err, qr/trailing `\\n' removed/s, "noted `\\n' removal");
+
+       for my $lk (qw(ei inbox)) {
+               my $d = "$home/newline\n$lk";
+               mkdir $d;
+               open my $fh, '>', "$d/$lk.lock" or BAIL_OUT "open $d/$lk.lock";
+               for my $fl (qw(-I --only)) {
+                       ok(!lei('q', $fl, $d, 'whatever'),
+                               "newline $lk.lock fails with q $fl");
+                       like($lei_err, qr/`\\n' not allowed/,
+                               "error noted with q $fl");
+               }
+       }
+       lei_ok('sucks', \'yes, but hopefully less every day');
+       like($lei_out, qr/loaded features/, 'loaded features shown');
 SKIP: {
-       skip 'no curl', 3 unless which('curl');
-       $lei->(qw(q --only http://127.0.0.1:99999/bogus/ t:m));
+       skip 'no curl', 3 unless require_cmd('curl', 1);
+       lei(qw(q --only http://127.0.0.1:99999/bogus/ t:m));
        is($? >> 8, 3, 'got curl exit for bogus URL');
-       $lei->(qw(q --only http://127.0.0.1:99999/bogus/ t:m -o), "$home/junk");
-       is($? >> 8, 3, 'got curl exit for bogus URL with Maildir');
+       lei(qw(q --only http://127.0.0.1:99999/bogus/ t:m -o), "$home/junk");
+       is($? >> 8, 3, 'got curl exit for bogus URL with Maildir') or
+               diag $lei_err;
        is($lei_out, '', 'no output');
 }; # /SKIP
 };
@@ -149,4 +180,8 @@ test_lei(sub {
        $test_fail->();
 });
 
+test_lei({ mods => [] }, sub {
+       lei_ok('sucks', \'no optional modules required');
+});
+
 done_testing;