Skip Menu |

This queue is for tickets about the Dist-Zilla-Tester-DieHard CPAN distribution.

Report information
The Basics
Id: 117511
Status: resolved
Priority: 0/
Queue: Dist-Zilla-Tester-DieHard

People
Owner: VDB [...] cpan.org
Requestors: RIVY [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: v0.6.3
Fixed in: v0.6.4



Subject: installation test failure (missing updated error text from Dist::Zilla::Tester)
Installation is broken on MSWin32 (and most likely cygwin, as well) because the comparison text used in the test is out of date. It changed in the patch "https://github.com/rjbs/Dist-Zilla/commit/31826257eeb872d8e4e3e03c27a1479b970f8f71". I've attached a diff which fixes the test and allows the installation to complete.
Subject: diff-t-diehard.t.txt
diff -r 0f7b66d0d5ab t/die-hard.t --- a/t/die-hard.t Mon Nov 16 23:23:42 2015 +0000 +++ b/t/die-hard.t Sun Sep 04 21:06:01 2016 -0500 @@ -168,7 +168,7 @@ plan skip_all => 'Dist::Zilla too old for this test'; }; $bad_tester_args->{ add_files }->{ 'bad\file' } = ''; - $expected_error = qr{^File name 'bad\\file' does not seem to be legal on the current OS}; + $expected_error = qr{^Unix path 'bad\\file' does not seem to be portable to the current OS}; } else { # On other OSes let us make temp directory read-only. chmod( 0555, "$tmp_root" );
Hi, thanks for reporting. I updated the test to allow both variants. Will upload it soon (I still hope cpantesters.org will recover nearest days).
See the version v0.6.3_01.
v0.6.4 passed tests on 5 mswin32 machines, with both Dist::Zilla 5.x and 6.x. I am closing the bug. Feel free to reopen it or enter a new one if you note any test failure. Thanks.