]> Sergey Matveev's repositories - public-inbox.git/commitdiff
t/sigfd: use PublicInbox::DS::block_signals
authorEric Wong <e@80x24.org>
Thu, 6 May 2021 17:09:23 +0000 (17:09 +0000)
committerEric Wong <e@80x24.org>
Thu, 6 May 2021 17:13:00 +0000 (17:13 +0000)
We already use PublicInbox::DS in this test and I've always
found the terminology of sig* APIs confusing :x

t/sigfd.t

index 3a383d798bd120019fbb0823e187c8eee6095547..bb854147023d8eb522c2a0ddbe5d88ab6032d7aa 100644 (file)
--- a/t/sigfd.t
+++ b/t/sigfd.t
@@ -6,15 +6,14 @@ use POSIX qw(:signal_h);
 use Errno qw(ENOSYS);
 use PublicInbox::Syscall qw(SFD_NONBLOCK);
 require_ok 'PublicInbox::Sigfd';
+use PublicInbox::DS;
 
 SKIP: {
        if ($^O ne 'linux' && !eval { require IO::KQueue }) {
                skip 'signalfd requires Linux or IO::KQueue to emulate', 10;
        }
-       my $new = POSIX::SigSet->new;
-       $new->fillset or die "sigfillset: $!";
-       my $old = POSIX::SigSet->new;
-       sigprocmask(SIG_SETMASK, $new, $old) or die "sigprocmask $!";
+
+       my $old = PublicInbox::DS::block_signals();
        my $hit = {};
        my $sig = {};
        local $SIG{HUP} = sub { $hit->{HUP}->{normal}++ };
@@ -25,7 +24,6 @@ SKIP: {
        }
        my $sigfd = PublicInbox::Sigfd->new($sig, 0);
        if ($sigfd) {
-               require PublicInbox::DS;
                ok($sigfd, 'Sigfd->new works');
                kill('HUP', $$) or die "kill $!";
                kill('INT', $$) or die "kill $!";