This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
File::Path: synch with CPAN version 2.18
[perl5.git] / cpan / File-Path / t / Path.t
index a33c15a..b265aee 100644 (file)
@@ -1,9 +1,22 @@
+#! /usr/bin/env perl
 # Path.t -- tests for module File::Path
 
 use strict;
 
-use Test::More tests => 129;
+use Test::More tests => 167;
 use Config;
+use Fcntl ':mode';
+use lib './t';
+use FilePathTest qw(
+    _run_for_warning
+    _run_for_verbose
+    _cannot_delete_safe_mode
+    _verbose_expected
+    create_3_level_subdirs
+    cleanup_3_level_subdirs
+);
+use Errno qw(:POSIX);
+use Carp;
 
 BEGIN {
     use_ok('Cwd');
@@ -11,11 +24,15 @@ BEGIN {
     use_ok('File::Spec::Functions');
 }
 
-eval "use Test::Output";
-my $has_Test_Output = $@ ? 0 : 1;
-
 my $Is_VMS = $^O eq 'VMS';
 
+my $fchmod_supported = 0;
+if (open my $fh, curdir()) {
+    my ($perm) = (stat($fh))[2];
+    $perm &= 07777;
+    eval { $fchmod_supported = chmod( $perm, $fh); };
+}
+
 # first check for stupid permissions second for full, so we clean up
 # behind ourselves
 for my $perm (0111,0777) {
@@ -24,10 +41,12 @@ for my $perm (0111,0777) {
     chmod $perm, "mhx", $path;
 
     my $oct = sprintf('0%o', $perm);
+
     ok(-d "mhx", "mkdir parent dir $oct");
     ok(-d $path, "mkdir child dir $oct");
 
     rmtree("mhx");
+
     ok(! -e "mhx", "mhx does not exist $oct");
 }
 
@@ -108,6 +127,7 @@ sub count {
 
     $before = count(curdir());
     gisle('2nd', 1);
+
     is(count(curdir()), $before + 1, "second after $before");
 
     chdir updir();
@@ -120,11 +140,13 @@ sub count {
     open my $f, '>', 'foo.dat';
     close $f;
     my $before = count(curdir());
+
     cmp_ok($before, '>', 0, "ARGV $before");
     {
         local @ARGV = (1);
         mkpath('3rd', !shift, 0755);
     }
+
     is(count(curdir()), $before + 1, "third after $before");
 
     $before = count(curdir());
@@ -132,6 +154,7 @@ sub count {
         local @ARGV = (1);
         mkpath('4th', !shift, 0755);
     }
+
     is(count(curdir()), $before + 1, "fourth after $before");
 
     chdir updir();
@@ -152,16 +175,21 @@ SKIP: {
 
     rmtree($dir, {error => \$error});
     my $nr_err = @$error;
+
     is($nr_err, 1, "ancestor error");
 
     if ($nr_err) {
         my ($file, $message) = each %{$error->[0]};
+
         is($file, $dir, "ancestor named");
         my $ortho_dir = $^O eq 'MSWin32' ? File::Path::_slash_lc($dir2) : $dir2;
         $^O eq 'MSWin32' and $message
             =~ s/\A(cannot remove path when cwd is )(.*)\Z/$1 . File::Path::_slash_lc($2)/e;
+
         is($message, "cannot remove path when cwd is $ortho_dir", "ancestor reason");
+
         ok(-d $dir2, "child not removed");
+
         ok(-d $dir, "ancestor not removed");
     }
     else {
@@ -172,14 +200,18 @@ SKIP: {
     }
     chdir $cwd;
     rmtree($dir);
+
     ok(!(-d $dir), "ancestor now removed");
 };
 
 my $count = rmtree({error => \$error});
+
 is( $count, 0, 'rmtree of nothing, count of zero' );
+
 is( scalar(@$error), 0, 'no diagnostic captured' );
 
 @created = mkpath($tmp_base, 0);
+
 is(scalar(@created), 0, "skipped making existing directories (old style 1)")
     or diag("unexpectedly recreated @created");
 
@@ -187,10 +219,13 @@ $dir = catdir($tmp_base,'C');
 # mkpath returns unix syntax filespecs on VMS
 $dir = VMS::Filespec::unixify($dir) if $Is_VMS;
 @created = make_path($tmp_base, $dir);
+
 is(scalar(@created), 1, "created directory (new style 1)");
+
 is($created[0], $dir, "created directory (new style 1) cross-check");
 
 @created = mkpath($tmp_base, 0, 0700);
+
 is(scalar(@created), 0, "skipped making existing directories (old style 2)")
     or diag("unexpectedly recreated @created");
 
@@ -198,26 +233,35 @@ $dir2 = catdir($tmp_base,'D');
 # mkpath returns unix syntax filespecs on VMS
 $dir2 = VMS::Filespec::unixify($dir2) if $Is_VMS;
 @created = make_path($tmp_base, $dir, $dir2);
+
 is(scalar(@created), 1, "created directory (new style 2)");
+
 is($created[0], $dir2, "created directory (new style 2) cross-check");
 
 $count = rmtree($dir, 0);
+
 is($count, 1, "removed directory unsafe mode");
 
+my $expected_count = _cannot_delete_safe_mode($dir2) ? 0 : 1;
+
 $count = rmtree($dir2, 0, 1);
-my $removed = $Is_VMS ? 0 : 1;
-is($count, $removed, "removed directory safe mode");
+
+is($count, $expected_count, "removed directory safe mode");
 
 # mkdir foo ./E/../Y
 # Y should exist
 # existence of E is neither here nor there
 $dir = catdir($tmp_base, 'E', updir(), 'Y');
 @created =mkpath($dir);
+
 cmp_ok(scalar(@created), '>=', 1, "made one or more dirs because of ..");
+
 cmp_ok(scalar(@created), '<=', 2, "made less than two dirs because of ..");
+
 ok( -d catdir($tmp_base, 'Y'), "directory after parent" );
 
 @created = make_path(catdir(curdir(), $tmp_base));
+
 is(scalar(@created), 0, "nothing created")
     or diag(@created);
 
@@ -232,11 +276,14 @@ rmtree( $dir, $dir2,
     }
 );
 
+
 is(scalar(@$error), 0, "no errors unlinking a and z");
+
 is(scalar(@$list),  4, "list contains 4 elements")
     or diag("@$list");
 
 ok(-d $dir,  "dir a still exists");
+
 ok(-d $dir2, "dir z still exists");
 
 $dir = catdir($tmp_base,'F');
@@ -244,27 +291,42 @@ $dir = catdir($tmp_base,'F');
 $dir = VMS::Filespec::unixify($dir) if $Is_VMS;
 
 @created = mkpath($dir, undef, 0770);
+
 is(scalar(@created), 1, "created directory (old style 2 verbose undef)");
+
 is($created[0], $dir, "created directory (old style 2 verbose undef) cross-check");
+
 is(rmtree($dir, undef, 0), 1, "removed directory 2 verbose undef");
 
 @created = mkpath($dir, undef);
+
 is(scalar(@created), 1, "created directory (old style 2a verbose undef)");
+
 is($created[0], $dir, "created directory (old style 2a verbose undef) cross-check");
+
 is(rmtree($dir, undef), 1, "removed directory 2a verbose undef");
 
 @created = mkpath($dir, 0, undef);
+
 is(scalar(@created), 1, "created directory (old style 3 mode undef)");
+
 is($created[0], $dir, "created directory (old style 3 mode undef) cross-check");
+
 is(rmtree($dir, 0, undef), 1, "removed directory 3 verbose undef");
 
-$dir = catdir($tmp_base,'G');
-$dir = VMS::Filespec::unixify($dir) if $Is_VMS;
+SKIP: {
+    skip "fchmod of directories not supported on this platform", 3 unless $fchmod_supported;
+    $dir = catdir($tmp_base,'G');
+    $dir = VMS::Filespec::unixify($dir) if $Is_VMS;
+
+    @created = mkpath($dir, undef, 0400);
 
-@created = mkpath($dir, undef, 0200);
-is(scalar(@created), 1, "created write-only dir");
-is($created[0], $dir, "created write-only directory cross-check");
-is(rmtree($dir), 1, "removed write-only dir");
+    is(scalar(@created), 1, "created read-only dir");
+
+    is($created[0], $dir, "created read-only directory cross-check");
+
+    is(rmtree($dir), 1, "removed read-only dir");
+}
 
 # borderline new-style heuristics
 if (chdir $tmp_base) {
@@ -278,23 +340,47 @@ $dir   = catdir('a', 'd1');
 $dir2  = catdir('a', 'd2');
 
 @created = make_path( $dir, 0, $dir2 );
+
 is(scalar @created, 3, 'new-style 3 dirs created');
 
 $count = remove_tree( $dir, 0, $dir2, );
+
 is($count, 3, 'new-style 3 dirs removed');
 
 @created = make_path( $dir, $dir2, 1 );
+
 is(scalar @created, 3, 'new-style 3 dirs created (redux)');
 
 $count = remove_tree( $dir, $dir2, 1 );
+
 is($count, 3, 'new-style 3 dirs removed (redux)');
 
 @created = make_path( $dir, $dir2 );
+
 is(scalar @created, 2, 'new-style 2 dirs created');
 
 $count = remove_tree( $dir, $dir2 );
+
 is($count, 2, 'new-style 2 dirs removed');
 
+$dir = catdir("a\nb", 'd1');
+$dir2 = catdir("a\nb", 'd2');
+
+SKIP: {
+  # Better to search for *nix derivatives?
+  # Not sure what else doesn't support newline in paths
+  skip "$^O doesn't allow newline in paths", 2
+    if $^O =~ m/^(MSWin32|VMS)$/;
+
+  @created = make_path( $dir, $dir2 );
+
+  is(scalar @created, 3, 'new-style 3 dirs created in parent with newline');
+
+  $count = remove_tree( $dir, $dir2 );
+
+  is($count, 2, 'new-style 2 dirs removed in parent with newline');
+}
+
 if (chdir updir()) {
     pass("chdir parent");
 }
@@ -303,32 +389,17 @@ else {
 }
 
 SKIP: {
-    skip "This is not a MSWin32 platform", 1
-        unless $^O eq 'MSWin32';
-
-    my $UNC_path_taint = $ENV{PERL_FILE_PATH_UNC_TESTDIR};
-    skip "PERL_FILE_PATH_UNC_TESTDIR environment variable not set", 1
-        unless defined($UNC_path_taint);
-
-    my ($UNC_path) = ($UNC_path_taint =~ m{^([/\\]{2}\w+[/\\]\w+[/\\]\w+)$});
-    
-    skip "PERL_FILE_PATH_UNC_TESTDIR environment variable does not point to a directory", 1
-        unless -d $UNC_path;
-    
-    my $removed = rmtree($UNC_path);
-    cmp_ok($removed, '>', 0, "removed $removed entries from $UNC_path");
-}
-
-SKIP: {
     # test bug http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=487319
-    skip "Don't need Force_Writeable semantics on $^O", 4
+    skip "Don't need Force_Writeable semantics on $^O", 6
         if grep {$^O eq $_} qw(amigaos dos epoc MSWin32 MacOS os2);
-    skip "Symlinks not available", 4 unless $Config{d_symlink};
+    skip "Symlinks not available", 6 unless $Config{d_symlink};
     $dir  = 'bug487319';
     $dir2 = 'bug487319-symlink';
     @created = make_path($dir, {mask => 0700});
-    is(scalar @created, 1, 'bug 487319 setup');
+
+    is( scalar @created, 1, 'bug 487319 setup' );
     symlink($dir, $dir2);
+
     ok(-e $dir2, "debian bug 487319 setup symlink") or diag($dir2);
 
     chmod 0500, $dir;
@@ -336,29 +407,39 @@ SKIP: {
     remove_tree($dir2);
 
     my $mask = (stat $dir)[2];
+
     is( $mask, $mask_initial, 'mask of symlink target dir unchanged (debian bug 487319)');
 
     # now try a file
-    my $file = catfile($dir, 'file');
+    #my $file = catfile($dir, 'file');
+    my $file  = 'bug487319-file';
+    my $file2 = 'bug487319-file-symlink';
     open my $out, '>', $file;
     close $out;
 
+    ok(-e $file, 'file exists');
+
     chmod 0500, $file;
     $mask_initial = (stat $file)[2];
 
-    my $file2 = catfile($dir, 'symlink');
     symlink($file, $file2);
+
+    ok(-e $file2, 'file2 exists');
     remove_tree($file2);
 
     $mask = (stat $file)[2];
+
     is( $mask, $mask_initial, 'mask of symlink target file unchanged (debian bug 487319)');
 
     remove_tree($dir);
+    remove_tree($file);
 }
 
 # see what happens if a file exists where we want a directory
 SKIP: {
-    my $entry = catdir($tmp_base, "file");
+    my $entry = catfile($tmp_base, "file");
+    skip "VMS can have a file and a directory with the same name.", 4
+        if $Is_VMS;
     skip "Cannot create $entry", 4 unless open OUT, "> $entry";
     print OUT "test file, safe to delete\n", scalar(localtime), "\n";
     close OUT;
@@ -376,53 +457,6 @@ SKIP: {
         or diag(@created);
 }
 
-my $extra =  catdir(curdir(), qw(EXTRA 1 a));
-
-SKIP: {
-    skip "extra scenarios not set up, see eg/setup-extra-tests", 14
-        unless -e $extra;
-    skip "Symlinks not available", 14 unless $Config{d_symlink};
-
-    my ($list, $err);
-    $dir = catdir( 'EXTRA', '1' );
-    rmtree( $dir, {result => \$list, error => \$err} );
-    is(scalar(@$list), 2, "extra dir $dir removed");
-    is(scalar(@$err), 1, "one error encountered");
-
-    $dir = catdir( 'EXTRA', '3', 'N' );
-    rmtree( $dir, {result => \$list, error => \$err} );
-    is( @$list, 1, q{remove a symlinked dir} );
-    is( @$err,  0, q{with no errors} );
-
-    $dir = catdir('EXTRA', '3', 'S');
-    rmtree($dir, {error => \$error});
-    is( scalar(@$error), 1, 'one error for an unreadable dir' );
-    eval { ($file, $message) = each %{$error->[0]}};
-    is( $file, $dir, 'unreadable dir reported in error' )
-        or diag($message);
-
-    $dir = catdir('EXTRA', '3', 'T');
-    rmtree($dir, {error => \$error});
-    is( scalar(@$error), 1, 'one error for an unreadable dir T' );
-    eval { ($file, $message) = each %{$error->[0]}};
-    is( $file, $dir, 'unreadable dir reported in error T' );
-
-    $dir = catdir( 'EXTRA', '4' );
-    rmtree($dir,  {result => \$list, error => \$err} );
-    is( scalar(@$list), 0, q{don't follow a symlinked dir} );
-    is( scalar(@$err),  2, q{two errors when removing a symlink in r/o dir} );
-    eval { ($file, $message) = each %{$err->[0]} };
-    is( $file, $dir, 'symlink reported in error' );
-
-    $dir  = catdir('EXTRA', '3', 'U');
-    $dir2 = catdir('EXTRA', '3', 'V');
-    rmtree($dir, $dir2, {verbose => 0, error => \$err, result => \$list});
-    is( scalar(@$list),  1, q{deleted 1 out of 2 directories} );
-    is( scalar(@$error), 1, q{left behind 1 out of 2 directories} );
-    eval { ($file, $message) = each %{$err->[0]} };
-    is( $file, $dir, 'first dir reported in error' );
-}
-
 {
     $dir = catdir($tmp_base, 'ZZ');
     @created = mkpath($dir);
@@ -433,206 +467,176 @@ SKIP: {
     ok(!-e $dir, "blow it away via \@ARGV");
 }
 
-SKIP: {
-    my $skip_count = 8; # DRY
-    skip "getpwent() not implemented on $^O", $skip_count
-        unless $Config{d_getpwent};
-    skip "getgrent() not implemented on $^O", $skip_count
-        unless $Config{d_getgrent};
-    skip 'not running as root', $skip_count
-        unless $< == 0;
-    skip "darwin's nobody and nogroup are -1", $skip_count
-        if $^O eq 'darwin';
-
-    my $dir_stem = $dir = catdir($tmp_base, 'owned-by');
-
-    # find the highest uid ('nobody' or similar)
-    my $max_uid   = 0;
-    my $max_user = undef;
-    while (my @u = getpwent()) {
-        if ($max_uid < $u[2]) {
-            $max_uid  = $u[2];
-            $max_user = $u[0];
-        }
-    }
-    skip 'getpwent() appears to be insane', $skip_count
-        unless $max_uid > 0;
-
-    # find the highest gid ('nogroup' or similar)
-    my $max_gid   = 0;
-    my $max_group = undef;
-    while (my @g = getgrent()) {
-        if ($max_gid < $g[2]) {
-            $max_gid = $g[2];
-            $max_group = $g[0];
-        }
-    }
-    skip 'getgrent() appears to be insane', $skip_count
-        unless $max_gid > 0;
-
-    $dir = catdir($dir_stem, 'aaa');
-    @created = make_path($dir, {owner => $max_user});
-    is(scalar(@created), 2, "created a directory owned by $max_user...");
-    my $dir_uid = (stat $created[0])[4];
-    is($dir_uid, $max_uid, "... owned by $max_uid");
-
-    $dir = catdir($dir_stem, 'aab');
-    @created = make_path($dir, {group => $max_group});
-    is(scalar(@created), 1, "created a directory owned by group $max_group...");
-    my $dir_gid = (stat $created[0])[5];
-    is($dir_gid, $max_gid, "... owned by group $max_gid");
-
-    $dir = catdir($dir_stem, 'aac');
-    @created = make_path($dir, {user => $max_user, group => $max_group});
-    is(scalar(@created), 1, "created a directory owned by $max_user:$max_group...");
-    ($dir_uid, $dir_gid) = (stat $created[0])[4,5];
-    is($dir_uid, $max_uid, "... owned by $max_uid");
-    is($dir_gid, $max_gid, "... owned by group $max_gid");
-
-    SKIP: {
-        skip 'Test::Output not available', 1
-               unless $has_Test_Output;
-
-        # invent a user and group that don't exist
-        do { ++$max_user  } while (getpwnam($max_user));
-        do { ++$max_group } while (getgrnam($max_group));
-
-        $dir = catdir($dir_stem, 'aad');
-        stderr_like(
-            sub {make_path($dir, {user => $max_user, group => $max_group})},
-            qr{\Aunable to map $max_user to a uid, ownership not changed: .* at \S+ line \d+
-unable to map $max_group to a gid, group ownership not changed: .* at \S+ line \d+\b},
-            "created a directory not owned by $max_user:$max_group..."
-        );
+SKIP : {
+    my $skip_count = 18;
+    # this test will fail on Windows, as per:
+    #   http://perldoc.perl.org/perlport.html#chmod
+
+    skip "Windows chmod test skipped", $skip_count
+        if $^O eq 'MSWin32';
+    skip "fchmod() on directories is not supported on this platform", $skip_count
+        unless $fchmod_supported;
+    my $mode;
+    my $octal_mode;
+    my @inputs = (
+      0777, 0700, 0470, 0407,
+      0433, 0400, 0430, 0403,
+      0111, 0100, 0110, 0101,
+      0731, 0713, 0317, 0371,
+      0173, 0137);
+    my $input;
+    my $octal_input;
+
+    foreach (@inputs) {
+        $input = $_;
+        $dir = catdir($tmp_base, sprintf("chmod_test%04o", $input));
+        # We can skip from here because 0 is last in the list.
+        skip "Mode of 0 means assume user defaults on VMS", 1
+          if ($input == 0 && $Is_VMS);
+        @created = mkpath($dir, {chmod => $input});
+        $mode = (stat($dir))[2];
+        $octal_mode = S_IMODE($mode);
+        $octal_input = sprintf "%04o", S_IMODE($input);
+        SKIP: {
+           skip "permissions are not fully supported by the filesystem", 1
+                if (($^O eq 'MSWin32' || $^O eq 'cygwin') && ((Win32::FsType())[1] & 8) == 0);
+            is($octal_mode,$input, "create a new directory with chmod $input ($octal_input)");
+           }
+        rmtree( $dir );
     }
 }
 
-SKIP: {
-    skip 'Test::Output not available', 14
-        unless $has_Test_Output;
+my $dir_base = catdir($tmp_base,'output');
+my $dir_a    = catdir($dir_base, 'A');
+my $dir_b    = catdir($dir_base, 'B');
 
-    SKIP: {
-        $dir = catdir('EXTRA', '3');
-        skip "extra scenarios not set up, see eg/setup-extra-tests", 3
-            unless -e $dir;
-
-        $dir = catdir('EXTRA', '3', 'U');
-        stderr_like( 
-            sub {rmtree($dir, {verbose => 0})},
-            qr{\Acannot make child directory read-write-exec for [^:]+: .* at \S+ line \d+},
-            q(rmtree can't chdir into root dir)
-        );
+is(_run_for_verbose(sub {@created = mkpath($dir_a, 1)}),
+    _verbose_expected('mkpath', $dir_base, 0, 1)
+    . _verbose_expected('mkpath', $dir_a, 0),
+    'mkpath verbose (old style 1)'
+);
 
-        $dir = catdir('EXTRA', '3');
-        stderr_like( 
-            sub {rmtree($dir, {})},
-            qr{\Acannot make child directory read-write-exec for [^:]+: .* at (\S+) line (\d+)
-cannot make child directory read-write-exec for [^:]+: .* at \1 line \2
-cannot make child directory read-write-exec for [^:]+: .* at \1 line \2
-cannot remove directory for [^:]+: .* at \1 line \2},
-            'rmtree with file owned by root'
-        );
+is(_run_for_verbose(sub {@created = mkpath([$dir_b], 1)}),
+    _verbose_expected('mkpath', $dir_b, 0),
+    'mkpath verbose (old style 2)'
+);
 
-        stderr_like( 
-            sub {rmtree('EXTRA', {})},
-            qr{\Acannot remove directory for [^:]+: .* at (\S+) line (\d+)
-cannot remove directory for [^:]+: .* at \1 line \2
-cannot make child directory read-write-exec for [^:]+: .* at \1 line \2
-cannot make child directory read-write-exec for [^:]+: .* at \1 line \2
-cannot make child directory read-write-exec for [^:]+: .* at \1 line \2
-cannot remove directory for [^:]+: .* at \1 line \2
-cannot unlink file for [^:]+: .* at \1 line \2
-cannot restore permissions to \d+ for [^:]+: .* at \1 line \2
-cannot make child directory read-write-exec for [^:]+: .* at \1 line \2
-cannot remove directory for [^:]+: .* at \1 line \2},
-            'rmtree with insufficient privileges'
-        );
-    }
+my $verbose_expected;
 
-    my $base = catdir($tmp_base,'output');
-    $dir  = catdir($base,'A');
-    $dir2 = catdir($base,'B');
+# Must determine expectations while directories still exist.
+$verbose_expected = _verbose_expected('rmtree', $dir_a, 1)
+                  . _verbose_expected('rmtree', $dir_b, 1);
 
-    stderr_like(
-        sub { rmtree( undef, 1 ) },
-        qr/\ANo root path\(s\) specified\b/,
-        "rmtree of nothing carps sensibly"
-    );
+is(_run_for_verbose(sub {$count = rmtree([$dir_a, $dir_b], 1, 1)}),
+    $verbose_expected,
+    'rmtree verbose (old style)'
+);
 
-    stderr_like(
-        sub { rmtree( '', 1 ) },
-        qr/\ANo root path\(s\) specified\b/,
-        "rmtree of empty dir carps sensibly"
-    );
+# In case we didn't delete them in safe mode.
+rmtree($dir_a) if -d $dir_a;
+rmtree($dir_b) if -d $dir_b;
 
-    stderr_is( sub { make_path() }, '', "make_path no args does not carp" );
-    stderr_is( sub { remove_tree() }, '', "remove_tree no args does not carp" );
+is(_run_for_verbose(sub {@created = mkpath( $dir_a,
+                                            {verbose => 1, mask => 0750})}),
+    _verbose_expected('mkpath', $dir_a, 0),
+    'mkpath verbose (new style 1)'
+);
 
-    stdout_is(
-        sub {@created = mkpath($dir, 1)},
-        "mkdir $base\nmkdir $dir\n",
-        'mkpath verbose (old style 1)'
-    );
+is(_run_for_verbose(sub {@created = mkpath($dir_b, 1, 0771)}),
+    _verbose_expected('mkpath', $dir_b, 0),
+    'mkpath verbose (new style 2)'
+);
 
-    stdout_is(
-        sub {@created = mkpath([$dir2], 1)},
-        "mkdir $dir2\n",
-        'mkpath verbose (old style 2)'
-    );
+$verbose_expected = _verbose_expected('rmtree', $dir_a, 1)
+                  . _verbose_expected('rmtree', $dir_b, 1);
 
-    stdout_is(
-        sub {$count = rmtree([$dir, $dir2], 1, 1)},
-        "rmdir $dir\nrmdir $dir2\n",
-        'rmtree verbose (old style)'
-    );
+is(_run_for_verbose(sub {$count = rmtree([$dir_a, $dir_b], 1, 1)}),
+    $verbose_expected,
+    'again: rmtree verbose (old style)'
+);
 
-    stdout_is(
-        sub {@created = mkpath($dir, {verbose => 1, mask => 0750})},
-        "mkdir $dir\n",
-        'mkpath verbose (new style 1)'
-    );
+rmtree($dir_a) if -d $dir_a;
+rmtree($dir_b) if -d $dir_b;
 
-    stdout_is(
-        sub {@created = mkpath($dir2, 1, 0771)},
-        "mkdir $dir2\n",
-        'mkpath verbose (new style 2)'
-    );
+is(_run_for_verbose(sub {@created = make_path( $dir_a, $dir_b,
+                                               {verbose => 1, mode => 0711});}),
+      _verbose_expected('make_path', $dir_a, 1)
+    . _verbose_expected('make_path', $dir_b, 1),
+    'make_path verbose with final hashref'
+);
 
-    SKIP: {
-        $file = catdir($dir2, "file");
-        skip "Cannot create $file", 2 unless open OUT, "> $file";
-        print OUT "test file, safe to delete\n", scalar(localtime), "\n";
-        close OUT;
+$verbose_expected = _verbose_expected('remove_tree', $dir_a, 0)
+                  . _verbose_expected('remove_tree', $dir_b, 0);
 
-        ok(-e $file, "file created in directory");
+is(_run_for_verbose(sub {@created = remove_tree( $dir_a, $dir_b,
+                                                 {verbose => 1});}),
+    $verbose_expected,
+    'remove_tree verbose with final hashref'
+);
 
-        stdout_is(
-            sub {$count = rmtree($dir, $dir2, {verbose => 1, safe => 1})},
-            "rmdir $dir\nunlink $file\nrmdir $dir2\n",
-            'rmtree safe verbose (new style)'
+rmtree($dir_a) if -d $dir_a;
+rmtree($dir_b) if -d $dir_b;
+
+# Have to re-create these 2 directories so that next block is not skipped.
+@created = make_path(
+    $dir_a,
+    $dir_b,
+    { mode => 0711 }
+);
+is(@created, 2, "2 directories created");
+
+SKIP: {
+    $file = catfile($dir_b, "file");
+    skip "Cannot create $file", 2 unless open OUT, "> $file";
+    print OUT "test file, safe to delete\n", scalar(localtime), "\n";
+    close OUT;
+
+    $verbose_expected = _verbose_expected('rmtree', $dir_a, 1)
+                      . _verbose_expected('unlink', $file, 0)
+                      . _verbose_expected('rmtree', $dir_b, 1);
+
+    ok(-e $file, "file created in directory");
+
+    is(_run_for_verbose(sub {$count = rmtree( $dir_a, $dir_b,
+                                              {verbose => 1, safe => 1})}),
+        $verbose_expected,
+        'rmtree safe verbose (new style)'
+    );
+    rmtree($dir_a) if -d $dir_a;
+    rmtree($dir_b) if -d $dir_b;
+}
+
+{
+    my $base = catdir( $tmp_base, 'output2');
+    my $dir  = catdir( $base, 'A');
+    my $dir2 = catdir( $base, 'B');
+
+    {
+        my $warn = _run_for_warning( sub {
+            my @created = make_path(
+                $dir,
+                $dir2,
+                { mode => 0711, foo => 1, bar => 1 }
+            );
+        } );
+        like($warn,
+            qr/Unrecognized option\(s\) passed to mkpath\(\) or make_path\(\):.*?bar.*?foo/,
+            'make_path with final hashref warned due to unrecognized options'
         );
     }
-}
 
-SKIP: {
-    skip "extra scenarios not set up, see eg/setup-extra-tests", 11
-        unless -d catdir(qw(EXTRA 1));
-
-    rmtree 'EXTRA', {safe => 0, error => \$error};
-    is( scalar(@$error), 10, 'seven deadly sins' ); # well there used to be 7
-
-    rmtree 'EXTRA', {safe => 1, error => \$error};
-    is( scalar(@$error), 9, 'safe is better' );
-    for (@$error) {
-        ($file, $message) = each %$_;
-        if ($file =~  /[123]\z/) {
-            is(index($message, 'cannot remove directory: '), 0, "failed to remove $file with rmdir")
-                or diag($message);
-        }
-        else {
-            like($message, qr(\Acannot (?:restore permissions to \d+|chdir to child|unlink file): ), "failed to remove $file with unlink")
-                or diag($message)
-        }
+    {
+        my $warn = _run_for_warning( sub {
+            my @created = remove_tree(
+                $dir,
+                $dir2,
+                { foo => 1, bar => 1 }
+            );
+        } );
+        like($warn,
+            qr/Unrecognized option\(s\) passed to remove_tree\(\):.*?bar.*?foo/,
+            'remove_tree with final hashref failed due to unrecognized options'
+        );
     }
 }
 
@@ -642,21 +646,314 @@ SKIP: {
     rmtree($tmp_base, {result => \$list} );
     is(ref($list), 'ARRAY', "received a final list of results");
     ok( !(-d $tmp_base), "test base directory gone" );
-    
+
     my $p = getcwd();
     my $x = "x$$";
     my $xx = $x . "x";
-    
+
     # setup
     ok(mkpath($xx), "make $xx");
     ok(chdir($xx), "... and chdir $xx");
     END {
-         ok(chdir($p), "... now chdir $p");
-         ok(rmtree($xx), "... and finally rmtree $xx");
+#         ok(chdir($p), "... now chdir $p");
+#         ok(rmtree($xx), "... and finally rmtree $xx");
+       chdir($p);
+       rmtree($xx);
     }
-    
+
     # create and delete directory
     my $px = catdir($p, $x);
     ok(mkpath($px), 'create and delete directory 2.07');
     ok(rmtree($px), '.. rmtree fails in File-Path-2.07');
+    chdir updir();
+}
+
+my $windows_dir = 'C:\Path\To\Dir';
+my $expect = 'c:/path/to/dir';
+is(
+    File::Path::_slash_lc($windows_dir),
+    $expect,
+    "Windows path unixified as expected"
+);
+
+{
+    my ($x, $message, $object, $expect, $rv, $arg, $error);
+    my ($k, $v, $second_error, $third_error);
+    local $! = ENOENT;
+    $x = $!;
+
+    $message = 'message in a bottle';
+    $object = '/path/to/glory';
+    $expect = "$message for $object: $x";
+    $rv = _run_for_warning( sub {
+        File::Path::_error(
+            {},
+            $message,
+            $object
+        );
+    } );
+    like($rv, qr/^$expect/,
+        "no \$arg->{error}: defined 2nd and 3rd args: got expected error message");
+
+    $object = undef;
+    $expect = "$message: $x";
+    $rv = _run_for_warning( sub {
+        File::Path::_error(
+            {},
+            $message,
+            $object
+        );
+    } );
+    like($rv, qr/^$expect/,
+        "no \$arg->{error}: defined 2nd arg; undefined 3rd arg: got expected error message");
+
+    $message = 'message in a bottle';
+    $object = undef;
+    $expect = "$message: $x";
+    $arg = { error => \$error };
+    File::Path::_error(
+        $arg,
+        $message,
+        $object
+    );
+    is(ref($error->[0]), 'HASH',
+        "first element of array inside \$error is hashref");
+    ($k, $v) = %{$error->[0]};
+    is($k, '', 'key of hash is empty string, since 3rd arg was undef');
+    is($v, $expect, "value of hash is 2nd arg: $message");
+
+    $message = '';
+    $object = '/path/to/glory';
+    $expect = "$message: $x";
+    $arg = { error => \$second_error };
+    File::Path::_error(
+        $arg,
+        $message,
+        $object
+    );
+    is(ref($second_error->[0]), 'HASH',
+        "first element of array inside \$second_error is hashref");
+    ($k, $v) = %{$second_error->[0]};
+    is($k, $object, "key of hash is '$object', since 3rd arg was defined");
+    is($v, $expect, "value of hash is 2nd arg: $message");
+
+    $message = '';
+    $object = undef;
+    $expect = "$message: $x";
+    $arg = { error => \$third_error };
+    File::Path::_error(
+        $arg,
+        $message,
+        $object
+    );
+    is(ref($third_error->[0]), 'HASH',
+        "first element of array inside \$third_error is hashref");
+    ($k, $v) = %{$third_error->[0]};
+    is($k, '', "key of hash is empty string, since 3rd arg was undef");
+    is($v, $expect, "value of hash is 2nd arg: $message");
+}
+
+{
+    # https://rt.cpan.org/Ticket/Display.html?id=117019
+    # remove_tree(): Permit re-use of options hash without issuing a warning
+
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| ZoYhvc6RmGnl S2CrQ0lju0o7 lvOqVYWpfhcP | );
+    my @created;
+    @created = File::Path::make_path($deepest, { mode => 0711 });
+    is(scalar(@created), 3, "Created 3 subdirectories");
+
+    my $x = '';
+    my $opts = { error => \$x };
+    File::Path::remove_tree($deepest, $opts);
+    ok(! -d $deepest, "directory '$deepest' removed, as expected");
+
+    my $warn;
+    $warn = _run_for_warning( sub { File::Path::remove_tree($next_deepest, $opts); } );
+    ok(! $warn, "CPAN 117019: No warning thrown when re-using \$opts");
+    ok(! -d $next_deepest, "directory '$next_deepest' removed, as expected");
+
+    $warn = _run_for_warning( sub { File::Path::remove_tree($least_deep, $opts); } );
+    ok(! $warn, "CPAN 117019: No warning thrown when re-using \$opts");
+    ok(! -d $least_deep, "directory '$least_deep' removed, as expected");
+}
+
+{
+    # Corner cases with respect to arguments provided to functions
+    my $count;
+
+    $count = remove_tree();
+    is($count, 0,
+        "If not provided with any paths, remove_tree() will return a count of 0 things deleted");
+
+    $count = remove_tree('');
+    is($count, 0,
+        "If not provided with any paths, remove_tree() will return a count of 0 things deleted");
+
+    my $warn;
+    $warn = _run_for_warning( sub { $count = rmtree(); } );
+    like($warn, qr/No root path\(s\) specified/s, "Got expected carp");
+    is($count, 0,
+        "If not provided with any paths, remove_tree() will return a count of 0 things deleted");
+
+    $warn = _run_for_warning( sub {$count = rmtree(undef); } );
+    like($warn, qr/No root path\(s\) specified/s, "Got expected carp");
+    is($count, 0,
+        "If provided only with an undefined value, remove_tree() will return a count of 0 things deleted");
+
+    $warn = _run_for_warning( sub {$count = rmtree(''); } );
+    like($warn, qr/No root path\(s\) specified/s, "Got expected carp");
+    is($count, 0,
+        "If provided with an empty string for a path, remove_tree() will return a count of 0 things deleted");
+
+    $count = make_path();
+    is($count, 0,
+        "If not provided with any paths, make_path() will return a count of 0 things created");
+
+    $count = mkpath();
+    is($count, 0,
+        "If not provided with any paths, make_path() will return a count of 0 things created");
+}
+
+SKIP: {
+    my $skip_count = 3;
+    skip "Windows will not set this error condition", $skip_count
+        if $^O eq 'MSWin32';
+
+    # mkpath() with hashref:  case of phony user
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| Hhu1KpF4EVAV vUj5k37bih8v Vkdw02POXJxj | );
+    my (@created, $error);
+    my $user = join('_' => 'foobar', $$);
+    @created = mkpath($deepest, { mode => 0711, user => $user, error => \$error });
+#    TODO: {
+#        local $TODO = "Notwithstanding the phony 'user', mkpath will actually create subdirectories; should it?";
+#        is(scalar(@created), 0, "No subdirectories created");
+#    }
+    is(scalar(@$error), 1, "caught error condition" );
+    my ($file, $message) = each %{$error->[0]};
+    like($message,
+        qr/unable to map $user to a uid, ownership not changed/s,
+        "Got expected error message for phony user",
+    );
+
+    cleanup_3_level_subdirs($least_deep);
+}
+
+{
+    # mkpath() with hashref:  case of valid uid
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| b5wj8CJcc7gl XTJe2C3WGLg5 VZ_y2T0XfKu3 | );
+    my (@created, $error);
+    my $warn;
+    local $SIG{__WARN__} = sub { $warn = shift };
+    @created = mkpath($deepest, { mode => 0711, uid => $>, error => \$error });
+    SKIP: {
+        my $skip_count = 1;
+        skip "Warning should only appear on Windows", $skip_count
+            unless $^O eq 'MSWin32';
+        like($warn,
+            qr/Option\(s\) implausible on Win32 passed to mkpath\(\) or make_path\(\)/,
+            'make_path with final hashref warned due to options implausible on Win32'
+        );
+    }
+    is(scalar(@created), 3, "Provide valid 'uid' argument: 3 subdirectories created");
+
+    cleanup_3_level_subdirs($least_deep);
+}
+
+SKIP: {
+    my $skip_count = 3;
+    skip "getpwuid() and getgrgid() not implemented on Windows", $skip_count
+        if $^O eq 'MSWin32';
+
+    # mkpath() with hashref:  case of valid owner
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| aiJEDKaAEH25 nqhXsBM_7_bv qfRj4cur4Jrs | );
+    my (@created, $error);
+    my $name = getpwuid($>);
+    @created = mkpath($deepest, { mode => 0711, owner => $name, error => \$error });
+    is(scalar(@created), 3, "Provide valid 'owner' argument: 3 subdirectories created");
+
+    cleanup_3_level_subdirs($least_deep);
+}
+
+SKIP: {
+    my $skip_count = 5;
+    skip "Windows will not set this error condition", $skip_count
+        if $^O eq 'MSWin32';
+
+    # mkpath() with hashref:  case of phony group
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| nOR4lGRMdLvz NnwkEHEVL5li _3f1Kv6q77yA | );
+    my (@created, $error);
+    my $bad_group = join('_' => 'foobarbaz', $$);
+    @created = mkpath($deepest, { mode => 0711, group => $bad_group, error => \$error });
+#    TODO: {
+#        local $TODO = "Notwithstanding the phony 'group', mkpath will actually create subdirectories; should it?";
+#        is(scalar(@created), 0, "No subdirectories created");
+#    }
+    is(scalar(@$error), 1, "caught error condition" );
+    my ($file, $message) = each %{$error->[0]};
+    like($message,
+        qr/unable to map $bad_group to a gid, group ownership not changed/s,
+        "Got expected error message for phony user",
+    );
+
+    cleanup_3_level_subdirs($least_deep);
+}
+
+{
+    # mkpath() with hashref:  case of valid group
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| BEcigvaBNisY rd4lJ1iZRyeS OyQnDPIBxP2K | );
+    my (@created, $error);
+    my $warn;
+    local $SIG{__WARN__} = sub { $warn = shift };
+    @created = mkpath($deepest, { mode => 0711, group => $(, error => \$error });
+    SKIP: {
+        my $skip_count = 1;
+        skip "Warning should only appear on Windows", $skip_count
+            unless $^O eq 'MSWin32';
+        like($warn,
+            qr/Option\(s\) implausible on Win32 passed to mkpath\(\) or make_path\(\)/,
+            'make_path with final hashref warned due to options implausible on Win32'
+        );
+    }
+    is(scalar(@created), 3, "Provide valid 'group' argument: 3 subdirectories created");
+
+    cleanup_3_level_subdirs($least_deep);
+}
+
+SKIP: {
+    my $skip_count = 3;
+    skip "getpwuid() and getgrgid() not implemented on Windows", $skip_count
+        if $^O eq 'MSWin32';
+
+    # mkpath() with hashref:  case of valid group
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| IayhWFDvys8X gTd6gaeuFzmV VVI6UWLJCOEC | );
+    my (@created, $error);
+    my $group_name = (getgrgid($())[0];
+    @created = mkpath($deepest, { mode => 0711, group => $group_name, error => \$error });
+    is(scalar(@created), 3, "Provide valid 'group' argument: 3 subdirectories created");
+
+    cleanup_3_level_subdirs($least_deep);
+}
+
+SKIP: {
+    my $skip_count = 3;
+    skip "getpwuid() and getgrgid() not implemented on Windows", $skip_count
+        if $^O eq 'MSWin32';
+
+    # mkpath() with hashref:  case of valid owner and group
+    my ($least_deep, $next_deepest, $deepest) =
+        create_3_level_subdirs( qw| xsmOvlnxOqJc olsGlBSoVUpp tDuRilkD35rd | );
+    my (@created, $error);
+    my $name = getpwuid($>);
+    my $group_name = (getgrgid($())[0];
+    @created = mkpath($deepest, { mode => 0711, owner => $name, group => $group_name, error => \$error });
+    is(scalar(@created), 3, "Provide valid 'owner' and 'group' 'group' arguments: 3 subdirectories created");
+
+    cleanup_3_level_subdirs($least_deep);
 }