Skip Menu |

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

Report information
The Basics
Id: 66399
Status: open
Priority: 0/
Queue: Test-AutoLoader
People
Owner: Nobody in particular
Requestors: SILASMONK [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.03
Fixed in: (no value)

History Show all quoted text
  Sat Mar 05 05:23:44 2011 SILASMONK [...] cpan.org - Ticket created
Subject: test script does not handle dependencies correctly
Many test runs of this script error with :"Parse errors: Bad plan. You planned 54 tests but ran 64." You should either: 1.) Change the number of tests to 64 and skip according to the modules available. (Strategy half-heartedly followed at the moment). or 2.) Don't declare the number of tests but only when you know what modules are available. use Test::More; .... plan tests => $number_of_expected_tests;
  Thu Apr 26 05:36:58 2012 ppisar [...] redhat.com - Correspondence added
From: ppisar [...] redhat.com
Dne So 05.bře.2011 05:23:44, SILASMONK napsal(a): Show quoted text
> Many test runs of this script error with :"Parse errors: Bad plan. You > planned 54 tests but ran 64." You should either: > > 1.) Change the number of tests to 64 and skip according to the modules > available. (Strategy half-heartedly followed at the moment). >
Attached patch implements it.
Subject: Test-AutoLoader-0.03-Fix-test-plan-number.patch
From f58959906ee7e8f704992c632d7af11b61c68668 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <ppisar@redhat.com> Date: Thu, 26 Apr 2012 11:33:21 +0200 Subject: [PATCH] Fix test plan number <https://rt.cpan.org/Public/Bug/Display.html?id=66399> --- t/00basic.t | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/t/00basic.t b/t/00basic.t index a6b2345..94f1a4f 100644 --- a/t/00basic.t +++ b/t/00basic.t @@ -7,7 +7,7 @@ BEGIN { eval "use Test::Pod 0.95"; $NO_TEST_POD = $@; } -use Test::More tests=>54; +use Test::More tests=>64; use POSIX; use strict; @@ -71,5 +71,5 @@ foreach my $test (@tests) { if ($CAN_CHMOD) { chmod 0755, @unreadable or warn "Couldn't chmod @unreadable back: $!\n"; } else { - SKIP:{skip "Couldn't set up unreadable directory for test",5} + SKIP:{skip "Couldn't set up unreadable directory for test",6} } -- 1.7.7.6
  Thu Apr 26 05:36:59 2012 The RT System itself - Status changed from 'new' to 'open'