Bug #76802 for Test-Simple: Test::More loads Exporter, conceals omission of "use Exporter"

Wed Apr 25 11:38:54 2012 mjp [...] cpan.org - Ticket created

Subject:

Test::More loads Exporter, conceals omission of "use Exporter"

Math::Counting sets up @EXPORT and %EXPORT_TAGS and @ISA, but forgets to actually require Exporter. As a consequence, simple use of the module fails, since no subs are exported. Unit tests pass, however, since Test::More pulls in Exporter, concealing the author's omission. Can Test::Simple and friends perhaps rely on their own bundled version of core libs like Exporter?

Wed Apr 25 16:13:00 2012 mschwern [...] cpan.org - Correspondence added

Thanks for the report. I appreciate the problem, and it is a genuine one. Unfortunately the proposed solution does not scale. In the past we've striven to load as few modules as possible, but as Test::Builder and Test::More evolve they have to be able to make use of other core modules, any of which might use Exporter. Or Carp. Or whatever. Right problem, wrong solution. Other solutions welcome to be considered.

Wed Apr 25 16:13:02 2012 The RT System itself - Status changed from 'new' to 'open'

Fri Oct 28 14:50:40 2016 exodist7 [...] gmail.com - Correspondence added

On Wed Apr 25 08:38:54 2012, MJP wrote: Show quoted text

> Math::Counting sets up @EXPORT and %EXPORT_TAGS and @ISA, but forgets > to actually require > Exporter. As a consequence, simple use of the module fails, since no > subs are exported. > > Unit tests pass, however, since Test::More pulls in Exporter, > concealing the author's omission. > > Can Test::Simple and friends perhaps rely on their own bundled version > of core libs like > Exporter?

This has been discussed a few times on #toolchain. Test::More can use core modules despite this problem, rejecting ticket.

Fri Oct 28 14:50:41 2016 exodist7 [...] gmail.com - Status changed from 'open' to 'rejected'

Bug #76802 for Test-Simple: Test::More loads Exporter, conceals omission of "use Exporter"

Preferred bug tracker