Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

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

History Show all quoted text
  Wed Oct 15 08:24:53 2014 contyk [...] cpan.org - Ticket created
Subject: syntax_ok($module) only works if the module name contains ::
_module_to_path() currently assumes the argument is a filename if it doesn't contain '::'. This breaks syntax_ok() for many modules out there. Simply removing this condition (lib/Test/Strict.pm:503) should fix the issue.
  Wed Oct 15 10:30:21 2014 SZABGAB [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #99505] syntax_ok($module) only works if the module name contains ::
Date: Wed, 15 Oct 2014 16:30:10 +0200
To: bug-Test-Strict [...] rt.cpan.org
From: Gabor Szabo <SZABGAB [...] cpan.org>
Could you, please, send test cases that will show this change does not break other cases? You can send them as a pull request for https://github.com/szabgab/Test-Strict
  Wed Oct 15 10:30:22 2014 The RT System itself - Status changed from 'new' to 'open'
  Wed Oct 15 11:29:55 2014 contyk [...] cpan.org - Correspondence added
On Wed Oct 15 10:30:21 2014, SZABGAB wrote: Show quoted text
> Could you, please, send test cases that will show this change does not > break other cases? You can send them as a pull request for > https://github.com/szabgab/Test-Strict
I think the current test suite covers any possible breakage with make_extensionless_perl_file1() in 01all.t already. The tests pass with this change. Anyhow, I've sent you this one line change pull request :) Thanks, Petr