Skip Menu |

This queue is for tickets about the IO-Iron CPAN distribution.

Report information
The Basics
Id: 98625
Status: resolved
Priority: 0/
Queue: IO-Iron

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

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



Subject: eval block use does not protect
http://search.cpan.org/grep?release=IO-Iron-0.10;string=Test::Spelling;n=1;C=0 There I find 11:eval { use Test::Spelling; }; This does not work as intended since use is an implicit BEGIN block and does its work before the eval is seen. This leads to fail reports that complain they did not find Test::Spellings like this one: http://www.cpantesters.org/cpan/report/e96c1f74-2dee-11e4-94fc-96b94915a708 HTH&&Thanks, (Disclaimer: this was discovered by statistical analysis, I'm not speaking as a user)
Subject: Re: [rt.cpan.org #98625] eval block use does not protect
Date: Fri, 05 Sep 2014 09:50:14 +0200
To: bug-IO-Iron [...] rt.cpan.org
From: mikkoi <mikkoi [...] kapsi.fi>
Hello, Andreas, Thank you for reporting. I will fix this by moving the test from subdir t/ to subdir xt/release or xt/author since this is a release or author test. -- Mikko Koivunalho On 2014-09-05 04:27, Andreas Koenig via RT wrote: Show quoted text
> Thu Sep 04 22:27:37 2014: Request 98625 was acted upon. > Transaction: Ticket created by ANDK > Queue: IO-Iron > Subject: eval block use does not protect > Broken in: 0.10 > Severity: (no value) > Owner: Nobody > Requestors: ANDK@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=98625 > > > > http://search.cpan.org/grep?release=IO-Iron-0.10;string=Test::Spelling;n=1;C=0 > > There I find > > 11:eval { use Test::Spelling; }; > > This does not work as intended since use is an implicit BEGIN block > and does its work before the eval is seen. This leads to fail reports > that complain they did not find Test::Spellings like this one: > > http://www.cpantesters.org/cpan/report/e96c1f74-2dee-11e4-94fc-96b94915a708 > > HTH&&Thanks, > (Disclaimer: this was discovered by statistical analysis, I'm not > speaking as a user)
Fixed by moving test to xt/author. Tests are created by Dist::Zilla plugin now.