X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FFile%2FCopy.t;h=1107f53bf0cb973cd4e250c365704b1e9624aa8d;hb=68c65ec0adda9d2b3fc21f30f68be0ef10de5ad8;hp=5e7538355c1d74822c7490e89cd0ea4a753dcbcc;hpb=754f2cd0b9642a979bd5574039c0a20a0cfd79ca;p=p5sagit%2Fp5-mst-13.2.git diff --git a/lib/File/Copy.t b/lib/File/Copy.t index 5e75383..1107f53 100755 --- a/lib/File/Copy.t +++ b/lib/File/Copy.t @@ -1,15 +1,20 @@ -#!./perl +#!./perl -w BEGIN { - chdir 't' if -d 't'; - @INC = '../lib'; + if( $ENV{PERL_CORE} ) { + chdir 't' if -d 't'; + @INC = '../lib'; + } } +use strict; +use warnings; + use Test::More; my $TB = Test::More->builder; -plan tests => 60; +plan tests => 91; # We're going to override rename() later on but Perl has to see an override # at compile time to honor it. @@ -25,7 +30,7 @@ foreach my $code ("copy()", "copy('arg')", "copy('arg', 'arg', 'arg', 'arg')", ) { eval $code; - like $@, qr/^Usage: /; + like $@, qr/^Usage: /, "'$code' is a usage error"; } @@ -38,21 +43,22 @@ for my $cross_partition_test (0..1) { } # First we create a file - open(F, ">file-$$") or die; + open(F, ">file-$$") or die $!; binmode F; # for DOSISH platforms, because test 3 copies to stdout printf F "ok\n"; close F; copy "file-$$", "copy-$$"; - open(F, "copy-$$") or die; - $foo = ; + open(F, "copy-$$") or die $!; + my $foo = ; close(F); - is -s "file-$$", -s "copy-$$"; + is -s "file-$$", -s "copy-$$", 'copy(fn, fn): files of the same size'; - is $foo, "ok\n"; + is $foo, "ok\n", 'copy(fn, fn): same contents'; + print("# next test checks copying to STDOUT\n"); binmode STDOUT unless $^O eq 'VMS'; # Copy::copy works in binary mode # This outputs "ok" so its a test. copy "copy-$$", \*STDOUT; @@ -62,32 +68,32 @@ for my $cross_partition_test (0..1) { open(F,"file-$$"); copy(*F, "copy-$$"); open(R, "copy-$$") or die "open copy-$$: $!"; $foo = ; close(R); - is $foo, "ok\n"; + is $foo, "ok\n", 'copy(*F, fn): same contents'; unlink "copy-$$" or die "unlink: $!"; open(F,"file-$$"); copy(\*F, "copy-$$"); close(F) or die "close: $!"; open(R, "copy-$$") or die; $foo = ; close(R) or die "close: $!"; - is $foo, "ok\n"; + is $foo, "ok\n", 'copy(\*F, fn): same contents'; unlink "copy-$$" or die "unlink: $!"; require IO::File; - $fh = IO::File->new(">copy-$$") or die "Cannot open copy-$$:$!"; - binmode $fh or die; + my $fh = IO::File->new(">copy-$$") or die "Cannot open copy-$$:$!"; + binmode $fh or die $!; copy("file-$$",$fh); $fh->close or die "close: $!"; open(R, "copy-$$") or die; $foo = ; close(R); - is $foo, "ok\n"; + is $foo, "ok\n", 'copy(fn, io): same contents'; unlink "copy-$$" or die "unlink: $!"; require FileHandle; - my $fh = FileHandle->new(">copy-$$") or die "Cannot open copy-$$:$!"; - binmode $fh or die; + $fh = FileHandle->new(">copy-$$") or die "Cannot open copy-$$:$!"; + binmode $fh or die $!; copy("file-$$",$fh); $fh->close; - open(R, "copy-$$") or die; $foo = ; close(R); - is $foo, "ok\n"; + open(R, "copy-$$") or die $!; $foo = ; close(R); + is $foo, "ok\n", 'copy(fn, fh): same contents'; unlink "file-$$" or die "unlink: $!"; ok !move("file-$$", "copy-$$"), "move on missing file"; @@ -105,38 +111,44 @@ for my $cross_partition_test (0..1) { ok move("copy-$$", "file-$$"), 'move'; ok -e "file-$$", ' destination exists'; ok !-e "copy-$$", ' source does not'; - open(R, "file-$$") or die; $foo = ; close(R); - is $foo, "ok\n"; + open(R, "file-$$") or die $!; $foo = ; close(R); + is $foo, "ok\n", 'contents preserved'; - my $dest_mtime = (stat("file-$$"))[9]; - is $dest_mtime, $mtime, - "mtime preserved by copy()". - ($cross_partition_test ? " while testing cross-partition" : ""); + TODO: { + local $TODO = 'mtime only preserved on ODS-5 with POSIX dates and DECC$EFS_FILE_TIMESTAMPS enabled' if $^O eq 'VMS'; + + my $dest_mtime = (stat("file-$$"))[9]; + is $dest_mtime, $mtime, + "mtime preserved by copy()". + ($cross_partition_test ? " while testing cross-partition" : ""); + } + # trick: create lib/ if not exists - not needed in Perl core + unless (-d 'lib') { mkdir 'lib' or die $!; } copy "file-$$", "lib"; - open(R, "lib/file-$$") or die; $foo = ; close(R); - is $foo, "ok\n"; + open(R, "lib/file-$$") or die $!; $foo = ; close(R); + is $foo, "ok\n", 'copy(fn, dir): same contents'; unlink "lib/file-$$" or die "unlink: $!"; # Do it twice to ensure copying over the same file works. copy "file-$$", "lib"; - open(R, "lib/file-$$") or die; $foo = ; close(R); - is $foo, "ok\n"; + open(R, "lib/file-$$") or die $!; $foo = ; close(R); + is $foo, "ok\n", 'copy over the same file works'; unlink "lib/file-$$" or die "unlink: $!"; { my $warnings = ''; local $SIG{__WARN__} = sub { $warnings .= join '', @_ }; - ok copy("file-$$", "file-$$"); + ok copy("file-$$", "file-$$"), 'copy(fn, fn) succeeds'; - like $warnings, qr/are identical/; - ok -s "file-$$"; + like $warnings, qr/are identical/, 'but warns'; + ok -s "file-$$", 'contents preserved'; } move "file-$$", "lib"; open(R, "lib/file-$$") or die "open lib/file-$$: $!"; $foo = ; close(R); - is $foo, "ok\n"; - ok !-e "file-$$"; + is $foo, "ok\n", 'move(fn, dir): same contents'; + ok !-e "file-$$", 'file moved indeed'; unlink "lib/file-$$" or die "unlink: $!"; SKIP: { @@ -149,18 +161,19 @@ for my $cross_partition_test (0..1) { my $warnings = ''; local $SIG{__WARN__} = sub { $warnings .= join '', @_ }; - ok !copy("file-$$", "symlink-$$"); + ok !copy("file-$$", "symlink-$$"), 'copy to itself (via symlink) fails'; - like $warnings, qr/are identical/; + like $warnings, qr/are identical/, 'emits a warning'; ok !-z "file-$$", 'rt.perl.org 5196: copying to itself would truncate the file'; - unlink "symlink-$$"; - unlink "file-$$"; + unlink "symlink-$$" or die $!; + unlink "file-$$" or die $!; } SKIP: { - skip "Testing hard links", 3 if !$Config{d_link} or $^O eq 'MSWin32'; + skip "Testing hard links", 3 + if !$Config{d_link} or $^O eq 'MSWin32' or $^O eq 'cygwin'; open(F, ">file-$$") or die $!; print F "dummy content\n"; @@ -169,15 +182,107 @@ for my $cross_partition_test (0..1) { my $warnings = ''; local $SIG{__WARN__} = sub { $warnings .= join '', @_ }; - ok !copy("file-$$", "hardlink-$$"); + ok !copy("file-$$", "hardlink-$$"), 'copy to itself (via hardlink) fails'; - like $warnings, qr/are identical/; + like $warnings, qr/are identical/, 'emits a warning'; ok ! -z "file-$$", 'rt.perl.org 5196: copying to itself would truncate the file'; - unlink "hardlink-$$"; - unlink "file-$$"; + unlink "hardlink-$$" or die $!; + unlink "file-$$" or die $!; + } + + open(F, ">file-$$") or die $!; + binmode F; + print F "this is file\n"; + close F; + + my $copy_msg = "this is copy\n"; + open(F, ">copy-$$") or die $!; + binmode F; + print F $copy_msg; + close F; + + my @warnings; + local $SIG{__WARN__} = sub { push @warnings, join '', @_ }; + + # pie-$$ so that we force a non-constant, else the numeric conversion (of 0) + # is cached and we don't get a warning the second time round + is eval { copy("file-$$", "copy-$$", "pie-$$"); 1 }, undef, + "a bad buffer size fails to copy"; + like $@, qr/Bad buffer size for copy/, "with a helpful error message"; + unless (is scalar @warnings, 1, "There is 1 warning") { + diag $_ foreach @warnings; } + + is -s "copy-$$", length $copy_msg, "but does not truncate the destination"; + open(F, "copy-$$") or die $!; + $foo = ; + close(F); + is $foo, $copy_msg, "nor change the destination's contents"; + + unlink "file-$$" or die $!; + unlink "copy-$$" or die $!; +} + + +{ + # Just a sub to get better failure messages. + sub __ ($) { + join "" => map {(qw [--- --x -w- -wx r-- r-x rw- rwx]) [$_]} + split // => sprintf "%03o" => shift + } + # Testing permission bits. + my $src = "file-$$"; + my $copy1 = "copy1-$$"; + my $copy2 = "copy2-$$"; + my $copy3 = "copy3-$$"; + + open my $fh => ">", $src or die $!; + close $fh or die $!; + + open $fh => ">", $copy3 or die $!; + close $fh or die $!; + + my @tests = ( + [0000, 0777, 0777, 0777], + [0000, 0751, 0751, 0644], + [0022, 0777, 0755, 0206], + [0022, 0415, 0415, 0666], + [0077, 0777, 0700, 0333], + [0027, 0755, 0750, 0251], + [0777, 0751, 0000, 0215], + ); + my $old_mask = umask; + foreach my $test (@tests) { + my ($umask, $s_perm, $c_perm1, $c_perm3) = @$test; + # Make sure the copies doesn't exist. + ! -e $_ or unlink $_ or die $! for $copy1, $copy2; + + (umask $umask) // die $!; + chmod $s_perm => $src or die $!; + chmod $c_perm3 => $copy3 or die $!; + + open my $fh => "<", $src or die $!; + + copy ($src, $copy1); + copy ($fh, $copy2); + copy ($src, $copy3); + + my $perm1 = (stat $copy1) [2] & 0xFFF; + my $perm2 = (stat $copy2) [2] & 0xFFF; + my $perm3 = (stat $copy3) [2] & 0xFFF; + is (__$perm1, __$c_perm1, "Permission bits set correctly"); + is (__$perm2, __$c_perm1, "Permission bits set correctly"); + TODO: { + local $TODO = 'Permission bits inconsistent under cygwin' if $^O eq 'cygwin'; + is (__$perm3, __$c_perm3, "Permission bits not modified"); + } + } + umask $old_mask or die $!; + + # Clean up. + ! -e $_ or unlink $_ or die $! for $src, $copy1, $copy2, $copy3; }