Skip Menu |

This queue is for tickets about the GPS-Point CPAN distribution.

Report information
The Basics
Id: 52364
Status: resolved
Priority: 0/
Queue: GPS-Point

People
Owner: Nobody in particular
Requestors: wyant [...] cpan.org
Cc:
AdminCc:

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



Subject: t/005_value.t fails if Geo::ECEF is not installed.
The issue appears to be a miscount of the number of tests to skip if Geo::ECEF is not installed. It looks like one "ok" plus three "delta_ok" makes 4 tests total. The attached patch against the 0.15 release attempts to address the problem. Devel::Hide is useful when dealing with optional modules. perl -Mblib -MDevel::Hide=Geo::ECEF t/005_value.t on a system with all optional modules installed should confirm (or not!) my diagnosis.
Subject: 005_value.patch
--- t/005_value.old 2009-11-26 17:32:57.000000000 -0500 +++ t/005_value.t 2009-12-02 11:11:39.000000000 -0500 @@ -28,7 +28,7 @@ SKIP: { eval { require Geo::ECEF }; - skip "Geo::ECEF not installed", 3 if $@; + skip "Geo::ECEF not installed", 4 if $@; ok(1, "Running tests that require Geo::ECEF");
Fixed test in version 0.16. Thanks for the pointer on Devel::Hide.