This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Tidy t/io/binmode.t
authorNicholas Clark <nick@ccl4.org>
Sun, 6 Mar 2011 21:21:10 +0000 (21:21 +0000)
committerNicholas Clark <nick@ccl4.org>
Sun, 6 Mar 2011 22:48:15 +0000 (22:48 +0000)
Remove redundant C<use Config;>
If C<find PerlIO::Layer 'perlio'> fails, use skip() rather than ok() to skip.
Make the indenting on 2 inconsistent lines consistent with the rest of the file.
Die if Errno can't be loaded, unless it's miniperl.

t/io/binmode.t

index 41eff4a..473261e 100644 (file)
@@ -4,39 +4,37 @@ BEGIN {
     chdir 't' if -d 't';
     @INC = qw(. ../lib);
     require './test.pl';
+    eval 'use Errno';
+    die $@ if $@ and !is_miniperl();
 }
 
-use Config;
-BEGIN {
-    eval {require Errno; Errno->import;};
-}
+
 plan(tests => 9);
 
 ok( binmode(STDERR),            'STDERR made binary' );
-if (find PerlIO::Layer 'perlio') {
-  ok( binmode(STDERR, ":unix"),   '  with unix discipline' );
-} else {
-  ok(1,   '  skip unix discipline without PerlIO layers' );
+SKIP: {
+    skip('skip unix discipline without PerlIO layers', 1)
+       unless find PerlIO::Layer 'perlio';
+    ok( binmode(STDERR, ":unix"),   '  with unix discipline' );
 }
 ok( binmode(STDERR, ":raw"),    '  raw' );
 ok( binmode(STDERR, ":crlf"),   '  and crlf' );
 
 # If this one fails, we're in trouble.  So we just bail out.
 ok( binmode(STDOUT),            'STDOUT made binary' )      || exit(1);
-if (find PerlIO::Layer 'perlio') {
-  ok( binmode(STDOUT, ":unix"),   '  with unix discipline' );
-} else {
-  ok(1,   '  skip unix discipline without PerlIO layers' );
+SKIP: {
+    skip('skip unix discipline without PerlIO layers', 1)
+       unless find PerlIO::Layer 'perlio';
+    ok( binmode(STDOUT, ":unix"),   '  with unix discipline' );
 }
 ok( binmode(STDOUT, ":raw"),    '  raw' );
 ok( binmode(STDOUT, ":crlf"),   '  and crlf' );
 
 SKIP: {
-    skip "minitest", 1 if $ENV{PERL_CORE_MINITEST};
-    skip "no EBADF", 1 if (!exists &Errno::EBADF);
+    skip "no EBADF", 1 unless exists &Errno::EBADF;
 
     no warnings 'io', 'once';
     $! = 0;
     binmode(B);
-    ok($! == &Errno::EBADF);
+    cmp_ok($!, '==', Errno::EBADF());
 }