Bug #74430 for PlRPC: Tests require running syslog

Thu Jan 26 12:13:37 2012 ppisar [...] redhat.com - Ticket created

Subject:

Tests require running syslog

Running tests on Linux require access to /dev/log (running syslog), otherwise tests fail: no connection to syslog available - /dev/log is not a socket at /usr/share/perl5/vendor_perl/Net/Daemon/Log.pm line 114 Server process didn't create a file 'ndtest.prt'. at /usr/share/perl5/vendor_perl/Net/Daemon/Test.pm line 317. t/client.t .... Dubious, test returned 2 (wstat 512, 0x200) Failed 10/10 subtests [...] I think it would be better to redefine logfile RPC::PlClient and RPC::PlServer member in tests to allow testing even without syslog.

Fri Jan 27 05:34:18 2012 ppisar [...] redhat.com - Correspondence added

Subject:	Re: [rt.cpan.org #74430] AutoReply: Tests require running syslog
Date:	Fri, 27 Jan 2012 11:34:07 +0100
To:	Bugs in PlRPC via RT <bug-PlRPC [...] rt.cpan.org>
From:	Petr Pisar <ppisar [...] redhat.com>

On Thu, Jan 26, 2012 at 12:13:37PM -0500, Bugs in PlRPC via RT wrote: Show quoted text

> > I think it would be better to redefine logfile RPC::PlClient and > RPC::PlServer member in tests to allow testing even without syslog. >

Attached patch modifes tests in this way. -- Petr

Message body is not shown because sender requested not to inline it.

Download (untitled)
application/pgp-signature 230b

Message body not shown because it is not plain text.

Fri Jan 27 10:39:23 2012 TODDR [...] cpan.org - Taken

Fri Jan 27 11:58:26 2012 TODDR [...] cpan.org - Correspondence added

sending to file 1 causes output to go to STDOUT. I'm using File::Spec->devnull and sending it to a file handle proper. dev release sent to CPAN. I'll put out a prod release next week if no issues are found on CPAN testers.

Fri Jan 27 11:58:27 2012 The RT System itself - Status changed from 'new' to 'open'