Skip Menu |

This queue is for tickets about the Expect CPAN distribution.

Report information
The Basics
Id: 99871
Status: open
Priority: 0/
Queue: Expect
People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
Cc: ether [...] cpan.org
AdminCc:
Bug Information
Severity: (no value)
Broken in:
  • 1.29
  • 1.30
  • 1.31
  • 1.32
Fixed in: (no value)

History Show all quoted text
  Wed Oct 29 01:39:48 2014 ANDK [...] cpan.org - Ticket created
Subject: Could you make dependency on Test::Exception optional?
1.29 introduced a dependency on Test::Exception. It is unfortunate for such a basic building block of CPAN like Expect.pm to depend on something that comes much later in the bootstrapping dependency chains. Could you make the two tests that depend on Test::Exception skip_all if Test::Exception is not available? Or maybe this: throws_ok is is use three times now. Alternatively a two-liner would probably not be too hard to write? Something like eval { ... }; like $@, qr/.../; Bear with me, I make this suggestion on the background of recent fragility of Test::More and Test::Exception. Otherwise I would not look into such dependency details. But such issues creep in silently and hit us when we least need them. Thanks,
  Wed Oct 29 15:59:53 2014 ether [...] cpan.org - Cc ETHER added
  Fri Jan 15 09:24:24 2016 SYSPETE [...] cpan.org - Correspondence added
On Wed Oct 29 01:39:48 2014, ANDK wrote: Show quoted text
> 1.29 introduced a dependency on Test::Exception. It is unfortunate for > such a basic building block of CPAN like Expect.pm to depend on > something that comes much later in the bootstrapping dependency > chains. > > Could you make the two tests that depend on Test::Exception skip_all > if Test::Exception is not available? > > Or maybe this: throws_ok is is use three times now. Alternatively a > two-liner would probably not be too hard to write? Something like > > eval { ... }; > like $@, qr/.../; > > Bear with me, I make this suggestion on the background of recent > fragility of Test::More and Test::Exception. Otherwise I would not > look into such dependency details. But such issues creep in silently > and hit us when we least need them. > > Thanks,
PR submitted switching to eval+like: https://github.com/jacoby/expect.pm/pull/1
  Fri Jan 15 09:24:25 2016 The RT System itself - Status changed from 'new' to 'open'