Skip Menu |

This queue is for tickets about the Test-EOL CPAN distribution.

Report information
The Basics
Id: 77838
Status: resolved
Priority: 0/
Queue: Test-EOL

People
Owner: Nobody in particular
Requestors: paul [...] city-fan.org
Cc:
AdminCc:

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



Subject: Test-EOL 1.2 fails its own release tests (tabs in bundled Module::Install)
$ make test RELEASE_TESTING=1 PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/00-compile.t ............ ok t/11-all.t ................ ok t/12-fail.t ............... ok t/13-latin1.t ............. ok # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install.pm' on line 31' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/Base.pm' on line 7' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/Can.pm' on line 12' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/Fetch.pm' on line 9' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/Makefile.pm' on line 10' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/Metadata.pm' on line 9' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/Win32.pm' on line 9' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Failed test 'No tabs in '/builddir/build/BUILD/Test-EOL-1.2/t/../inc/Module/Install/WriteAll.pm' on line 9' # at /usr/share/perl5/vendor_perl/Test/NoTabs.pm line 89. # Looks like you failed 8 tests of 18. t/release-no-tabs.t ....... Dubious, test returned 8 (wstat 2048, 0x800) Failed 8/18 subtests t/release-pod-coverage.t .. ok t/release-pod-syntax.t .... ok Test Summary Report ------------------- t/release-no-tabs.t (Wstat: 2048 Tests: 18 Failed: 8) Failed tests: 3-10 Non-zero exit status: 8 Files=7, Tests=38, 0 wallclock secs ( 0.03 usr 0.01 sys + 0.47 cusr 0.07 csys = 0.58 CPU) Result: FAIL Failed 1/7 test programs. 8/38 subtests failed. Expanding the tabs in the bundled Module-Install distribution prior to running the test suite works around this problem for me.
You seem to have $ENV{RELEASE_TESTING} set - you shouldn't be setting this, as these tests shouldn't be run normally. You probably want to unset this! However, both Test::EOL and Test::NoTabs _should_ be ignoring the inc/ directory - and I've shipped new releases which do this - thanks for the heads up! :)
Subject: Re: [rt.cpan.org #77838] Test-EOL 1.2 fails its own release tests (tabs in bundled Module::Install)
Date: Sat, 16 Jun 2012 13:05:43 +0100
To: bug-Test-EOL [...] rt.cpan.org
From: Paul Howarth <paul [...] city-fan.org>
On Fri, 15 Jun 2012 18:24:32 -0400 "Tomas Doran via RT" <bug-Test-EOL@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=77838 > > > You seem to have $ENV{RELEASE_TESTING} set - you shouldn't be setting > this, as these tests shouldn't be run normally. You probably want to > unset this!
As a downstream maintainer of a large number of perl modules in Fedora, I find that running the release tests when we build the packages for those modules is a good way of ferreting out bugs in the release testing modules (as opposed to the actual module we're building), particular after new releases of important infrastructure like new versions of perl etc., which is why I generally run with RELEASE_TESTING and AUTHOR_TESTING set. Show quoted text
> However, both Test::EOL and Test::NoTabs _should_ be ignoring the inc/ > directory - and I've shipped new releases which do this - thanks for > the heads up! :)
My pleasure :-) Cheers, Paul.