Re: [REPATCH MANIFEST ext/B/t/o.t] Add tests for O

Message-ID: <20020510144325.26245.qmail@firewheel>

p4raw-id: //depot/perl@16535

diff --git a/MANIFEST b/MANIFEST
index 023f4e2..ef91ab6 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -88,6 +88,7 @@ ext/B/defsubs_h.PL    Generator for constant subroutines
 ext/B/Makefile.PL      Compiler backend makefile writer
 ext/B/NOTES            Compiler backend notes
 ext/B/O.pm             Compiler front-end module (-MO=...)
+ext/B/t/o.t            See if O works
 ext/B/ramblings/cc.notes       Compiler ramblings: notes on CC backend
 ext/B/ramblings/curcop.runtime Compiler ramblings: notes on curcop use
 ext/B/ramblings/flip-flop      Compiler ramblings: notes on flip-flop

diff --git a/ext/B/t/o.t b/ext/B/t/o.t
new file mode 100644 (file)
index 0000000..6575180
--- /dev/null
+++ b/ext/B/t/o.t
@@ -0,0 +1,76 @@
+#!./perl -w
+
+BEGIN {
+       chdir 't' if -d 't';
+       @INC = ('../lib', 'lib', '.');
+       require 'test.pl';
+}
+
+use strict;
+use Config;
+use File::Spec;
+use File::Path;
+
+my $path = File::Spec->catdir( 'lib', 'B' );
+unless (-d $path) {
+       mkpath( $path ) or skip_all( 'Cannot create fake module path' );
+}
+
+my $file = File::Spec->catfile( $path, 'success.pm' );
+local *OUT;
+open(OUT, '>', $file) or skip_all( 'Cannot write fake backend module');
+print OUT while <DATA>;
+close *OUT;
+
+plan( 9 ); # And someone's responsible.
+
+# use() makes it difficult to avoid O::import()
+require_ok( 'O' );
+
+my @args = ('-Ilib', '-MO=success,foo,bar', '-e', '1' );
+my @lines = get_lines( @args );
+
+is( $lines[0], '-e syntax OK', 'O.pm should not munge perl output without -qq');
+is( $lines[1], 'Compiling!', 'Output should not be saved without -q switch' );
+is( $lines[2], '(foo) <bar>', 'O.pm should call backend compile() method' );
+is( $lines[3], '[]', 'Nothing should be in $O::BEGIN_output without -q' );
+
+$args[1] = '-MO=-q,success,foo,bar';
+@lines = get_lines( @args );
+isnt( $lines[1], 'Compiling!', 'Output should not be printed with -q switch' );
+
+SKIP: {
+       skip( '-q redirection does not work without PerlIO', 2)
+               unless $Config{useperlio};
+       is( $lines[2], "[Compiling!", '... but should be in $O::BEGIN_output' );
+
+       $args[1] = '-MO=-qq,success,foo,bar';
+       @lines = get_lines( @args );
+       is( scalar @lines, 3, '-qq should suppress even the syntax OK message' );
+}
+
+$args[1] = '-MO=success,fail';
+@lines = get_lines( @args );
+like( $lines[0], qr/fail at .eval/,
+       'O.pm should die if backend compile() does not return a subref' );
+
+sub get_lines {
+       split(/[\r\n]+/, runperl( args => [ @_ ], stderr => 1 ));
+}
+
+END {
+       1 while unlink($file);
+}
+
+__END__
+package B::success;
+
+print "Compiling!\n";
+
+sub compile {
+       return 'fail' if ($_[0] eq 'fail');
+       print "($_[0]) <$_[1]>\n";
+       return sub { print "[$O::BEGIN_output]\n" };
+}
+
+1;