t/*.t: localize $SIG{__WARN__} changes

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

diff --git a/t/watch_filter_rubylang.t b/t/watch_filter_rubylang.t
index da383c15ef4b294cbf5c047395474a8f88e66354..09217d94fcfa368e1215ec0a53ad252d5f0cc440 100644 (file)
--- a/t/watch_filter_rubylang.t
+++ b/t/watch_filter_rubylang.t
@@ -1,21 +1,15 @@
-# Copyright (C) 2019 all contributors <meta@public-inbox.org>
+# Copyright (C) 2019-2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 use strict;
 use warnings;
-require './t/common.perl';
+use PublicInbox::TestCommon;
 use Test::More;
-use File::Temp qw/tempdir/;
 use PublicInbox::MIME;
 use PublicInbox::Config;
-my @mods = qw(Filesys::Notify::Simple DBD::SQLite Search::Xapian);
-foreach my $mod (@mods) {
-       eval "require $mod";
-       plan skip_all => "$mod missing for watch_filter_rubylang_v2.t" if $@;
-}
-
+require_mods(qw(Filesys::Notify::Simple DBD::SQLite Search::Xapian));
 use_ok 'PublicInbox::WatchMaildir';
 use_ok 'PublicInbox::Emergency';
-my $tmpdir = tempdir('watch-XXXXXX', TMPDIR => 1, CLEANUP => 1);
+my ($tmpdir, $for_destroy) = tmpdir();
 local $ENV{PI_CONFIG} = "$tmpdir/pi_config";
 
 my @v = qw(V1);
@@ -30,17 +24,17 @@ SKIP: {
 
 for my $v (@v) {
        my @warn;
-       $SIG{__WARN__} = sub { push @warn, @_ };
+       local $SIG{__WARN__} = sub { push @warn, @_ };
        my $cfgpfx = "publicinbox.$v";
-       my $mainrepo = "$tmpdir/$v";
+       my $inboxdir = "$tmpdir/$v";
        my $maildir = "$tmpdir/md-$v";
        my $spamdir = "$tmpdir/spam-$v";
        my $addr = "test-$v\@example.com";
-       my @cmd = ('blib/script/public-inbox-init', "-$v", $v, $mainrepo,
+       my @cmd = ('-init', "-$v", $v, $inboxdir,
                "http://example.com/$v", $addr);
-       is(system(@cmd), 0, 'public-inbox init OK');
+       ok(run_script(\@cmd), 'public-inbox init OK');
        if ($v eq 'V1') {
-               is(system('blib/script/public-inbox-index', $mainrepo), 0);
+               ok(run_script(['-index', $inboxdir]), 'v1 indexed');
        }
        PublicInbox::Emergency->new($spamdir);
 
@@ -70,15 +64,15 @@ spam
 EOF
        PublicInbox::Emergency->new($maildir)->prepare(\"$spam");
 
-       my %orig = (
-               "$cfgpfx.address" => $addr,
-               "$cfgpfx.mainrepo" => $mainrepo,
-               "$cfgpfx.watch" => "maildir:$maildir",
-               "$cfgpfx.filter" => 'PublicInbox::Filter::RubyLang',
-               "$cfgpfx.altid" => 'serial:alerts:file=msgmap.sqlite3',
-               "publicinboxwatch.watchspam" => "maildir:$spamdir",
-       );
-       my $config = PublicInbox::Config->new({%orig});
+       my $orig = <<EOF;
+$cfgpfx.address=$addr
+$cfgpfx.inboxdir=$inboxdir
+$cfgpfx.watch=maildir:$maildir
+$cfgpfx.filter=PublicInbox::Filter::RubyLang
+$cfgpfx.altid=serial:alerts:file=msgmap.sqlite3
+publicinboxwatch.watchspam=maildir:$spamdir
+EOF
+       my $config = PublicInbox::Config->new(\$orig);
        my $ibx = $config->lookup_name($v);
        ok($ibx, 'found inbox by name');
 
@@ -108,7 +102,7 @@ EOF
        }
        $w->scan('full');
 
-       $config = PublicInbox::Config->new({%orig});
+       $config = PublicInbox::Config->new(\$orig);
        $ibx = $config->lookup_name($v);
        ($tot, undef) = $ibx->search->reopen->query('b:spam');
        is($tot, 0, 'spam removed');