Skip Menu |

This queue is for tickets about the libnet CPAN distribution.

Report information
The Basics
Id: 132333
Status: resolved
Priority: 0/
Queue: libnet

People
Owner: Nobody in particular
Requestors: jkeenan [...] pobox.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



CC: shay [...] cpan.org
Subject: Replace hand-crafted test functions with Test::More equivalents
Date: Sun, 12 Apr 2020 18:29:32 -0400
To: bug-libnet [...] rt.cpan.org
From: James E Keenan <jkeenan [...] pobox.com>
Four test files in this distribution use hand-crafted testing subroutines found in t/libnet_t.pl. The four test files import these functions via "require t/libnet_t.pl". Note the absence of "./" at the start of the argument to require. That runs afoul of the no-dot-in-@INC rule instituted in perl-5.28. This violation requires a special work-around in the core distribution's t/TEST. However, the testing subroutines in t/libnet_t.pl appear to date from the very early days of Perl 5 when Test::More had not yet been written. The Test::More functions are drop-in replacements for the t/libnet_t.pl ones. Let's use them. This enables us to eliminate t/libnet_t.pl from this distribution, so we update the MANIFEST as well. See first patch attached. Application of this patch will enable us to remove 'cpan/libnet' from the list of distributions needing a workaround in the core's t/TEST. Some small meta touchups in second patch attached. Thank you very much. Jim Keenan

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

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

On Sun Apr 12 18:29:40 2020, jkeenan@pobox.com wrote: Show quoted text
> Four test files in this distribution use hand-crafted testing > subroutines found in t/libnet_t.pl. The four test files import these > functions via "require t/libnet_t.pl". Note the absence of "./" at the > start of the argument to require. That runs afoul of the no-dot-in-@INC > rule instituted in perl-5.28. This violation requires a special > work-around in the core distribution's t/TEST. > > However, the testing subroutines in t/libnet_t.pl appear to date from > the very early days of Perl 5 when Test::More had not yet been written. > The Test::More functions are drop-in replacements for the t/libnet_t.pl > ones. Let's use them. > > This enables us to eliminate t/libnet_t.pl from this distribution, so we > update the MANIFEST as well. See first patch attached. Application of > this patch will enable us to remove 'cpan/libnet' from the list of > distributions needing a workaround in the core's t/TEST. > > Some small meta touchups in second patch attached. > > Thank you very much. > Jim Keenan
In pull request form: https://github.com/steve-m-hay/perl-libnet/pull/39
Thanks for the ticket. I've responded to this on the PR in Github, so I'll close this ticket now to avoid any further duplication.