Bug #88679 for Perl-Critic: Config::Tiny 2.17 and after break Perl Critic test suite.

Sat Sep 14 09:56:56 2013 ambs [...] cpan.org - Ticket created

Subject:

Config::Tiny 2.17 and after break Perl Critic test suite.

Hey With recent Config::Tiny, Perl::Critic fails with: t/09_theme.t .................................. ok # Failed test 'Invalid profile path' # at t/10_user_profile.t line 295. # 'Could not parse profile "bogus": Failed to open file 'bogus' for reading: No such file or directory' # doesn't match '(?^msx:File [ ] 'bogus' [ ] does [ ] not [ ] exist)' # Looks like you failed 1 test of 41. t/10_user_profile.t ........................... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/41 subtests So, probably the best way to fix this and make it robust for future changes is to match only the Perl::Critic part of the error message. Thanks

Mon Sep 16 09:07:06 2013 zefram [...] fysh.org - Correspondence added

Subject:	Re: [rt.cpan.org #88679] Config::Tiny 2.17 and after break Perl Critic test suite.
Date:	Mon, 16 Sep 2013 14:06:47 +0100
To:	bug-Perl-Critic [...] rt.cpan.org
From:	Zefram <zefram [...] fysh.org>

I'm using the attached patch. This leaves the test still sensitive to the format of the error message, but accepting the new format as well as the old one. -zefram

Message body is not shown because sender requested not to inline it.

Mon Sep 16 09:07:07 2013 The RT System itself - Status changed from 'new' to 'open'

Thu Sep 26 01:42:58 2013 thaljef [...] cpan.org - Correspondence added

RT-Send-CC:

zefram [...] fysh.org

This has been fixed and released to a CPAN mirror near you as Perl-Critic-1.119. Thanks for reporting the problem. -Jeff

Thu Sep 26 01:42:59 2013 thaljef [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #88679 for Perl-Critic: Config::Tiny 2.17 and after break Perl Critic test suite.

Preferred bug tracker