From: chromatic <chromatic@wgz.org>
Date: Fri, 10 May 2002 07:43:25 +0000 (-0700)
Subject: Re: [REPATCH MANIFEST ext/B/t/o.t] Add tests for O
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=d4e33868a5dde8b2762a447652bbf3dc326c19ff;p=p5sagit%2Fp5-mst-13.2.git

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
--- 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
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;