Skip Menu |

This queue is for tickets about the Text-Trac CPAN distribution.

Report information
The Basics
Id: 127719
Status: open
Priority: 0/
Queue: Text-Trac

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

Bug Information
Severity: (no value)
Broken in:
  • 0.19
  • 0.20
Fixed in: (no value)



Subject: t/95-tidyall.t (with newer Perl::Tidy?)
t/95-tidyall.t started to fail: ... # [checked] lib/Text/Trac/LinkResolver.pm # *** needs tidying # Failed test 'lib/Text/Trac/LinkResolver.pm is tidy' # at t/95-tidyall.t line 10. # *** needs tidying # [checked] lib/Text/Trac/Heading.pm # *** needs tidying # Failed test 'lib/Text/Trac/Heading.pm is tidy' # at t/95-tidyall.t line 10. # *** needs tidying # Looks like you failed 2 tests of 49. t/95-tidyall.t ..................... Dubious, test returned 2 (wstat 512, 0x200) Failed 2/49 subtests ... There was a Perl::Tidy release recently, and probably something changed in the expected tidy output.
On 2018-11-18 03:07:51, SREZIC wrote: Show quoted text
> t/95-tidyall.t started to fail: > > There was a Perl::Tidy release recently, and probably something > changed in the expected tidy output.
While this test failure is indeed an issue that should be corrected, please don't allow pod, critic, coverage or kwalitee tests to run for normal user installs, as they make the installation fail unnecessarily when newer versions of the tester module introduces errors that you cannot anticipate. Also spelling tests are dependent on what dictionaries are installed locally, and critic tests use all plugins that are installed, neither of which can be predicted in advance. Either move these tests to xt/, or guard them with: plan skip_all => "These tests are for authors only!" unless $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};