Skip Menu |

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

Report information
The Basics
Id: 123288
Status: patched
Worked: 20 min
Priority: 0/
Queue: Test-Module-Runnable
People
Owner: DDRP [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sun Oct 15 11:34:52 2017 SREZIC [...] cpan.org - Ticket created
Subject: Random t/unique.t failures (0.2.2)
On some of my smoker systems t/unique.t fails. The test log starts like this: ... # Failed test 'result 926913649 not seen previously' # at t/lib/Private/Test/Module/Runnable/unique.pm line 104. # got: '0' # expected: '' # Looks like you failed 1 test of 5. # Failed test 'trial iteration 1/500' # at t/lib/Private/Test/Module/Runnable/unique.pm line 126. # Failed test 'result 163078812 not seen previously' # at t/lib/Private/Test/Module/Runnable/unique.pm line 104. # got: '0' # expected: '' # Looks like you failed 1 test of 5. ... Statistical analysis suggests that this happens for some List::MoreUtils versions. I don't see failures if 0.423 or newer is installed, so maybe this should be used as a minimum prereq version.
  Sun Oct 15 11:43:25 2017 DDRP [...] cpan.org - Correspondence added 20 min
On Sun Oct 15 11:34:52 2017, SREZIC wrote: Show quoted text
> On some of my smoker systems t/unique.t fails. The test log starts > like this: > > ... > # Failed test 'result 926913649 not seen previously' > # at t/lib/Private/Test/Module/Runnable/unique.pm line 104. > # got: '0' > # expected: '' > # Looks like you failed 1 test of 5. > > # Failed test 'trial iteration 1/500' > # at t/lib/Private/Test/Module/Runnable/unique.pm line 126. > > # Failed test 'result 163078812 not seen previously' > # at t/lib/Private/Test/Module/Runnable/unique.pm line 104. > # got: '0' > # expected: '' > # Looks like you failed 1 test of 5. > ... > > Statistical analysis suggests that this happens for some > List::MoreUtils versions. I don't see failures if 0.423 or newer is > installed, so maybe this should be used as a minimum prereq version.
Hi SREZIC, I've taken a slightly different approach in that I should have used ok(! rather than comparing against ''. Clearly there has been a behavior change at the version you mention, so my fix should mean both 0 and '' will pass now, rather than changing the minimum version of List::MoreUtils. Your reports have been very helpful, thankyou. Fix is in 0.2.3
  Sun Oct 15 11:43:26 2017 The RT System itself - Status changed from 'new' to 'open'
  Sun Oct 15 11:43:26 2017 DDRP [...] cpan.org - Status changed from 'open' to 'patched'
  Sun Oct 15 11:43:26 2017 DDRP [...] cpan.org - Taken