Skip Menu |

This queue is for tickets about the Image-ExifTool CPAN distribution.

Report information
The Basics
Id: 106809
Status: patched
Priority: 0/
Queue: Image-ExifTool
People
Owner: Nobody in particular
Requestors: ppisar [...] redhat.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 10.00
Fixed in: (no value)

History Show all quoted text
  Tue Sep 01 10:09:09 2015 ppisar [...] redhat.com - Ticket created
Subject: Tests fail without some optional modules
Many modules are used by "eval {require FOO}" idiom to make the dependency optional. I iterated over these modules and checked whether tests pass if the module cannot be loaded (by rewriting code to "eval {require XXX::FOO}"). If following modules are not available, tests will fail: Encode POSIX Time::Local Example of the failure: t/Canon.t .......... ok Test 4 differs beginning at line 26: Test gave: "[MakerNotes, CanonRaw, Time] 0 - Date/Time Original: 2003:11:10 17:39:26" Should be: "[MakerNotes, CanonRaw, Time] 0 - Date/Time Original: 2003:12:04 06:46:52" t/CanonRaw.t ....... Failed 1/7 subtests I recommend to adjust tests or make the modules a hard dependency.
  Tue Sep 01 11:11:36 2015 phil [...] owl.phy.queensu.ca - Correspondence added
Subject: Re: [rt.cpan.org #106809] Tests fail without some optional modules
Date: Tue, 1 Sep 2015 11:11:13 -0400
To: bug-Image-ExifTool [...] rt.cpan.org
From: Phil Harvey <phil [...] owl.phy.queensu.ca>
Good point. Thanks for the report. I'll look into this. - Phil
  Tue Sep 01 11:11:36 2015 The RT System itself - Status changed from 'new' to 'open'
  Tue Sep 01 11:58:03 2015 ether [...] cpan.org - Correspondence added
On 2015-09-01 08:11:36, phil@owl.phy.queensu.ca wrote: Show quoted text
> Good point. Thanks for the report. I'll look into this. > > - Phil >
Test::Require can allow you to skip an entire test file is an optional dependency is not present.
  Thu Sep 03 10:47:28 2015 EXIFTOOL [...] cpan.org - Correspondence added
On Tue Sep 01 11:58:03 2015, ETHER wrote: Show quoted text
> Test::Require can allow you to skip an entire test file is an optional > dependency is not present.
Do you mean Test::Requires? If so, then how do I deal with the dependency on Test::Requires itself? Seems like a catch 22. - Phil
  Mon Jan 16 14:11:11 2017 EXIFTOOL [...] cpan.org - Status changed from 'open' to 'patched'