Bug #109980 for Text-ECSV: t/pod-spell.t test fails

Mon Nov 30 15:08:30 2015 SREZIC [...] cpan.org - Ticket created

Subject:

t/pod-spell.t test fails

# Failed test 'POD spelling for blib/lib/Text/ECSV.pm' # at t/pod-spell.t line 11. # Errors: # AnnoCPAN # CSV # ECSV # Giersig # Kutej # aditional # dupplicate # ing # md # occure # # All incorrect words, by number of occurrences: # 1: AnnoCPAN, ECSV, aditional, dupplicate, ing, md, occure # 2: Giersig, Kutej # 5: CSV # Looks like you failed 1 test of 1. t/pod-spell.t ..... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/1 subtests

Mon Nov 30 18:22:35 2015 ether [...] cpan.org - Correspondence added

On 2015-11-30 12:08:30, SREZIC wrote: Show quoted text

> # Failed test 'POD spelling for blib/lib/Text/ECSV.pm' > # at t/pod-spell.t line 11. > # Errors: > # AnnoCPAN > # CSV > # ECSV > # Giersig > # Kutej > # aditional > # dupplicate > # ing > # md > # occure > # > # All incorrect words, by number of occurrences: > # 1: AnnoCPAN, ECSV, aditional, dupplicate, ing, md, occure > # 2: Giersig, Kutej > # 5: CSV > # Looks like you failed 1 test of 1. > t/pod-spell.t ..... > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/1 subtests

Please don't allow pod, critic or kwalitee tests to run for normal user installs, as they make the installation fail unnecessarily when newer versions of the tester introduce errors that you cannot anticipate. Also spelling tests are dependent on what dictionaries are installed locally, which cannot be predicted in advance. Either move them to xt/, or guard them with: plan skip_all => "These tests are for authors only!" unless $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};

Mon Nov 30 18:22:36 2015 The RT System itself - Status changed from 'new' to 'open'

Wed Dec 02 05:46:14 2015 jozef [...] kutej.net - Correspondence added

Subject:	Re: [rt.cpan.org #109980] t/pod-spell.t test fails
Date:	Wed, 02 Dec 2015 11:46:01 +0100
To:	bug-Text-ECSV [...] rt.cpan.org
From:	Jozef Kutej <jozef [...] kutej.net>

On 12/01/2015 12:22 AM, Karen Etheridge via RT wrote: Show quoted text

> Please don't allow pod, critic or kwalitee tests to run for normal user installs, as they make the installation fail unnecessarily when newer versions of the tester introduce errors that you cannot anticipate. Also spelling tests are dependent on what dictionaries are installed locally, which cannot be predicted in advance. > > Either move them to xt/, or guard them with: > > plan skip_all => "These tests are for authors only!" unless $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING}; >

Thank you for the introduction into author testing, patches are welcome: https://github.com/jozef/text-ecsv Best Regards Jozef

Sun Dec 11 08:12:01 2016 jozef [...] kutej.net - Correspondence added

fixed in just uploaded 0.02

Sun Dec 11 08:12:02 2016 jozef [...] kutej.net - Status changed from 'open' to 'resolved'

Sun Dec 11 08:12:02 2016 jozef [...] kutej.net - Taken