config: assume lists have multiple addresses

[public-inbox.git] / t / config.t

diff --git a/t/config.t b/t/config.t
index 48df08716c654fac0f3ca5852e739a85fd41b683..437f1d145b868d35854ae54e2e3ef021590d2e79 100644 (file)
--- a/t/config.t
+++ b/t/config.t
@@ -9,10 +9,8 @@ my $tmpdir = tempdir('pi-config-XXXXXX', TMPDIR => 1, CLEANUP => 1);
 
 {
        is(system(qw(git init -q --bare), $tmpdir), 0, "git init successful");
-       {
-               local $ENV{GIT_DIR} = $tmpdir;
-               is(system(qw(git config foo.bar hihi)), 0, "set config");
-       }
+       my @cmd = ('git', "--git-dir=$tmpdir", qw(config foo.bar hihi));
+       is(system(@cmd), 0, "set config");
 
        my $tmp = PublicInbox::Config->new("$tmpdir/config");
 
@@ -27,9 +25,14 @@ my $tmpdir = tempdir('pi-config-XXXXXX', TMPDIR => 1, CLEANUP => 1);
        my $cfg = PublicInbox::Config->new($f);
        is_deeply($cfg->lookup('meta@public-inbox.org'), {
                'mainrepo' => '/home/pi/meta-main.git',
-               'address' => 'meta@public-inbox.org',
+               'address' => [ 'meta@public-inbox.org' ],
+               'domain' => 'public-inbox.org',
+               'url' => 'http://example.com/meta',
                -primary_address => 'meta@public-inbox.org',
-               'listname' => 'meta',
+               'name' => 'meta',
+               feedmax => 25,
+               -httpbackend_limiter => undef,
+               nntpserver => undef,
        }, "lookup matches expected output");
 
        is($cfg->lookup('blah@example.com'), undef,
@@ -42,8 +45,45 @@ my $tmpdir = tempdir('pi-config-XXXXXX', TMPDIR => 1, CLEANUP => 1);
                              'test@public-inbox.org'],
                -primary_address => 'try@public-inbox.org',
                'mainrepo' => '/home/pi/test-main.git',
-               'listname' => 'test',
+               'domain' => 'public-inbox.org',
+               'name' => 'test',
+               feedmax => 25,
+               'url' => 'http://example.com/test',
+               -httpbackend_limiter => undef,
+               nntpserver => undef,
        }, "lookup matches expected output for test");
 }
 
+
+{
+       my $cfgpfx = "publicinbox.test";
+       my @altid = qw(serial:gmane:file=a serial:enamg:file=b);
+       my $config = PublicInbox::Config->new({
+               "$cfgpfx.address" => 'test@example.com',
+               "$cfgpfx.mainrepo" => '/path/to/non/existent',
+               "$cfgpfx.altid" => [ @altid ],
+       });
+       my $ibx = $config->lookup_name('test');
+       is_deeply($ibx->{altid}, [ @altid ]);
+}
+
+{
+       my $pfx = "publicinbox.test";
+       my %h = (
+               "$pfx.address" => 'test@example.com',
+               "$pfx.mainrepo" => '/path/to/non/existent',
+               "publicinbox.nntpserver" => 'news.example.com',
+       );
+       my %tmp = %h;
+       my $cfg = PublicInbox::Config->new(\%tmp);
+       my $ibx = $cfg->lookup_name('test');
+       is($ibx->{nntpserver}, 'news.example.com', 'global NNTP server');
+
+       delete $h{'publicinbox.nntpserver'};
+       $h{"$pfx.nntpserver"} = 'news.alt.example.com';
+       $cfg = PublicInbox::Config->new(\%h);
+       $ibx = $cfg->lookup_name('test');
+       is($ibx->{nntpserver}, 'news.alt.example.com','per-inbox NNTP server');
+}
+
 done_testing();