Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

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

History Show all quoted text
  Sat Aug 28 14:50:21 2010 ether [...] cpan.org - Ticket created
Subject: Suggestion for improvement: make failed requirements visible when tests are run with prove
Missing requirements are only visible to the runner of the tests when tests are run individually (that is, when the note() level of output is visible), and not when run in a group as via prove (when only diag() level output is visible). e.g. missing requirements should be visible via prove like so: t/100_io.t .............................. skipped: JSON::Any is required for this test t/101_io_atomic.t ....................... skipped: IO::AtomicFile is required for this test I'd be happy to submit a patch for this -- just point me to where you keep your repository! (I have a rw commit bit on http://git.shadowcat.co.uk/gitweb/gitweb.cgi.)
  Sun Aug 29 17:56:12 2010 tokuhirom [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #60842] Suggestion for improvement: make failed requirements visible when tests are run with prove
Date: Mon, 30 Aug 2010 06:56:02 +0900
To: bug-Test-Requires [...] rt.cpan.org
From: Tokuhiro Matsuno <tokuhirom [...] gmail.com>
Hi ether, Please send the patch with github pull request. The repository is here: http://github.com/tokuhirom/p5-test-requires On Sun, Aug 29, 2010 at 3:50 AM, Karen Etheridge via RT <bug-Test-Requires@rt.cpan.org> wrote: Show quoted text
> Sat Aug 28 14:50:21 2010: Request 60842 was acted upon. > Transaction: Ticket created by ETHER >       Queue: Test-Requires >     Subject: Suggestion for improvement: make failed requirements visible when >  tests are run with prove >   Broken in: 0.05 >    Severity: Normal >       Owner: Nobody >  Requestors: ether@cpan.org >      Status: new >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=60842 > > > > Missing requirements are only visible to the runner of the tests when > tests are run individually (that is, when the note() level of output is > visible), and not when run in a group as via prove (when only diag() > level output is visible). > > e.g. missing requirements should be visible via prove like so: > > t/100_io.t .............................. skipped: JSON::Any is required > for this test > t/101_io_atomic.t ....................... skipped: IO::AtomicFile is > required for this test > > I'd be happy to submit a patch for this -- just point me to where you > keep your repository!  (I have a rw commit bit on > http://git.shadowcat.co.uk/gitweb/gitweb.cgi.) > > > >
  Sun Aug 29 17:56:13 2010 The RT System itself - Status changed from 'new' to 'open'
  Fri Oct 15 19:35:11 2010 ether [...] cpan.org - Correspondence added
On Sun Aug 29 14:56:12 2010, tokuhirom@gmail.com wrote: Show quoted text
> Hi ether, > > Please send the patch with github pull request. > The repository is here: http://github.com/tokuhirom/p5-test-requires
The changes are ready - please pull from http://github.com/karenetheridge/p5-test-requires.
  Fri Oct 15 23:08:26 2010 ether [...] cpan.org - Correspondence added
I didn't add myself to the 'thanks' section; it seemed a bit narcissistic to do that myself. :)