X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2Fopen.t;h=1bf7dca3777d1ca328fe6bb0e211158a78a2cfdd;hb=026810a801d7c3ea221e8fa327f65fdbab0a69bd;hp=15a38bc99297ddf3a1e0d4b887d8b7b1ac2da16e;hpb=b429a72e21c23c9f086d1c98fa5499cd4403a815;p=p5sagit%2Fp5-mst-13.2.git diff --git a/lib/open.t b/lib/open.t index 15a38bc..1bf7dca 100644 --- a/lib/open.t +++ b/lib/open.t @@ -6,7 +6,7 @@ BEGIN { require Config; import Config; } -use Test::More tests => 16; +use Test::More tests => 23; # open::import expects 'open' as its first argument, but it clashes with open() sub import { @@ -18,42 +18,31 @@ ok( require 'open.pm', 'requiring open' ); # this should fail eval { import() }; -like( $@, qr/needs explicit list of disciplines/, +like( $@, qr/needs explicit list of PerlIO layers/, 'import should fail without args' ); -# the hint bits shouldn't be set yet -is( $^H & $open::hint_bits, 0, - 'hint bits should not be set in $^H before open import' ); - # prevent it from loading I18N::Langinfo, so we can test encoding failures -{ - local @INC; - $ENV{LC_ALL} = $ENV{LANG} = ''; - eval { import( 'IN', 'locale' ) }; - like( $@, qr/Cannot figure out an encoding/, - 'no encoding should be found without $ENV{LANG} or $ENV{LC_ALL}' ); -} - my $warn; local $SIG{__WARN__} = sub { $warn .= shift; }; -# and it shouldn't be able to find this discipline -eval{ import( 'IN', 'macguffin' ) }; -like( $warn, qr/Unknown discipline layer/, - 'should warn about unknown discipline with bad discipline provided' ); +# and it shouldn't be able to find this layer +$warn = ''; +eval q{ no warnings 'layer'; use open IN => ':macguffin' ; }; +is( $warn, '', + 'should not warn about unknown layer with bad layer provided' ); + +$warn = ''; +eval q{ use warnings 'layer'; use open IN => ':macguffin' ; }; +like( $warn, qr/Unknown PerlIO layer/, + 'should warn about unknown layer with bad layer provided' ); -# now load a real-looking locale -$ENV{LC_ALL} = ' .utf8'; -import( 'IN', 'locale' ); -is( ${^OPEN}, ":utf8\0", - 'should set a valid locale layer' ); +# open :locale logic changed since open 1.04, new logic +# difficult to test portably. -# and see if it sets the magic variables appropriately +# see if it sets the magic variables appropriately import( 'IN', ':crlf' ); -ok( $^H & $open::hint_bits, - 'hint bits should be set in $^H after open import' ); is( $^H{'open_IN'}, 'crlf', 'should have set crlf layer' ); # it should reset them appropriately, too @@ -62,16 +51,16 @@ is( $^H{'open_IN'}, 'raw', 'should have reset to raw layer' ); # it dies if you don't set IN, OUT, or IO eval { import( 'sideways', ':raw' ) }; -like( $@, qr/Unknown discipline class/, 'should croak with unknown class' ); +like( $@, qr/Unknown PerlIO layer class/, 'should croak with unknown class' ); # but it handles them all so well together import( 'IO', ':raw :crlf' ); is( ${^OPEN}, ":raw :crlf\0:raw :crlf", - 'should set multi types, multi disciplines' ); + 'should set multi types, multi layer' ); is( $^H{'open_IO'}, 'crlf', 'should record last layer set in %^H' ); SKIP: { - skip("no perlio, no :utf8", 4) unless $Config{'useperlio'}; + skip("no perlio, no :utf8", 12) unless (find PerlIO::Layer 'perlio'); eval <:utf8", "b"; - $ok = $a = 0; - for (@a) { - unless ( - ($c = syswrite(G, $_, 1)) == 1 && - systell(G) == ($a += bytes::length($_)) - ) { - print '# ord($_) == ', ord($_), "\n"; - print '# bytes::length($_) == ', bytes::length($_), "\n"; - print '# systell(G) == ', systell(G), "\n"; - print '# $a == ', $a, "\n"; - print '# $c == ', $c, "\n"; - print "not "; - last; - } - $ok++; + sub diagnostics { + print '# ord($_) == ', ord($_), "\n"; + print '# bytes::length($_) == ', bytes::length($_), "\n"; + print '# systell(G) == ', systell(G), "\n"; + print '# $a == ', $a, "\n"; + print '# $c == ', $c, "\n"; } - close G; - ok($ok == @a, - "on :utf8 streams syswrite() should work on characters, not bytes"); - open G, "<:utf8", "b"; - $ok = $a = 0; - for (@a) { - unless ( - ($c = sysread(G, $b, 1)) == 1 && - length($b) == 1 && - ord($b) == ord($_) && - systell(G) == ($a += bytes::length($_)) - ) { - print '# ord($_) == ', ord($_), "\n"; - print '# ord($b) == ', ord($b), "\n"; - print '# length($b) == ', length($b), "\n"; - print '# bytes::length($b) == ', bytes::length($b), "\n"; - print '# systell(G) == ', systell(G), "\n"; - print '# $a == ', $a, "\n"; - print '# $c == ', $c, "\n"; - last; + + my %actions = ( + syswrite => sub { syswrite G, shift; }, + 'syswrite len' => sub { syswrite G, shift, 1; }, + 'syswrite len pad' => sub { + my $temp = shift() . "\243"; + syswrite G, $temp, 1; }, + 'syswrite off' => sub { + my $temp = "\351" . shift(); + syswrite G, $temp, 1, 1; }, + 'syswrite off pad' => sub { + my $temp = "\351" . shift() . "\243"; + syswrite G, $temp, 1, 1; }, + ); + + foreach my $key (sort keys %actions) { + # syswrite() on should work on characters, not bytes + open G, ">:utf8", "b"; + + print "# $key\n"; + $ok = $a = 0; + for (@a) { + unless ( + ($c = $actions{$key}($_)) == 1 && + systell(G) == ($a += bytes::length($_)) + ) { + diagnostics(); + last; + } + $ok++; } - $ok++; + close G; + ok($ok == @a, + "on :utf8 streams syswrite() should work on characters, not bytes"); + + open G, "<:utf8", "b"; + $ok = $a = 0; + for (@a) { + unless ( + ($c = sysread(G, $b, 1)) == 1 && + length($b) == 1 && + ord($b) == ord($_) && + systell(G) == ($a += bytes::length($_)) + ) { + print '# ord($_) == ', ord($_), "\n"; + print '# ord($b) == ', ord($b), "\n"; + print '# length($b) == ', length($b), "\n"; + print '# bytes::length($b) == ', bytes::length($b), "\n"; + print '# systell(G) == ', systell(G), "\n"; + print '# $a == ', $a, "\n"; + print '# $c == ', $c, "\n"; + last; + } + $ok++; + } + close G; + ok($ok == @a, + "checking syswrite() output on :utf8 streams by reading it back in"); } - close G; - ok($ok == @a, - "checking syswrite() output on :utf8 streams by reading it back in"); +} +SKIP: { + skip("no perlio", 2) unless (find PerlIO::Layer 'perlio'); + skip("no Encode", 2) unless $Config{extensions} =~ m{\bEncode\b}; + + eval q[use Encode::Alias;use open ":std", ":locale"]; + is($@, '', 'can use :std and :locale'); + + use open IN => ':non-existent'; + eval { + require Symbol; # Anything that exists but we havn't loaded + }; + like($@, qr/Can't locate Symbol|Recursive call/i, + "test for an endless loop in PerlIO_find_layer"); } END {